CppCheck Report Update

[Maarten Hoes]

Caolán McNamara wrote
> On Sun, 2016-02-28 at 01:35 +0000, 

> cppcheck.libreoffice@

>  
>>> Note:
>>>     The script generating this report was run at :
>>>         2016-28-02_02:35:42 with user buildslave at host vm140 as
>>> /home/buildslave/source/dev-tools/cppcheck/cppcheck-report.sh -s
>>> /home/buildslave/source/libo-core -c /home/buildslave/source/cppcheck
>>> -w /home/buildslave/tmp/www
>>> 
>>>     It can be found and improved here:
>>>         https://gerrit.libreoffice.org/gitweb?p=dev-
>>> tools.git;a=blob;f=cppcheck/cppcheck-report.sh
> 
>> cppcheck-report.sh (since Feb 15) has --inline-suppr enabled in it so I
>> would have expected http://dev-builds.libreoffice.org/cppcheck_reports/
>> master/382.html#line-240 not to appear.
> 
>> Is that inline suppression markup wrong. Or does something need to be
>> done to refresh from git the cppcheck-report.sh used to build these
>> reports ?
>> 
> 
> Hi,
> 
> 
> A little late, and really only by accident that I ran into your post, but
> ...
> 
> 
> The git version of 'cppcheck-report.sh' seems indeed to have been updated
> on 2016-02-15 by you to include the '--inline-suppr' you mentioned.
> However, 'for reasons', the (git) version of the script that is located on
> vm140.documentfoundation.org isnt updated by 'git pull' on a regular
> basis. (I guess a cron job or something similar could be used to update
> the repo on a regular basis). So the older version was still used for
> creating the report. 
> 
> I just logged into vm140 (I still seems to have ssh-key access, which may
> or may not be what was intended) and did a 'git pull' for you, updating
> the repo to the latest version of 'cppcheck-report.sh', so the next run
> should include your changes.
> 
> Hope this helps and I didnt inadvertently mess something up.
> 
> 
> - Maarten
> 
> 
> 
> 
> 
> 
> C.
> _______________________________________________
> LibreOffice mailing list

> LibreOffice at .freedesktop

> https://lists.freedesktop.org/mailman/listinfo/libreoffice





--
View this message in context: http://nabble.documentfoundation.org/CppCheck-Report-Update-tp4176815p4185951.html
Sent from the Dev mailing list archive at Nabble.com.