Code review comment for lp:~albertomilone/gnome-control-center/randr-virtual

Revision history for this message
Bryce Harrington (bryce) wrote :

Thanks everyone for the ideas and discussions.

Like Timo mentioned above, alt-sysrq-k is still available, less likely
to get hit accidentally, and should work more reliably since it's
implemented in the kernel. There are definitely some kinds of lockups
where X can't do ctrl-alt-backspace.

If we didn't have alt-sysrq-k, then the OpenSuSE patch would be a good
alternative since it's already been tested. If upstream were to adopt
it, I think we'd take it, but as it currently stands, it seems
extraneous and probably unnecessary.

Regarding the help documentation, yes that will all need to be changed
to s/ctrl-alt-backspace/alt-sysrq-k/. That is done easily enough and
need not be a reason for retaining the shortcut.

Bryce

« Back to merge proposal