FS#80221 - [sway] missing a python-i3ipc dependency

Attached to Project: Arch Linux
Opened by Vianney Bouchaud (vianney) - Friday, 10 November 2023, 16:49 GMT
Last edited by Brett Cornwall (ainola) - Thursday, 16 November 2023, 05:27 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Jerome Leclanche (Adys)
Brett Cornwall (ainola)
Maxim Baz (maximbaz)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
The sway package contains [various scripts](https://gitlab.archlinux.org/archlinux/packaging/packages/sway/-/blob/main/PKGBUILD?ref_type=heads#L90) in `/usr/share/sway/scripts`. Some of those scripts are written in python and import i3ipc which is available from extra/python-i3ipc.

Would it be possible to add this dependency either as depends or optdepends to the sway package?

Additional info:
* sway: 1.8.1-3
This task depends upon

Closed by  Brett Cornwall (ainola)
Thursday, 16 November 2023, 05:27 GMT
Reason for closing:  Won't implement
Additional comments about closing:  We'll create a new package on the next release!
Comment by Vianney Bouchaud (vianney) - Friday, 10 November 2023, 16:51 GMT
Sorry for the markdown in the details, I wrongly assumed it would be evaluated.

If I may add: the `grimshot` utility also depends on:
* grim: actual screenshot binary
* slurp: window or area selection
* wl-copy (wl-clipboard): to copy screenshots to the clipboard
* jq: json parsing of sway messages, I assume
* notify-send (libnotify): for the --notify option
Comment by Brett Cornwall (ainola) - Sunday, 12 November 2023, 22:28 GMT
@maximbaz, I'm on the fence about this. Do we want to advertise deps on contrib scripts? This really expands the optdepends to a pretty overwhelming degree; Adding even more later might make it a little silly.
Comment by Vianney Bouchaud (vianney) - Monday, 13 November 2023, 20:44 GMT
My two cents: why not do like upstream? They moved the contrib scripts to another repository out of the sway github organization due to the lack of reviewers from the sway core contributors: https://github.com/swaywm/sway/commit/bb91b7f5fa7fddb582b8dddf208cc335d39da9e7

You could create a sway-contrib package and put there the dependencies it needs and have it be an optdepends of sway.
Comment by Maxim Baz (maximbaz) - Monday, 13 November 2023, 21:17 GMT
I agree with you Brett, it feels too excessive to maintain a matrix of contrib scripts and their corresponding dependencies, to properly keep track of all this...
Comment by Maxim Baz (maximbaz) - Monday, 13 November 2023, 21:30 GMT
Great point Vianney, contrib scripts are now indeed gone from the sway repo, so they will probably be gone from this package as well since the next sway release. If we were to add them back, packaging them separately sounds like a good and clean idea.
Comment by Brett Cornwall (ainola) - Thursday, 16 November 2023, 05:27 GMT
Thanks for bringing this up, Vianney! I'm going to go ahead and close this since the next release will come out with a new package including the collection of scripts.

Loading...