Patches 1 and 2 are sort of a set, as are patches 3 and 4.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

On Tue, May 24, 2011 at 01:24:54PM -0500, David Lehman wrote:
> Patches 1 and 2 are sort of a set, as are patches 3 and 4.
>

Those all look good.

--
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

Patch 1 cleans up bootloader password management. A potentially
troublesome add-in is code to continue with bootloader install
in the event that we fail to set up the password authentication.
My thinking was that unbootable is worse then insecure, but I'm
open to argument.

Patch 2 removes the "other OS" widget from the UI for GRUB2 in
favor of letting os-prober automatically detect and set up
entries for other OS.

Patch 3 fixes a little bug in the devicetree that was hampering
clumens' efforts at a "here's your system's storage as a ks
snippet" script.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

On Thu, Jul 14, 2011 at 12:59:56PM -0500, David Lehman wrote:
> Patch 1 cleans up bootloader password management. A potentially
> troublesome add-in is code to continue with bootloader install
> in the event that we fail to set up the password authentication.
> My thinking was that unbootable is worse then insecure, but I'm
> open to argument.

We should notify the user that it failed. If it is a kickstart install
I'd vote for not finishing since they specifically request setting up
the password.

Ack to the patches.

--
Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list