Code review comment for lp:~mterry/ubiquity/oem-config-merge

Revision history for this message
Evan (ev) wrote :

I've built a copy of your branch merged into trunk and run some tests on a recent live CD, and by and large things perform as one would expect. However, I ran into a problem when oem-config/enable is set. In debian/rules your branch adds oem to the list of reserved usernames, but that's the username we use for the temporary OEM user. This results in an impossible to proceed situation at the user setup page, as the page refuses to proceed with the unable to be edited name.

review: Needs Fixing

« Back to merge proposal