In March, I enhanced the genkernel code that copies the kernel build process work products to /boot to include symlink handling like that implemented in the manual kernel build "make install" process. Chris Gianelloni was kind enough to accept the enhancements and make them part of genkernel-3.4.8. Unfortunately, I introduced a logic flaw in the symlink tracking code that, on a kernel version change, results in the new kernel (and other work products) being copied into a file named after the previously compiled kernel version. This flaw only manifests when genkernel is being asked to manage symlinks (with the --symlink command line option or with SYMLINK="yes" genkernel.conf). I had also been working on other modest changes to genkernel before I discovered this rather embarassing flaw. They were too modest to release yet otherwise. Therefore, I'm submitting two patches. The first one fixes my --symlink bug. The second one does several small things: * Adds logic so that the --oldconfig command line option can be defaulted in genkernel.conf. * Adds logic so that the --tempdir command line option can be defaulted in genkernel.conf. Also adds missing documentation to the man page for the --tempdir command line option. This closes bug #144703. * Corrects the man page's description of the --debugfile command line option. This closes bug #150697. Reproducible: Sometimes Steps to Reproduce: 1. Emerge new version of kernel sources. 2. Use genkernel with the --symlink and --install options to compile the new kernel. 3. New kernel binary (and initrd and System.map) will be named incorrectly in /boot.
Created attachment 120554 [details, diff] Patch addresses --symlink bug.
Created attachment 120556 [details, diff] Patch corrects other minor issues (and closes two other bugs).
My bad again. This issue is "Reproducible Always" with the previously mentioned preconditions.
Thanks! I've added both patches to SVN and they'll show up in the next version of genkernel.
Created attachment 125824 [details, diff] Patch, additive to last one, fixes other --symlink issues. :( You guys are going to scrag me. I ran into another flaw in my --symlink code, which has prompted me to embark on a massive regression test. It works correctly in *all* corner cases now. I apologize profusely, but this is the *last* time I'll have to revisit this minor issue. - John
OK. I just added the latest patch to SVN so hopefully we're all done now. If not, please file up a new bug to keep the confusion down on my end. Thanks for the patch... =]
Please test genkernel 3.4.9_prer1 or better. This should be fixed now.
