On Thu, 2014-10-02 at 08:41 -0400, Josh Boyer wrote:
> On Thu, Oct 2, 2014 at 4:49 AM, Paul Bolle <pebolle@xxxxxxxxxx> wrote:
> > Here's a perl script I cobbled together that checks for references to
> > unknown Kconfig macros. Tested on top of current master. Invoke like
> > this:
> > perl "$WHAT_SHALL_I_CALL_THIS.pl" $SOME_DIR/kernel-3.16.fc20/linux-3.17.0-0.rc7.git1.1.fc20.x86_64
> >
> > (I stopped tracking rawhide long ago.)
> >
> > I'll send a patch generated with the aid of this script shortly.
> >
> > It seems overkill to invoke (something like) this every build. But
> > perhaps the package maintainers can include (something like) it before
> > they push changes into master.
>
> I would imagine it being most useful during the merge window kernels.
> Perhaps running it once per -rc build would be sufficient to keep
> things clean.
Thinking some more about this, maybe just once per merge window might do
the trick. Say after -rc3 or -rc4, when all the big changes have landed.
> > The call of "glob("config-*")" is an accident waiting to happen, but I
> > couldn't be bothered to replace it with a call of "git ls-files
> > "config-*"".
> >
> > Have fun!
>
> If you have no objections, I'll likely add this to the scripts/
> directory in the kernel pkg repo.
No, see the note about "public domain".
Note that it should be run from the package root (otherwise
"glob( "config-*" )" will return nothing). Tt would also be nice if it
was automagically supplied with the right build directory as an
argument. That requires some hooks in kernel.spec, I guess. I don't know
how to do that. kernel.spec is a scary thing... But once per merge
window is five or six times a year. That might as well be done manually.
> That way it's there and people that
> can actually read perl can modify it, etc.
... and have laugh or two about the rather peculiar style I have grown
used to. (So peculiar I can't even manage to follow my own whitespace
rules.)
> Have a preference on a
> name? I was thinking "zombie_kconfig.pl".
I used check-configs.pl locally, but your name is funnier.
Paul Bolle
_______________________________________________
kernel mailing list
kernel@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/kernel
