[dpdk-ci] check patch status duplicated in mailing list

Liu, Yong yong.liu at intel.com
Wed Nov 16 02:35:03 CET 2016


> -----Original Message-----
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Tuesday, November 15, 2016 5:24 PM
> To: Liu, Yong
> Cc: ci at dpdk.org
> Subject: Re: check patch status duplicated in mailing list
> 
> 2016-11-15 03:32, Liu, Yong:
> > Hi Thomas,
> > There're two checkpatch report in this mailing now. One of them is
> included in patchset status report.
> > These two reports are focusing on the same thing and sometimes
> mismatched in the result.
> > I guess it is caused by different version of checkpatch scripts have
> different rules, ours version is 0.32.
> >
> > Since newer checkpatch report has been integrated with patch work system.
> I suggest to remove checkpatch in patchset report. Looking for your option.
> 
> Yes I was going to suggest to remove checkpatch from Intel checks.
> 
> Also could we improve the check description?
> 	Test-Label: Intel Niantic on Fedora
> If it is doing only some compilation checks, it should be
> 	Test-Label: Intel compilation on Fedora
> 

Sure, we will change the description for patch set build check. 
BTW, we have enhanced our patch set build check from only build once to build with each patch. 
This new report will take the place of the existing one in the near future.
We will look for you suggestions on it:)

> Later this week, I would like to share the scripts used for checkpatch
> so that we can have the Intel checks integrated in patchwork.
> 
> PS: moving this discussion on ci at dpdk.org
> We should post only the test reports on test-report at dpdk.org
> and discuss on ci at dpdk.org.


More information about the ci mailing list