[PATCH] aiaiai: allow defining checkpatch.pl location
Artem Bityutskiy
dedekind1 at gmail.com
Mon Feb 3 06:26:37 EST 2014
On Mon, 2014-02-03 at 12:39 +0200, Artem Bityutskiy wrote:
> From: Jacob Keller <jacob.e.keller at intel.com>
>
> Instead of hardcoding the checkpatch.pl script into the repository, use the
> in-kernel script by default unless the user specifies a path with
> --checkpatch option. Additionally add an email.cfg option for specifying
> checkpatch.pl per project.
>
> A future patch will remove the external checkpatch.pl binary included in this
> project.
>
> Signed-off-by: Jacob Keller <jacob.e.keller at intel.com>
This looks generally like a good idea to me.
However, I think that we should do something similar for the coccinelle
scripts. Indeed, why carrying a private copy of them?
Doing this only for checkpatch.pl would be an incomplete step.
So we'd also need an option for coccinelle, similar to the --checkpatch
that you introduce.
And my thinking that may be it would be better to introduce a --upstream
option which would specify the upstream branch name, or tag, or commit
ID, or another refspec. Aiaiai would pick checkpatch.pl and coccinelle
scripts from there.
Or it could be a path to the upstream tree. And Aiaiai would pick
checkpatch.pl and coccinelle scripts from there.
I am not sure which is better. The latter is certainly easier to
implement.
I can try to implement one of these, which one do you prefer? I find
both useful, but would vote for the second one.
Thanks!
--
Best Regards,
Artem Bityutskiy
More information about the aiaiai
mailing list