~emersion/public-inbox

2 2

chayang not working as ret-code is always 0

Details
Message ID
<878ri9i45x.fsf@rmm6prod02.runbox.com>
DKIM signature
missing
Download raw message
Thanks for putting chayang together!

I don't quite get chayang to work.  As far as I can tell it seems to
build fine (clone repo, then build steps as in README).  When running it
from the command line it dims the screen just fine and, as expected,
either exits after the timeout or when I type/move the mouse.  However,
my understanding is that the ret-code in the latter case should be
non-zero but it is zero.  Thus `chayang && swaylock`` will still lock on
pressing keys/moving the mouse to abort.

```
...tmp/chayang/build(master)  >> ./chayang

  ### now moving the mouse or hitting a key

...tmp/chayang/build(master)  >> echo $?
0
```

Am I missing something?  Let me know if you need more info.

Swayloc v1.7, Sway v1.8, Arch Linux
Details
Message ID
<jbdjJRscBzwA2I_3O-u99z5yy9vMKSzwt7DCYwns8zKphQwOKo7jXQ40JevEqKplpRT2vNU56f8LUodEfyLd1uvWXKB5KEauxBF9_QA2NG0=@emersion.fr>
In-Reply-To
<878ri9i45x.fsf@rmm6prod02.runbox.com> (view parent)
DKIM signature
missing
Download raw message
On Wednesday, January 11th, 2023 at 14:29, mauro3@runbox.com <mauro3@runbox.com> wrote:

> I don't quite get chayang to work. As far as I can tell it seems to
> build fine (clone repo, then build steps as in README). When running it
> from the command line it dims the screen just fine and, as expected,
> either exits after the timeout or when I type/move the mouse. However,
> my understanding is that the ret-code in the latter case should be
> non-zero but it is zero. Thus `chayang && swaylock`` will still lock on
> pressing keys/moving the mouse to abort.

Sorry, this was a bug. Fixed in commit 03f80d8c6b21.
Details
Message ID
<87wn5tkogo.fsf@rmm6prod02.runbox.com>
In-Reply-To
<jbdjJRscBzwA2I_3O-u99z5yy9vMKSzwt7DCYwns8zKphQwOKo7jXQ40JevEqKplpRT2vNU56f8LUodEfyLd1uvWXKB5KEauxBF9_QA2NG0=@emersion.fr> (view parent)
DKIM signature
missing
Download raw message
On Wed, 2023-01-11 at 17:28, Simon Ser wrote:
> Sorry, this was a bug. Fixed in commit 03f80d8c6b21.

Thanks for the swift fix!  M
Reply to thread Export thread (mbox)