Bugzilla Bug 7903

Just so I dont forget..

Somebody looking to familiarize themselves with eclasses or such might want to 
look at making grub and lilo use a consistent pkg_setup() method.

Seems it keeps getting tweaked in one pkg, then the other, would be nice to 
have them share a function or something :/

Not volunteering immediately.  :)

Created an attachment (id=3946) [edit]
boot.eclass

Well, here's an initial import a bit reworked from the lilo pkg_setup().
Works fine here with lilo-22.3.3.ebuild. Maybe you can start CC'ing
people to test and improve this eclass :) azarah, spanky, seemant and others.

Maurizio

Please have also a look at bug #6820 where I submitted an ebuild for 
grub-0.92-r1. This ebuild contains a rewrite of pkg_setup that also takes 
into account the possibility of /boot beeing mounted read-only and remounts
it rw.

Markus

Havent looked yet at Markus Krainer's stuff (didnt feel like going back to
galeon1 and USE=-gtk2 moz to get it working properly), but that in lilo is 
tested, and works well.  We might just add the "ro" stuff to it.

j2ee in #gentoo asked me to comment on this.

not sure why, but here my comment:

yes, i do want this to get completed, that why i reported it as a bug :p
no, i dont want to code it, thats why i hoped somebody else would :p
im happy to see somebody jumped up to it :p

yes, people, please *do* make sure the eclass gets the best parts from lilo and 
grub and make it as good as can be.  i got tired of seeing one package get a 
nice fix, then the other get a fix, then they get out of sync.. you know how 
the rest goes.

so go ahead and commit the sucker, and change the .ebuilds to use it :p

Created an attachment (id=3993) [edit]
boot.eclass 

boot.eclass with support for /boot mounted as ready-only.
thanks Markus.
PLEASE test.

Maurizio aka j2ee

Created an attachment (id=4008) [edit]
boot.eclass updated. PLEASE test and report

Created an attachment (id=4026) [edit]
final (I hope) boot.eclass

this eclass works has been tested with lilo and grub and works fine.
Someone should update the lilo & grub ebuild to include this.

Maurizio

Maurizio,

consider /boot not mounted and the following fstab entry:
/dev/hda1   /boot       ext2        ro,noatime

then your pkg_setup will mount /boot read-only!

Markus

alright i added the last attachment as

  mount-boot.eclass

more testing needed; if you have another fix for this in
the meantime, attach as a patch against the .eclass please...

thank you very much to everybody involved for your continuing
contributions/participation!  i really appreciate it :)

sys-apps/quik <- you think you want to incorporate this eclass into it too ?

Created an attachment (id=4052) [edit]
mount-boot.eclass

markus: tnx, fixed :)
spanky: think so
woodchip: It's your turn now :) Works fine here with both lilo and grub.

Maurizio

sys-apps/memtest86 <- might want to throw into this sucker too :D

if people could try lilo-22.3.3-r1 and comment please.

please, again, if you have a fix, attach it as a unified
diff against the eclass in portage, not the whole eclass,
ive already spent time cleaning whitespace issues, so... :)

markus, did you have an update for grub-0.92, aside from
the eclass addition?  i think you posted an -r1 attachment,
but again i cant easily see atm what your changes were
if any (or just related to the eclass).

hopefully a bunch of these lilo/grub related bugs can be
closed up fairly soon (*yay*).  im aware memtest86 should
use the eclass too ;).

Created an attachment (id=4100) [edit]
oops :) I'm sorry, here's the diff.

Created an attachment (id=4101) [edit]
revised mount-boot.diff

auff .. I was still naming the eclass "boot" instead of "mount-boot"
fixed.

Maurizio

*** Bug 8256 has been marked as a duplicate of this bug. ***

Created an attachment (id=4106) [edit]
grub.diff.tgz

contains grub.ebuild.diff (0.92 to 0.92-r2!) and ChangeLog.diff

okay i've committed a bunch of work on this into CVS.

as for the last attachment re: grub, i'll look at that asap.

any known problems with either grub-0.92 or lilo-22.3.3-r1?
im getting awefully itchy on unmasking both soon.

i merged the last of markus' changes into grub-0.92-r1

that pretty much completes this item.

thanks again to everyone.