On Mon, Nov 02, 2015 at 10:28:35AM +0000, Van Haaren, Harry wrote: > > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > > Subject: Re: [dpdk-dev] [PATCH 1/2] scripts: add checkpatch wrapper > > > > 2015-10-30 16:16, Bruce Richardson: > > > Another request, can you perhaps also fix the script for situations where > > > checkpatch.pl is not in the kernel tree. > > > <snip> > > > OK > > I prefer using it in the Linux tree because it is updated with "git pull". > > I would like to suggest including the checkpatch.pl script itself in the dpdk > tree, as this would ensure that we are all running the exact same version of > checkpatch. > > My previous patchset had errors that I had not detected because I ran an > older checkpatch.pl, and I think there are others who have similar issues > that the checkpatch version provides more/less errors. > > If included in the repo, we would all automatically upgrade when the next > checkpatch.pl is merged - providing consistency. > > -Harry
+1 /Bruce