Re: [Gluster-devel] Clang failures update
Few updates for people interested to solve the clang issues.
- Use Bugzilla ID: 1622665 in your commits
- https://bugzilla.redhat.com/show_bug.cgi?id=1622665
- If you are interested in fixing the issue check for latest issues @
https://build.gluster.org/job/clang-scan/lastCompletedBuild/clangScanBuildBugs/
- And update your name as owner for issues listed @
https://docs.google.com/spreadsheets/d/1Wmx3C5IhZ89X_mITGQIbWZUtiXWN_taEZmRyI6WM5r0/edit?usp=sharing
(everyone has comment access for now, ping me and I can add you to edit
access / or use 'suggest' option).
Lets see if eventually we get to ~0 defects :-)
On Fri, Aug 10, 2018 at 2:21 PM, Amar Tumballi wrote:
>
>
> On Fri, Aug 10, 2018 at 1:50 PM, Nigel Babu wrote:
>
>> Hello folks,
>>
>> Based on Yaniv's feedback, I've removed deadcode.DeadStores checker. We
>> are left with 161 failures. I'm going to move this to 140 as a target for
>> now. The job will continue to be yellow and we need to fix at least 21
>> failures by 31 Aug. That's about 7 issues per week to fix.
>>
>
> Can we try to do 130 as the target till Aug 31st? That means 10 issues per
> week, ie, 2 per working day. Should be possible to achieve IMO :-)
>
> -Amar
>
>
>>
>> If anyone wants me to change the goal posts for this one, please let me
>> know.
>>
>>
>> If you want to run this on your local Fedora 27 machine, it should work
>> fine.
>>
>
> If you want to run this on a Fedora 28 machine, you'll need to do a
>> little bit of a hack. Search for PYTHONDEV_CPPFLAGS in configure.ac and
>> add this line right below the existing line:
>>
>> PYTHONDEV_CPPFLAGS=`echo ${PYTHONDEV_CPPFLAGS} | sed -e
>> 's/-fcf-protection//g'`
>>
>> Fedora 28 has GCC 8.0 and clang 7.0, this is the root cause of this
>> failure and in a future version, this should work without the need for this
>> hack.
>>
>> --
>> nigelb
>>
>> ___
>> Gluster-devel mailing list
>> Gluster-devel@gluster.org
>> https://lists.gluster.org/mailman/listinfo/gluster-devel
>>
>
>
>
> --
> Amar Tumballi (amarts)
>
--
Amar Tumballi (amarts)
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel
Re: [Gluster-devel] Clang failures update
On Fri, Aug 10, 2018 at 1:50 PM, Nigel Babu wrote:
> Hello folks,
>
> Based on Yaniv's feedback, I've removed deadcode.DeadStores checker. We
> are left with 161 failures. I'm going to move this to 140 as a target for
> now. The job will continue to be yellow and we need to fix at least 21
> failures by 31 Aug. That's about 7 issues per week to fix.
>
Can we try to do 130 as the target till Aug 31st? That means 10 issues per
week, ie, 2 per working day. Should be possible to achieve IMO :-)
-Amar
>
> If anyone wants me to change the goal posts for this one, please let me
> know.
>
>
> If you want to run this on your local Fedora 27 machine, it should work
> fine.
>
If you want to run this on a Fedora 28 machine, you'll need to do a
> little bit of a hack. Search for PYTHONDEV_CPPFLAGS in configure.ac and
> add this line right below the existing line:
>
> PYTHONDEV_CPPFLAGS=`echo ${PYTHONDEV_CPPFLAGS} | sed -e
> 's/-fcf-protection//g'`
>
> Fedora 28 has GCC 8.0 and clang 7.0, this is the root cause of this
> failure and in a future version, this should work without the need for this
> hack.
>
> --
> nigelb
>
> ___
> Gluster-devel mailing list
> Gluster-devel@gluster.org
> https://lists.gluster.org/mailman/listinfo/gluster-devel
>
--
Amar Tumballi (amarts)
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel
[Gluster-devel] Clang failures update
Hello folks,
Based on Yaniv's feedback, I've removed deadcode.DeadStores checker. We are
left with 161 failures. I'm going to move this to 140 as a target for now.
The job will continue to be yellow and we need to fix at least 21 failures
by 31 Aug. That's about 7 issues per week to fix.
If anyone wants me to change the goal posts for this one, please let me
know.
If you want to run this on your local Fedora 27 machine, it should work
fine. If you want to run this on a Fedora 28 machine, you'll need to do a
little bit of a hack. Search for PYTHONDEV_CPPFLAGS in configure.ac and add
this line right below the existing line:
PYTHONDEV_CPPFLAGS=`echo ${PYTHONDEV_CPPFLAGS} | sed -e
's/-fcf-protection//g'`
Fedora 28 has GCC 8.0 and clang 7.0, this is the root cause of this failure
and in a future version, this should work without the need for this hack.
--
nigelb
___
Gluster-devel mailing list
Gluster-devel@gluster.org
https://lists.gluster.org/mailman/listinfo/gluster-devel
