Bug 15913 - [mga6] create a grub2-common or grub2-tools package
Summary: [mga6] create a grub2-common or grub2-tools package
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal minor
Target Milestone: Mageia 6
Assignee: Barry Jackson
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 416
  Show dependency treegraph
 
Reported: 2015-05-12 14:02 CEST by Thierry Vignaud
Modified: 2015-07-11 14:12 CEST (History)
0 users

See Also:
Source RPM: grub2
CVE:
Status comment:


Attachments
split spec (10.41 KB, text/plain)
2015-05-12 23:04 CEST, Barry Jackson
Details
split spec - without a couple of typos ;) (10.43 KB, text/plain)
2015-05-12 23:14 CEST, Barry Jackson
Details

Description Thierry Vignaud 2015-05-12 14:02:13 CEST
we should create a grub2-common or grub2-tools package (like FC does)

Most files are identical (binaries, translation catalogs, most of man pages, /boot/grub2/* ...)

eg: FC split common tools in grub2-tools that is required by both grub2 & grub2-tools:
http://rpmfind.net/linux/rpm2html/search.php?query=grub2-tools&submit=Search+...

Obviously that means grub2-tools should conflicts against older grub2{,-efi} b/c of the files moving

Reproducible: 

Steps to Reproduce:
Thierry Vignaud 2015-05-12 14:02:28 CEST

Blocks: (none) => 416

Comment 1 Barry Jackson 2015-05-12 19:58:39 CEST
I think -common is more meaningful.
So that would be:
grub2 requiring grub2-common
grub2-efi requiring grub2-common
The conflict we currently have between grub2 and grub2-efi would then not be needed, although I have not yet considered the implications of having both installed together.

Is the above what you have in mind?
Comment 2 Thierry Vignaud 2015-05-12 20:37:36 CEST
Using -tools would reduce difference with FC though.
About conflicts, we could add explicit conflict tags to prevent both to be installed.
As the binaries are identical, I guess grub-install looks for either /usr/lib/grub/foobar either though glob() or depending on platform && whether UEFI is enabled or not?
Comment 3 Barry Jackson 2015-05-12 20:47:34 CEST
(In reply to Thierry Vignaud from comment #2)
> Using -tools would reduce difference with FC though.
OK 
> About conflicts, we could add explicit conflict tags to prevent both to be
> installed.
> As the binaries are identical, I guess grub-install looks for either
> /usr/lib/grub/foobar either though glob() or depending on platform &&
> whether UEFI is enabled or not?

Yes - this I'm not certain about, which is why for safety I kept both packages totally separate in the past.

I will build test packages initially without the conflict between grub2 and grub2 -efi to see what happens when both are installed together.

I did wonder about just adding the uefi stuff to the grub2 package, but then the unused stuff would just be bloat.
Comment 4 Barry Jackson 2015-05-12 23:00:51 CEST
OK I have built and tested the split package in both legacy and UEFI  systems with no regressions that I can find.
I have kept the conflicts between grub2 and grub2-efi because of the problems that the respective %posts could do on install of the second package - having thought a little more.

Do you want this committed or shall I wait for Mga6 Cauldron to open.

During testing I accepted rpmnew and noticed that I lost failsafe from/etc/default/grub.
I know that you said this would be handled by drak* but I still feel that we should have GRUB_DISABLE_RECOVERY=false in the default in the package.
Comment 5 Barry Jackson 2015-05-12 23:04:54 CEST
Created attachment 6526 [details]
split spec
Comment 6 Barry Jackson 2015-05-12 23:14:16 CEST
Created attachment 6527 [details]
split spec - without a couple of typos ;)

Attachment 6526 is obsolete: 0 => 1

Comment 7 Thierry Vignaud 2015-05-13 06:11:18 CEST
As stated in summary, this'll wait mga6 :-)
Comment 8 Thierry Vignaud 2015-05-19 09:38:09 CEST
(In reply to Barry Jackson from comment #4)
> Do you want this committed or shall I wait for Mga6 Cauldron to open.

we wait for mga6 Cauldron to open

> During testing I accepted rpmnew and noticed that I lost failsafe
> from/etc/default/grub.

On next drakboot run or on next kernel update, it'll be re-added (see below)

> I know that you said this would be handled by drak* but I still feel that we
> should have GRUB_DISABLE_RECOVERY=false in the default in the package.

Whatever is the default, we don't care as drakboot will add it :-) :
http://gitweb.mageia.org/software/drakx/tree/perl-install/bootloader.pm#n1798
Comment 9 Barry Jackson 2015-05-19 10:10:29 CEST
(In reply to Thierry Vignaud from comment #8)
 
> Whatever is the default, we don't care as drakboot will add it :-) :
> http://gitweb.mageia.org/software/drakx/tree/perl-install/bootloader.pm#n1798

Some users who decide to set it true (like me) will not be too happy about a kernel update setting it back to false again.

FWIW I still think this should have been set by grub2 in /etc/default/grub on grub2 install (to false) and then left to the user to edit at will.
IMHO it's not correct to hard code and overwrite a config like this.

I appreciate that only new installs would have had the failsafe entries and not upgraded systems, but anyone wanting it could have easily added it.
Samuel Verschelde 2015-06-06 15:58:20 CEST

Target Milestone: --- => Mageia 6

Comment 10 Barry Jackson 2015-07-11 14:12:20 CEST
Fixed in Cauldron (Mga6)

Status: NEW => RESOLVED
Resolution: (none) => FIXED


Note You need to log in before you can comment on or make changes to this bug.