headers
andres | 25 Jul 2012 18:14

[patch] multiple slock instances make screen black

To reproduce run `slock & slock`. The first gets the password and 
exits, but
the second did not manage to grab keyboard and mouse. The actual bug is 
that
it did not understand that it didn't, so it just hangs there and keeps 
the
screen blank unitl it is killed.

A patch for this is attatched. If it does not manage to lock any screen
(maybe because they are already locked) it exits with code 1.
Attachment (multislock.patch): text/x-diff, 1084 bytes
Permalink | Reply |
headers
Kai Hendry | 30 Jul 2012 21:53
Picon
Picon
Favicon
Gravatar

Re: [patch] multiple slock instances make screen black

Thanks Andres for the contribution. Hopefully Anselm will get the time
to review and merge in your code.

Just wanted to quickly mention that
https://github.com/scklss/slock/commits/master is now
https://github.com/suckless/slock/commits/master

Unfortunately github don't have any mechanisms to move/redirect your
issues to the new repo. So I hope you don't mind recloning too much.

I've asked github admins to kill https://github.com/scklss entirely
except for a redirect notice.

Greetings from Berlin,
Permalink | Reply |
headers
Anselm R Garbe | 2 Aug 2012 21:54
Picon
Gravatar

Re: [patch] multiple slock instances make screen black


On Wed, Jul 25, 2012 at 04:14:04PM +0000, andres <at> krutt.org wrote:
> To reproduce run `slock & slock`. The first gets the password and
> exits, but
> the second did not manage to grab keyboard and mouse. The actual bug
> is that
> it did not understand that it didn't, so it just hangs there and
> keeps the
> screen blank unitl it is killed.
> 
> A patch for this is attatched. If it does not manage to lock any screen
> (maybe because they are already locked) it exits with code 1.

Thanks, applied.

-Anselm
Permalink | Reply |

Gmane