dbus: interface to implement 'qdbus io.thp.billboard / render'

[teleshoes]

No description provided.

[thp]

Above you say QDBusConnection::sessionBus().registerService(DBUS_SERVICE);, but you say bus="system" here -- I guess it should be the session bus?

[teleshoes]

yep! i originally used the system bus since its generally more reliable, but i didnt realize that the only reason it worked was my /etc/dbus-1/system.conf.
apparently, i forgot to change it everywhere

[thp]

After renaming the D-Bus method and everything added in the patch from render to refresh, maybe also add a simple billboard-refresh subdirectory (qmake SUBDIRS) that installs a small shell script called billboard-refresh to /usr/bin/ that does the D-Bus call via dbus-send.

Or if you want to go all the way, name the directory billboard-cli, the script also billboard-cli and have it accept a sub-command as first parameter, and for now just implement the refresh subcommand to call the D-Bus method. In this case, it should probably not be a script, but can be a (non-GUI) Qt application and use QtDBus for calling the method.

What do you think?

[teleshoes]

sure, ill add billboard-cli. as for render vs refresh, i think render is more appropriate. how about having refresh as a synonym in the dubs wrapper script?

[teleshoes]

i prefer to target /opt/billboard/bin for consistency. i hate it when programs install some executables in one dir and some in another, even though its somewhat more justifiable for binaries vs scripts

[teleshoes]

i decided a script was more appropriate than a QT app for a glorified dbus wrapper. billboard-refresh is too specific, though {if the change-property dbus method is added, it will be nice to have a place to put it, e.g.: billboard-cli --set-prop PROP VAL}

[thp]

Added some more comments. After that, this should be ready to go in as well.

[thp]

Merged, thanks!

[teleshoes]

woo! thanks!