Tails

#2 Updated by intrigeri 2016-12-04 10:21:58

• Subject changed from Pressing the power button does not immediately trigger emergency shutdown to Pressing the power button does not immediately trigger emergency shutdown on some machines

#3 Updated by intrigeri 2016-12-04 13:13:56

• Status changed from New to In Progress
• Assignee set to sajolida
• % Done changed from 0 to 10
• QA Check set to Info Needed

spriver wrote:
> This seems to be hardware related,

Indeed. I bet it works on most machines. At least it works on the MacBook Pro I have here too.

I’ve verified that our settings for this are correct on 2.7.1. It’s a different matter for 3.0 though, as GNOME doesn’t offer the possibility to configure the power button to trigger a shutdown (except in VMs, where it’s the default action) anymore:

• https://github.com/GNOME/gnome-settings-daemon/commit/69d9d8b1eda2d50acca5b4da58003afb8b82a7f1
• https://bugzilla.gnome.org/show_bug.cgi?id=755953

The available actions are: nothing, suspend, hibernate.

IMO we should set it to “nothing” right now, and later to “suspend” once Bug #11729 (which blocks #11395, and in turn Bug #11052) is done. Thoughts?

#4 Updated by sajolida 2016-12-05 14:58:31

• Assignee changed from sajolida to intrigeri

I agree with “nothing” right now. I’m not sure about “suspend” in the future and would instead stick to the GNOME default since we’ll already have graphical way to suspend with #11395.

Shall we use this ticket to implement ‘“nothing” right now’?

#5 Updated by intrigeri 2016-12-05 15:48:15

• Assignee changed from intrigeri to sajolida

> I’m not sure about “suspend” in the future and would instead stick to the GNOME default since we’ll already have graphical way to suspend with #11395.

I think that “suspend” is the default in GNOME on Stretch.

> Shall we use this ticket to implement ‘“nothing” right now’?

The way I see it, we currently have something that’s useful and works for most people. Due to changes in GNOME we’ll have to drop it in Tails 3.0, and use either “nothing” or the new GNOME defaults instead. But I see no strong reason to remove this piece of functionality right now (we have tons of other useful functionality that doesn’t work for 100% of our users, and we rarely use this as an argument to drop it). But I’m happy to listen and understand why you want to switch to “nothing” in 2.x :)

#6 Updated by sajolida 2016-12-06 09:31:55

• Assignee changed from sajolida to intrigeri

You said IMO we should set it to "nothing" right now in Bug #12004#note-3 so I was merely agreeing with you. I suggested this originally in the description of this ticket with I'm not sure it's worth keeping this feature.

Now, regarding when exactly to implement that I don’t really care if it’s in 2.x or 3.x and it’s definitely not high prio. But I think we should aim at sticking to the GNOME default at some point.

But keep in mind that having this feature work on some computers and not on others can be a bit sensitive. We have implemented as one option for emergency shutdown and Tails is live and can be used on different laptops. So if one day I have to use a different laptop and am trained to do emergency shutdown this way and it doesn’t work on this particular laptop, then I might get confused, panic, etc.

#7 Updated by intrigeri 2016-12-06 10:15:37

• Assignee changed from intrigeri to sajolida

> You said IMO we should set it to "nothing" right now in Bug #12004#note-3 so I was merely agreeing with you.

I meant “on the feature/stretch branch”. Sorry I’ve been super unclear!

> Now, regarding when exactly to implement that I don’t really care if it’s in 2.x or 3.x and it’s definitely not high prio. But I think we should aim at sticking to the GNOME default at some point.

ACK. Please just decide if we switch to “nothing” in 2.x or 3.x and I’ll gladly implement it :)

> But keep in mind that having this feature work on some computers and not on others can be a bit sensitive. We have implemented as one option for emergency shutdown and Tails is live and can be used on different laptops. So if one day I have to use a different laptop and am trained to do emergency shutdown this way and it doesn’t work on this particular laptop, then I might get confused, panic, etc.

Sure. Sadly, that’s also true for some of the other ways to trigger a shutdown (e.g. it doesn’t work with some SD-card readers).

#9 Updated by sajolida 2016-12-07 08:10:28

• Assignee changed from sajolida to intrigeri
• QA Check changed from Info Needed to Dev Needed

#10 Updated by intrigeri 2016-12-07 08:35:49

• Target version set to Tails_3.0

>> ACK. Please just decide if we switch to “nothing” in 2.x or 3.x and I’ll gladly implement it :)

> If it’s the same amount of work to do it in 2.x and 3.x (eg. no migration work for 3.x), then I’d say do it on 2.x. If it’s more work to do it on 2.x and port it to 3.x, then I’d say do it on 3.x.

Thanks!

The prefs name have changed so doing it on 2.x would imply a (little, but still) additional work ⇒ I’ll do that for 3.0.
We’ll need a note for it in the release notes. I’m considering adding a news/version_3.0 on feature/stretch, so that at least I can add notes there for you to consider later.

#13 Updated by intrigeri 2016-12-21 11:27:12

• Subject changed from Pressing the power button does not immediately trigger emergency shutdown on some machines to Update power button handling for Stretch
• % Done changed from 10 to 50
• QA Check changed from Dev Needed to Ready for QA
• Type of work changed from Research to Code

Done in Git.

#14 Updated by intrigeri 2016-12-21 13:46:59

• Status changed from In Progress to Resolved
• % Done changed from 50 to 100
• QA Check changed from Ready for QA to Pass

Works.