[Gluster-devel] ./rfc.sh not pushing patch to gerrit
rgowdapp at redhat.com
Fri Oct 5 08:31:16 UTC 2018
What is the agreed upon clang version for Glusterfs project? Is it clang-6?
On Fri, Oct 5, 2018 at 1:58 PM Raghavendra Gowdappa <rgowdapp at redhat.com>
> clang-4.0.1 pushes patch, but still doesn't understand some keys in
> [rgowdapp at rgowdapp glusterfs]$ ./rfc.sh
> [detached HEAD 401a7b6] cluster/dht: clang-format dht-common.c
> 1 file changed, 10674 insertions(+), 11166 deletions(-)
> rewrite xlators/cluster/dht/src/dht-common.c (88%)
> [detached HEAD 2bf5031] cluster/dht: fixes to unlinking invalid linkto file
> 1 file changed, 236 insertions(+), 268 deletions(-)
> Successfully rebased and updated refs/heads/1635145.
> YAML:35:23: error: unknown key 'SplitEmptyFunction'
> SplitEmptyFunction: true
> On Fri, Oct 5, 2018 at 10:47 AM Raghavendra Gowdappa <rgowdapp at redhat.com>
>> We should document (better still add checks in rfc.sh and warn user to
>> upgrade) that we need clang version x or greater.
>> On Fri, Oct 5, 2018 at 10:45 AM Sachidananda URS <surs at redhat.com> wrote:
>>> On Fri, Oct 5, 2018 at 10:41 AM, Raghavendra Gowdappa <
>>> rgowdapp at redhat.com> wrote:
>>>> General options:
>>>> -help - Display available options (-help-hidden
>>>> for more)
>>>> -help-list - Display list of available options
>>>> (-help-list-hidden for more)
>>>> -version - Display the version of this program
>>>> [rgowdapp at rgowdapp ~]$ clang-format -version ; echo $?
>>>> LLVM (http://llvm.org/):
>>>> LLVM version 3.4.2
>>>> Optimized build.
>>>> Built Dec 7 2015 (09:37:36).
>>>> Default target: x86_64-redhat-linux-gnu
>>>> Host CPU: x86-64
>>> It is a bug then, which they've fixed later on. Upgrade is the only
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Gluster-devel