search

nwg-piotr / nwg-wrapper

Wrapper to display a script output or a text file content on the desktop in sway or other wlroots-based compositors
MIT License
139 stars 8 forks source link

Signalling 8 to refresh doesn't work #24

Closed MelihDarcanxyz closed 1 year ago

MelihDarcanxyz commented 2 years ago

As the title says, script doesn't refresh when I signal 8 with pkill. But instead, if I run nwg-wrapper with --sig_refresh 7 flag, it does work if I signal 7. I have no idea why. Also, 7 is just random and 8 is the default of nwg_wrapper. Any help appreciated. Thanks.

MelihDarcanxyz commented 2 years ago

I guess this is where things go wrong: https://github.com/nwg-piotr/nwg-wrapper/blob/89af959d7099b9d2ca82761ea618504c424e87b1/nwg_wrapper/main.py#L229

    parser.add_argument("-sr",
                        "--sig_refresh",
                        type=int,
                        default="99",
                        help="custom Signal number to refresh the script; default: 8")
MelihDarcanxyz commented 2 years ago

Opened a PR in: https://github.com/nwg-piotr/nwg-wrapper/pull/25

MelihDarcanxyz commented 1 year ago

Fixed with: https://github.com/nwg-piotr/nwg-wrapper/pull/25