In app-admin/grubconfig-0.5.1 (0.4 and 0.5 too) sys-boot/grub stand in DEPEND, not in REDEPEND. It runs but I think it's more correct to have it in RDEPEND because certainly grub is needed to run grubconfig ;) Also fixed emake in src_compile() to emake || die "Install failed" Patch and new ebuild attached
Created attachment 38308 [details, diff] Patch to fix (R)DEPEND and emake
Created attachment 38310 [details] The new ebuild with patch applied
Sorry. I mean grubconf not gruconfig
To be honest, this is probably still wrong. I've just been testing grubconf on amd64 and it works great; the problem here is that amd64 users who don't use multilib, plus x86 users who don't want to compile their own grub, will instead need this dependency to be on grub-static. I'm not too au fait with "or" dependencies yet, but I'll have a look see what I can find in the tree and perhaps copy what someone else has done. In the meantime, if you can come up with a way to make this dependency "grub or grub-static" without pulling in grub when grub-static is installed, then feel free to post a new ebuild patch - it would be much appreciated.
Maybe we could introduce something like PROVIDES="virtual/grub" in sys-boot/grub and sys-boot/grub-static and then have "virtual/grub" in (R)DEPEND instead of app-admin/grubconf How about that?
Another possibility would be to introduce an internal (or maybe external) USE flag grub-static an then have something like grub-static? (sys-boot/grub-static) !grub-static (sys-boot/grub) in (R)DEPEND What do you think is better?
I really don't know which is better. When I get back from Birmingham (I'm here on business until Sunday) I'll look into this one more closely to see if anyone else has come across a similar issue before.
Okay, the fix is quite simple; I'll implement it after the England match ;-p
Oops, I forgot - fixed in CVS, please test.
