BUG: mx-installer 18.4.04 wrong group-ownership of user-account

[Adrian]

Is this something related to persistency by any chance? Did any of you use persistency at any point?

[anticapitalista]

On my box, it is an old install of antiX. I think something else must have changed the permissions xdg perhaps? There was a security upgrade of xdg-utils a few days ago.

[Adrian]

On my box, it is an old install of antiX. I think something else must have changed the permissions xdg perhaps? There was a security upgrade of xdg-utils a few days ago.

That's a good point, it might happen after the install, however, I don't see that happening with fully updated MX16 and MX17 (I did see the xdg-utils update).

[dolphin_oracle]

I just did a fresh install with the 32 bit iso.

all my directories were owned by $USER:$USER except for hte desktop folder. it was owned by $USER:users.

I see this in the installers.

Code: Select all

// fix the ownership, demo=newuser
    cmd = QString("chown -R demo.users %1").arg(dpath);
    if (shellcmd->run(cmd.toUtf8()) != 0) {
        setCursor(QCursor(Qt::ArrowCursor));
        QMessageBox::critical(0, QString::null,
                              tr("Sorry, failed to set ownership of user directory."));
        return false;
    }

should that be something like demo.users:demo.users

[ChrisUK]

I don't think it's the xdg-utils update... I do regular backups with Timeshift (which preserves permissions etc) - a random snapshot taken on 9th of May still has the group ID of 100 for Pictures etc. That was over two weeks before the aforementioned update. See attached - notice the correct group id of the two folders created after the initial install (bin & Updater)

Oh - Distro version and date of install: MX-17.1_x64 Horizon December 10, 2017

[Adrian]

should that be something like demo.users:demo.users

No, demo.users is fine. But users id is indeed 100. Now the question is why I see that 1000 on my installed system?

[Adrian]

Should we change users to demo?

cmd = QString("chown -R demo.users %1").arg(dpath);

Also, see the explanation about . or : in chown: https://serverfault.com/questions/19429 ... p-in-chown

[dolphin_oracle]

Should we change users to demo?

cmd = QString("chown -R demo.users %1").arg(dpath);

Also, see the explanation about . or : in chown: https://serverfault.com/questions/19429 ... p-in-chown

Ah.

I think that may be it. This would be for any folders that exist prior to xdg-user-dirs doing its thing.

[Adrian]

OK, let's replace everywhere with:

Code: Select all

cmd = QString("chown -R demo:demo %1").arg(dpath);

fehlix, you were right, thanks for the bug report! What is confusing is that some folders are handled by xdg and some are modified by installer.

[dolphin_oracle]

Ok I've been working with gazelle-installer today anyway. I'll send up the change in a bit.