[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