Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 23246 - lilo-22.5.1.ebuild does not umount boot partition
Summary: lilo-22.5.1.ebuild does not umount boot partition
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Core system (show other bugs)
Hardware: All Linux
: High normal
Assignee: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 40562 53413
  Show dependency tree
 
Reported: 2003-06-21 15:48 UTC by Carsten Lohrke (RETIRED)
Modified: 2004-06-09 09:59 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Carsten Lohrke (RETIRED) gentoo-dev 2003-06-21 15:48:32 UTC
the ebuild checks if /sbin/lilo fails, if not there is no reason to let the boot
partition still be mounted

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Donny Davies (RETIRED) gentoo-dev 2003-07-03 02:07:39 UTC
awwww isnt that cute.
Comment 2 Carsten Lohrke (RETIRED) gentoo-dev 2003-11-20 16:46:36 UTC
Mhh, just having a look at my open bug reports.

I suppose this "bug" is at least as cute as you Donny, heh!? ;)
If you don't think so, please mark it wontfix and forget it.
Comment 3 Donny Davies (RETIRED) gentoo-dev 2003-11-21 16:28:04 UTC
Carlo, I dunno, I just dont really feel like working on it.

Feel free to attach your idea of what the patch should be, if you want.

I didnt close it wontfix as I see some merit to what you're saying.
Comment 4 Tony Vroon (RETIRED) gentoo-dev 2004-06-07 12:22:21 UTC
-r4 will run dolilo in the ebuild, the /boot partition will be properly restored to it's previous state (be it unmounted or mounted r/o).

Will be fixed once -r4 hits the tree.
Comment 5 Tony Vroon (RETIRED) gentoo-dev 2004-06-09 09:59:28 UTC
The lilo-22.5.9 ebuild runs the dolilo wrapper, which restores the original state of /boot after running lilo. Be it unmounted, mounted r/o or mounted r/w.
This ebuild is in CVS and should appear in the tree soon.