Re: [yocto] [patchtest-oe][PATCH] test_patch_cve.py: fix cve tag checking logic

2018-11-09 Thread Michael Halstead
When updating patchtest-oe to include the CVE fixes I also cleaned up repositories in the share directory removing the patchwork credentials in the process. I've restored the patchwork credentials and posted the test results from local backups. I've opened a bug to collect ideas for testing patcht

Re: [yocto] [patchtest-oe][PATCH] test_patch_cve.py: fix cve tag checking logic

2018-11-08 Thread Mittal, Anuj
On Wed, 2018-11-07 at 09:01 +, Richard Purdie wrote: > On Fri, 2018-11-02 at 14:03 +0800, Chen Qi wrote: > > The current logic for checking cve tag is not correct. It errors > > out if and only if the patch contains a line which begins with > > CVE-- and contains nothing else. > > > >

Re: [yocto] [patchtest-oe][PATCH] test_patch_cve.py: fix cve tag checking logic

2018-11-07 Thread Richard Purdie
On Fri, 2018-11-02 at 14:03 +0800, Chen Qi wrote: > The current logic for checking cve tag is not correct. It errors > out if and only if the patch contains a line which begins with > CVE-- and contains nothing else. > > It will not error out if the patch contains no CVE information, nor >

Re: [yocto] [patchtest-oe][PATCH] test_patch_cve.py: fix cve tag checking logic

2018-11-07 Thread Richard Purdie
On Fri, 2018-11-02 at 14:03 +0800, Chen Qi wrote: > The current logic for checking cve tag is not correct. It errors > out if and only if the patch contains a line which begins with > CVE-- and contains nothing else. > > It will not error out if the patch contains no CVE information, nor >

[yocto] [patchtest-oe][PATCH] test_patch_cve.py: fix cve tag checking logic

2018-11-01 Thread Chen Qi
The current logic for checking cve tag is not correct. It errors out if and only if the patch contains a line which begins with CVE-- and contains nothing else. It will not error out if the patch contains no CVE information, nor will it error out if the patch contains line like below. 'Fi