Even I encountered the same. If you have clang version less than 6, it
exits. Upgrading clang version fixed it.

Regards,
Poornima

On Fri, Oct 5, 2018, 9:58 AM Sachidananda URS <s...@redhat.com> wrote:

>
>
> On Fri, Oct 5, 2018 at 9:45 AM, Raghavendra Gowdappa <rgowd...@redhat.com>
> wrote:
>
>>
>>
>> On Fri, Oct 5, 2018 at 9:34 AM Raghavendra Gowdappa <rgowd...@redhat.com>
>> wrote:
>>
>>>
>>>
>>> On Fri, Oct 5, 2018 at 9:11 AM Kaushal M <kshlms...@gmail.com> wrote:
>>>
>>>> On Fri, Oct 5, 2018 at 9:05 AM Raghavendra Gowdappa <
>>>> rgowd...@redhat.com> wrote:
>>>> >
>>>> >
>>>> >
>>>> > On Fri, Oct 5, 2018 at 8:53 AM Amar Tumballi <atumb...@redhat.com>
>>>> wrote:
>>>> >>
>>>> >> Can you try below diff in your rfc, and let me know if it works?
>>>> >
>>>> >
>>>> > No. it didn't. I see the same error.
>>>> >  [rgowdapp@rgowdapp glusterfs]$ ./rfc.sh
>>>> > + rebase_changes
>>>> > + GIT_EDITOR=./rfc.sh
>>>> > + git rebase -i origin/master
>>>> > [detached HEAD e50667e] 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 0734847] cluster/dht: fixes to unlinking invalid
>>>> linkto file
>>>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>>>> > [detached HEAD 7aeba07] rfc.sh: test - DO NOT MERGE
>>>> >  1 file changed, 8 insertions(+), 3 deletions(-)
>>>> > Successfully rebased and updated refs/heads/1635145.
>>>> > + check_backport
>>>> > + moveon=N
>>>> > + '[' master = master ']'
>>>> > + return
>>>> > + assert_diverge
>>>> > + git diff origin/master..HEAD
>>>> > + grep -q .
>>>> > ++ git log -n1 --format=%b
>>>> > ++ grep -ow -E
>>>> '([fF][iI][xX][eE][sS]|[uU][pP][dD][aA][tT][eE][sS])(:)?[[:space:]]+(gluster\/glusterfs)?(bz)?#[[:digit:]]+'
>>>> > ++ awk -F '#' '{print $2}'
>>>> > + reference=1635145
>>>> > + '[' -z 1635145 ']'
>>>> > ++ clang-format --version
>>>> > + clang_format='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'
>>>>
>>>> This is a pretty old version of clang. Maybe this is the problem?
>>>>
>>>
>>> Yes. That's what I suspected too. Trying to get repos for the upgrade.
>>>
>>
>> But, what's surprising is that script exits.
>>
>
> What is the return code of clang-format? If it is non-zero then script
> will silently exit because that is what
> it is told to do.
>
> `#!/bin/sh -e' means exit on error.
>
> -sac
>
> _______________________________________________
> Gluster-devel mailing list
> Gluster-devel@gluster.org
> https://lists.gluster.org/mailman/listinfo/gluster-devel
_______________________________________________
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel

Reply via email to