Bug 15914 - [mga6] move 93_memtest from grub2 to memtest86+ (like Fedora does)
Summary: [mga6] move 93_memtest from grub2 to memtest86+ (like Fedora does)
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-11-16 01:37 CET (History)
0 users

See Also:
Source RPM: grub2, memtest86+
CVE:
Status comment:


Attachments

Description Thierry Vignaud 2015-05-12 14:02:15 CEST
/etc/grub.d/93_memtest should be in memtest86+, not ingrub2 (like FC does).

With proper conflicts tags in grub2 on memtest86+ and vice versa b/c of the file move

Reproducible: 

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

Blocks: (none) => 416

Comment 1 Barry Jackson 2015-05-12 23:39:45 CEST
I'm a bit puzzled by this.

Why should a file that is part of grub2 be placed in a totally unrelated package?

How would memtest86+ get the file if grub2 is not installed?
/etc/grub.d is owned by grub2.

As it is, grub2 requires memtest86+ and has this script to incorporate it in the menu. 
Memtest86+ cannot require grub2.

I'm totally lost. I don't understand the reason for this or how it can be accomplished.

I'm probably missing something here :\

Please explain ;)
David Walser 2015-05-13 02:25:04 CEST

Summary: [mga6] move 93_memtest from grub2 to memtest86+ (like FC does) => [mga6] move 93_memtest from grub2 to memtest86+ (like Fedora does)

Comment 2 Thierry Vignaud 2015-05-13 11:53:29 CEST
1) for your later remark: I fail to see where grub2 requires memetest86+.
If it were true, it would a bug (grub2 should not requires memtest86+ as it doesn't need it in order to work).

2) The 93_memtest file is not part of grub2.
It's a config file for starting memtest86+ from grub2.
And no, we don't want memtest86+ to requires grub2.

What we want is to place this config file in the right package.
When it's installed, memtest86+ inserts an entry in the bootloader by calling "bootloader-config"
For grub2, that doesn't work as grub2 doesn't work the lilo/grub way, update-grub2 autogenerate a config according to files in /boot/* & config in /et/grub.d

Hence the specific bit for memtest86+.
Which should came with memtest86+ (like the call to bootloader-config).
Like done in other distro & like suggested in the original bug report (bug #9991)

Likewise, if I would want to add an entry for booting the installer from grub2, I would add a /etc/grub.d/99_drakx file in drakx-installer-images, not in grub2.
Comment 3 Barry Jackson 2015-05-13 19:40:30 CEST
OK thanks for taking the time to give a full explanation - I now understand and will implement it ;)
Comment 4 Thierry Vignaud 2015-05-13 20:15:19 CEST
But wait for mga6 to be open :-)
Samuel Verschelde 2015-06-06 15:28:40 CEST

Target Milestone: --- => Mageia 6

Comment 5 Barry Jackson 2015-11-16 01:37:28 CET
Fixed

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


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