[PATCH 2/6] install: Move Payload postInstall() after bootloader

Gene Czarcinski gczarcinski at gmail.com
Thu May 22 14:47:53 UTC 2014


On 04/29/2014 04:38 AM, Vratislav Podzimek wrote:
> On Sat, 2014-04-26 at 17:30 +0000, Colin Walters wrote:
>> None of the current Payload subclasses are sensitive to ordering with
>> respect to the bootloader. The forthcoming OSTreePayload class will
>> require postprocessing of the bootloader, so let's just swap the
>> ordering.
>> ---
>>   pyanaconda/install.py | 6 +++---
>>   1 file changed, 3 insertions(+), 3 deletions(-)
>>
> This one looks good to me.
>
>
It does not look that good to me because it broke Live Install.  See
https://bugzilla.redhat.com/show_bug.cgi?id=1099627

pjones has changed grubby (grubby-8.35-1) and doing a Live Install now 
produces a working grub.cfg.  However, it looks strange and different 
from a "normal" grub.cfg because the rescue menuentry is first and the 
regular kernel is second.

Question: Is there a reason that prohibits installing the bootloader 
after the post-installation setup tasks?  Something break in the new 
functionality that you are adding?

Gene


More information about the anaconda-patches mailing list