Bug 15914

Summary:	[mga6] move 93_memtest from grub2 to memtest86+ (like Fedora does)
Product:	Mageia	Reporter:	Thierry Vignaud <thierry.vignaud>
Component:	RPM Packages	Assignee:	Barry Jackson <zen25000>
Status:	RESOLVED FIXED	QA Contact:
Severity:	minor
Priority:	Normal
Version:	Cauldron
Target Milestone:	Mageia 6
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	grub2, memtest86+	CVE:
Status comment:
Bug Depends on:
Bug Blocks:	416

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