Summary: | small eclass task -> bootloader.eclass | ||
---|---|---|---|
Product: | Portage Development | Reporter: | Donny Davies (RETIRED) <woodchip> |
Component: | Unclassified | Assignee: | Donny Davies (RETIRED) <woodchip> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | andrei.ivanov, azarah, h3y, mkrainer, vapier |
Priority: | High | ||
Version: | unspecified | ||
Hardware: | x86 | ||
OS: | All | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 8256 | ||
Attachments: |
boot.eclass
boot.eclass boot.eclass updated. PLEASE test and report final (I hope) boot.eclass mount-boot.eclass oops :) I'm sorry, here's the diff. revised mount-boot.diff grub.diff.tgz |
Description
Donny Davies (RETIRED)
2002-09-13 23:41:06 UTC
Created attachment 3946 [details]
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 attachment 3993 [details]
boot.eclass
boot.eclass with support for /boot mounted as ready-only.
thanks Markus.
PLEASE test.
Maurizio aka j2ee
Created attachment 4008 [details]
boot.eclass updated. PLEASE test and report
Created attachment 4026 [details]
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 attachment 4052 [details]
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 attachment 4100 [details]
oops :) I'm sorry, here's the diff.
Created attachment 4101 [details]
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 attachment 4106 [details]
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. |