main/postmarketos-ui-*: fix for lightdm autologin when package is reinstalled.

When the UI package is reinstalled (due to the package being recompiled or upgraded), it runs $pkgname.post-upgrade instead of $pkgname.post-install which would run the sed replacement LightDM autologin.

This will cause LightDM to show the greeter when the UI package is rebuilt, and the user ran pmbootstrap install without deleting the old chroot.

changed the description

By Asriel Dreemurr on 2019-05-27T06:51:12

Do you mean that without this patch autologin will be broken on package updates? And !379 (merged) is not merged yet, is this MR supposed to be merged after that?

And I think pkgrel++ are missing in all APKBUILDs

By Alexey Min on 2019-05-27T08:55:09

added 1 commit

• 787f8c55 - main/postmarketos-ui-*: fix for lightdm autologin when package is reinstalled.

Compare with previous version

By Danct12 on 2019-05-27T10:04:00

I don't fully understand why the old chroot is reused when running pmbootstrap install twice, is there any benefit to that rather than just wiping the chroot and starting over @ollieparanoid?

By Bart Ribbers on 2019-05-27T14:21:56

Well, it is faster. (Not such a huge benefit, but still.)

FWIW, I'll look at this and !379 (merged) once packages from master are rebuilt against the new qt version (#269 (closed)). Thanks for the MR @asriel-danctnix!

By Oliver Smith on 2019-05-28T21:50:29

@ollieparanoid Anything needs to be done with this MR?

By Danct12 on 2019-05-31T01:54:37

LGTM, I'll merge it.

By Oliver Smith on 2019-05-31T01:59:31

Actually, rebasing on master causes a merge conflict. And we still have this xfce4 issue standing out: !379 (comment 176474355)

so it is probably wise to wait with this MR until it is fixed.

By Oliver Smith on 2019-05-31T02:06:26

added 45 commits

• 787f8c55...b446c06b - 44 commits from branch postmarketOS:master
• 5b1a20fb - main/postmarketos-ui-*: fix lightdm autologin after reinstall (!385 (merged))

Compare with previous version

By Oliver Smith on 2019-06-04T18:44:07

resolved all discussions

By Oliver Smith on 2019-06-04T18:45:38

The patch is rebased. Since it only adds a post-upgrade script, it won't influence whether xfce is working after an initial install or not. So no need to wait for the issue to get resolved.

Thanks guys, merging!

By Oliver Smith on 2019-06-04T18:47:20

approved this merge request

By Oliver Smith on 2019-06-04T18:48:18

merged

By Oliver Smith on 2019-06-04T18:48:25

mentioned in commit 5b1a20fb

By Asriel Dreemurr on 2019-06-11T15:01:36