[PATCH configure] BZ #783220 - change default admin email to 'root at localhost'

Mo Morsi mmorsi at redhat.com
Thu Feb 2 15:34:54 UTC 2012


> NACK, for a few reasons.
>
> - This doesn't apply and needs rebased due to the changes for BZ#740089
>    in 814ae5ed0aacd5aa482b213be339d6470630561d (removing interactive mode
>    gets rid of profiles/custom.pp)
>
> - The dc:create_user task requires a last name, and ultimately
>    interprets the empty string as nil, causing it to display usage and
>    quit
>
> - root at localhost fails email address validation; I'd propose using
>    root at localhost.localdomain instead which is accepted

Done. Follow up patches sent.

   -Mo



More information about the aeolus-devel mailing list