[DONE] package notify-broadcast?

johnraff · 2016-09-26 07:24:00

The notify-broadcast script is quite small (~1kB) and I think it might be handy to have available in BunsenLabs for devs or users who are writing root-owned scripts to send messages to users on the system. (Originally I wrote it so the 32bit flash update script I was working on could inform users when the plugin had been upgraded, or if it hit an error.)

At first I thought of putting it in bunsen-common, as it's so small, but it didn't seem quite that essential to BL. OTOH, if it went in bunsen-utilities then any package which wanted to use it would have to pull in the whole 34kB of b-u. (not all that much, agreed)

After dismissing the idea of making a whole package just for that little script, I thought of creating a new package called bunsen-sysutils for utilities mainly expected to be run as root, by system maintainers. (It would still only have this one script in at first though.)

Another question - all our executable names start with bl- so the script would have to be renamed bl-notify-broadcast to fit in. I do rather like notify-broadcast by analogy with notify-send though...

Any thoughts?

Last edited by johnraff (2016-10-19 03:48:14)

Sector11 · 2016-09-26 17:09:48

As someone that doesn't write "root-owned-scripts" or user owned scripts I'm not qualified to answer.

In fact I don't even know what this does - being a 64bit user I didn't follow the 32bit flash update script. Sorry.

damo · 2016-09-26 17:38:34

johnraff wrote:

...
Another question - all our executable names start with bl- so the script would have to be renamed bl-notify-broadcast to fit in. I do rather like notify-broadcast by analogy with notify-send though...
Any thoughts?

I guess it should really be bl-notify-broadcast, for consistency

Head_on_a_Stick · 2016-09-26 18:27:24

I think it should be added to bunsen-utilities, I don't think many will notice an extra 34KiB.

Bearded_Blunder · 2016-09-26 23:28:36

I think even fewer will actually care about an extra 34K if they're wanting to use it. 34M and maybe there'd be a concern about pulling "the whole XX of b-u", maybe... Lean as an ideal is good, but there comes a point where the last few KiB isn't worth the extra effort.

johnraff · 2016-09-27 04:01:15

About "the extra 34kB": it's not really about a user wanting to use this script directly, so much as some other package needing to pull it in as a dependency. But I still agree it's not all that much.

OTOH bunsen-utilities itself has some dependencies that would be brought in too - if they weren't on the system already (most of them probably would be though):

Package: bunsen-utilities
Architecture: all
Depends: bunsen-common, python2.7, python-gtk2, python-xlib, upower, yad, wmctrl, xdotool (>=1:2.20110530.1-3), python-dbus
Recommends: gmrun, light-locker, openbox, nitrogen | feh, scrot

OK drop it in bunsen-utilities, name the executable bl-notify-broadcast, and packages that want to use it can make bunsen-utilities a recommend instead of a hard dependency - it's only a frill, after all.

Sounds good?

Head_on_a_Stick · 2016-09-27 06:15:11

Perfect

johnraff · 2016-09-29 08:41:53

bunsen-utilities 8.7.1-1 on GitHub now holds bl-notify-broadcast and bl-notify-failure@.service.

This has been pushed to "master" so it will be ready for any deuterium packages that might want to use it.

#1 2016-09-26 07:24:00

[DONE] package notify-broadcast?

#2 2016-09-26 17:09:48

Re: [DONE] package notify-broadcast?

#3 2016-09-26 17:38:34

Re: [DONE] package notify-broadcast?

#4 2016-09-26 18:27:24

Re: [DONE] package notify-broadcast?

#5 2016-09-26 23:28:36

Re: [DONE] package notify-broadcast?

#6 2016-09-27 04:01:15

Re: [DONE] package notify-broadcast?

#7 2016-09-27 06:15:11

Re: [DONE] package notify-broadcast?

#8 2016-09-29 08:41:53

Re: [DONE] package notify-broadcast?

Board footer