New username feature breaks Mobile install

[Undef-a]

On testing the new release while packing for Debian I noticed that the Mobile module now cannot get past the User Password screen. No matter whether the default username is left unchanged or replaced with a custom one the user cannot continue the install.

The following log line is seen on pressing "continue":

qrc:/user_pass.qml:138: TypeError: Cannot assign to read-only property "username"

Looking at the code, this is exactly what I'd expect.

1. calamares-extensions/modules/mobile/Config.h

   Line 27 in 349acad

   Q_PROPERTY( QString username READ username CONSTANT FINAL )

   is marked CONSTANT FINAL, so attempting to write it from QML will always fail.
2. There is no setUsername( QString username ) function defined in https://github.com/calamares/calamares-extensions/blob/calamares/modules/mobile/Config.cpp
3. There is no code in

   calamares-extensions/modules/mobile/UsersJob.cpp

   Line 52 in 349acad

   { { "sh", "-c", m_cmdPasswd + " " + m_username }, m_password + "\n" + m_password + "\n" },

   to deal with the new username. Even if 1 and 2 were working correctly the UsersJob would fail because the custom user does not exist on the system.

@nmschulte: Sorry I couldn't review the PR at the time. Are you interested in taking on the last bit to get username changing working? Otherwise I'll probably propose a patch to disable the feature.

[nmschulte]

Thanks for the details; I'll look into this soon, yes.

[nmschulte]

3. Even if 1 and 2 were working correctly the UsersJob would fail because the custom user does not exist on the system.

How is the user setup presently? I guess m_username doesn't control the created user, but identifies it so the password can be managed?

[Undef-a]

That's it exactly. The username (the original config option) was only there to allow distributions to configure the user that would have their password set on install. The user itself is created when the image is created, not at install time. To actually change the name of the user you'd need to do something like this: https://wiki.mobian.org/doku.php?id=tweaks&s[]=username#changing-the-default-username-from-mobian (note this is an example and really isn't ready for use as-is)

[nmschulte]

I haven't tested #26, but it's where my mind's at currently about it ; I haven't checked any Mobian images yet either for initial content.

[saedmohmed]

#25 (comment)