Re: linux-2.6 piuparts error
* maximilian attems | 2009-11-17 11:10:46 [+0100]:
>0m54.3s ERROR: FAIL: Package purging left files on system:
> /etc/kernel-img.conf not owned
oh
>so either it is a failure to generate such a file in postinst (as we
>shouldn't really need it!?), or one would want to remove it on purge,
>which opens another can of worms as this file is unowned in real boxes
>and usualy created by d-i.
>
>the best argument is that due to the convulated questions postinst
>wanted to ask you in the past we had never reached that point of
>generating this conffile in postinst.
What is the long term goal? Do you plan do to get rid of kernel-img.conf
and let the individual boot loader handle this kind of things?
You are changing the default behavior of palo here. However the values
are written very late. link_in_boot for instance is only evalueted at
the begin of the script. So it should not really matter for postinst at
least.
The do_symlink thing is only required by people that don't have their
setup right, isn't it? So grub refers to /vmlinuz but kernel itself is
not in / but in /boot. So if those people would fix their boxes we could
get rid of do_symlink for instance right?
>
>diff --git a/linux-2.6/debian/templates/temp.image.plain/postinst b/linux-2.6/debian/templates/temp.image.plain/postinst
>index ae8c797..43dba4d 100755
>--- a/linux-2.6/debian/templates/temp.image.plain/postinst
>+++ b/linux-2.6/debian/templates/temp.image.plain/postinst
>@@ -478,24 +478,10 @@ sub handle_missing_link {
> die("Failed to copy " . $realimageloc . "$image_name to "
> . $image_dest . "$kimage .\n");
> }
>- }
>- else {
>+ } else {
> if (! $have_conffile) {
> # current default
> $do_symlink = "Yes";
>-
>- if (open(CONF, ">$CONF_LOC")) {
>- print CONF "# Kernel Image management overrides\n";
>- print CONF "# See kernel-img.conf(5) for details\n";
>- if ($loader =~ /palo/i) {
>- print CONF "link_in_boot = Yes\n";
>- print CONF "do_symlinks = Yes\n";
>- print CONF "relative_links = Yes\n";
>- print CONF "do_bootloader = No\n";
>- }
>- close CONF;
>- }
>- $have_conffile = "Yes";
> }
> }
>
Sebastian
Reply to: