[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#486725: wasted code in .config, I'll run lilo myself always



Package: linux-image-2.6.25-2-686
Version: 2.6.25-5

Excuse me, but in linux-image-2.6.25-2-686.config,
my $loader            = ""; # lilo, silo, quik, palo...
my $Loader          = "NoLOADER"; #
$Loader             = "LILO"     if $loader =~ /^lilo/io;
how will the third line ever fire, as you already set $loader to an
empty string, and there is nothing intervening to change it.

Yes, maybe you change it somewhere else, but at least here you must
admit you have wasted code.

Also looking at perlop(1), it looks like /o isn't needed for this kind
of regexp, but of course doesn't hurt either.

Also $explicit_do_loader is not documented in kernel-img.conf.5.gz.
OK, maybe it is a secret option. I dare not use it.

Also kernel-img.conf.5.gz is not included in the package, but in a
non-pure-doc different package. Bad policy.

Anyway, all I know is using plain dpkg -i to upgrade the kernel this
time, I could not reboot.

That is why I now do
  #kernel-img.conf.5.gz says do_bootloader defaults to yes, but then
  #why wasn't the /boot/map file updated? So:
  test "$(find /boot -type f -cnewer /boot/map)" && lilo
in my shutdown script, lest I not be able to boot tomorrow.




Reply to: