[SRU] Cinnamon session doesn't shutdown until all apps close regardless if they act to Cinnamon's logout request

[Joshua Peisach]

Reference to upstream commit: csm-manager.c: Don't let programs that exit without responding to cinnamon-session's request delay the shutdown.

This fixes the following issue: https://github.com/linuxmint/cinnamon/issues/3752

SRU TEMPLATE:

[Impact]

 * When Cinnamon is exiting, it will wait for apps to close before it shutsdown.
* Using normal shutdown procedures it can be overwritten (e.g. shutdown -a) and will save time.
* The commit upstream removes delay in Cinnamon Session shutdown.

[Test Case]

 * Open an app that requires time to shutdown-e.g. Steam.
 * Shut down and look for the time that it takes to shutdown.
 * Force shutdown Cinnamon from terminal
 * Notice it takes no time to shutdown.

[Regression Potential]

 * Regression is nearly impossible due to this coming from upstream, and the new versions in Groovy. Development will probably still continue through patches.

 * This is also something GNOME from upstream patched.

 * Regression could be possible if things such as memory/RAM speed change or an update of Cinnamon (such as muffin Window Manager) occur overtime.

 * As new apps come by and it takes time to actually safely close the applications, this can cause possible lag and unsafe possible corruption but this is very unlikely.

[Other Info]

 * The bug patch is from upstream (see the commit that closed the issue above)
 * This is part of the bug fixes in cinnamon-session upstream.

An SRU is strongly recommended. Above is the template. Debdiff will be applied in comments.

ProblemType: Bug
DistroRelease: Ubuntu 20.04
Package: cinnamon-session (not installed)
ProcVersionSignature: Ubuntu 5.4.0-40.44-generic 5.4.44
Uname: Linux 5.4.0-40-generic x86_64
ApportVersion: 2.20.11-0ubuntu27.3
Architecture: amd64
CasperMD5CheckResult: skip
CurrentDesktop: ubuntu:GNOME
Date: Sun Jul 12 15:46:08 2020
InstallationDate: Installed on 2020-04-23 (80 days ago)
InstallationMedia: Ubuntu 20.04 LTS "Focal Fossa" - Release amd64 (20200423)
ProcEnviron:
 TERM=xterm-256color
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: cinnamon-session
UpgradeStatus: No upgrade log present (probably fresh install)

[Joshua Peisach (itzswirlz)]

Debdiff/New DSC File/Patch File:

[Joshua Peisach (itzswirlz)]

LP doesn't allow you to send multiple attachments so:

[Joshua Peisach (itzswirlz)]

Patch affects other patches-I will be patching and fixing the patch shortly

[Joshua Peisach (itzswirlz)]

https://github.com/linuxmint/cinnamon/issues/3752

[Joshua Peisach (itzswirlz)]

Final patch available.

[Joshua Peisach (itzswirlz)]

Switch to 'in progress' as we are waiting MOTU

[Fantu (fantonifabio)]

I also saw this issue, about the patch no issue is reported about it from a fast search and gnome-session have same change from one year without revert or change about it, so seems safe.

[Joshua Peisach (itzswirlz)]

Groovy Beta is like tomorrow... so I'll definitely know if it's a problem. I doubt it'll be

[Graham Inggs (ginggs)]

Fixed in version 4.6.1 which is in groovy

https://github.com/linuxmint/cinnamon-session/commit/12454209ed5814cb8d45a5a5b376e5e330f00a95
https://salsa.debian.org/cinnamon-team/cinnamon-session/-/commit/1f3027dbf874c2cf4a18e7f95c7e654e13d34ecc

[Graham Inggs (ginggs)]

Joshua, there are some unnecessary changes in your debdiff (trailing white space), e.g.

- case CSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:
+ case CSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:

-_finished_playing_logout_sound (ca_context *c, uint32_t id, int error, void *userdata)
+_finished_playing_logout_sound (ca_context *c, uint32_t id, int error, void *userdata)

Would you please trim it down to only what's needed?

Also, this:

Bug: https://bugs.launchpad.net/ubuntu/+source/cinnamon-session/+bug/1887312
Bug-<Vendor>: https://github.com/linuxmint/cinnamon/issues/3752

should be:

Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/cinnamon-session/+bug/1887312
Bug: https://github.com/linuxmint/cinnamon/issues/3752

[Joshua Peisach (itzswirlz)]

Alright-I see some activity so I'll go through these:

"Joshua, there are some unnecessary changes in your debdiff (trailing white space), e.g.

- case CSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:
+ case CSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:"

Most of the time I blame Linux Mint. Atom automatically deletes trailing white spaces-so I'll check the diff to see if it removed the whitespace or added whitespaces.

But if you really want I can manually remove it.

"Also, this:

Bug: https://bugs.launchpad.net/ubuntu/+source/cinnamon-session/+bug/1887312
Bug-<Vendor>: https://github.com/linuxmint/cinnamon/issues/3752

should be:

Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/cinnamon-session/+bug/1887312
Bug: https://github.com/linuxmint/cinnamon/issues/3752"
Okay. I'll fix that ASAP

[Joshua Peisach (itzswirlz)]

Yes-these are whitespaces. I don't think you should be upset about these being removed:

- case CSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:
+ case CSM_MANAGER_LOGOUT_SHUTDOWN_INTERACT:"

[Joshua Peisach (itzswirlz)]

Okay, here changes the Bug and Bug-Ubuntu fields.

The whtiespaces are whitespace removals, so I think we can keep it.

[Graham Inggs (ginggs)]

Please trim your debdiff down to only what is needed.
Having unnecessary changes just makes things harder to review.

[Graham Inggs (ginggs)]

Uploaded, thanks.

[Brian Murray (brian-murray)]

Hello Joshua, or anyone else affected,

Accepted cinnamon-session into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/cinnamon-session/4.4.1-1ubuntu0.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

[Joshua Peisach (itzswirlz)]

Hey there,

Sorry for not verifying... I'm sorry for leaving it in proposed. Anyways, Test Case!

[Test Case]
1) Update Ubuntu Cinnamon Remix 20.04.2 (testing, stable technically)
2) Enable backports
3) Upgrade, reboot
4) Download and install Steam
5) Login to Steam
6) When you are logged in, Steam should log out just as fast as it did when rebooting with no windows open.

Fix, verification complete. (Focal)

[Launchpad Janitor (janitor)]

This bug was fixed in the package cinnamon-session - 4.4.1-1ubuntu0.1

---------------
cinnamon-session (4.4.1-1ubuntu0.1) focal; urgency=medium

  * Don't let apps that ignore Cinnamon's exit req delay shutdown (LP: #1887312)

 -- Joshua Peisach <email address hidden> Wed, 15 Jul 2020 12:12:09 -0400

[Brian Murray (brian-murray)]

The verification of the Stable Release Update for cinnamon-session has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.