xfce4-panel-profiles help message

Emanuele Petriglia xfce-dev at emanuelepetriglia.com
Thu Sep 26 13:59:32 CEST 2019


Hi,

In these days I'm using 'xfce4-panel-profiles' to make backup and
restore my 'xfce4-panel' configuration; it is really an useful tool. I
sent two patches[1][2] to fix some issues about this program, but I
found other annoyances:

1. The help message is built[3] using argv[0], but the program is called
using a wrapper script, so this is the result (at least on Fedora 30):

    $ xfce4-panel-profiles any-args
    Xfce Panel Profiles - Usage:

/usr/share/xfce4-panel-profiles/xfce4-panel-profiles/xfce4-panel-profiles.py
: load graphical user interface.

/usr/share/xfce4-panel-profiles/xfce4-panel-profiles/xfce4-panel-profiles.py
save <filename> : save current configuration.

/usr/share/xfce4-panel-profiles/xfce4-panel-profiles/xfce4-panel-profiles.py
load <filename> : load configuration from file.

  Is it not better to hardcode the program's name (or extract only the
last part of the path)?

2. I cannot check the current version of the program with '-v' or
'--version' option, because there is not a real command line parser[4].

3. I cannot load a single file configuration[5], it must be wrapped
inside a '.tar' file. Why do not add the option to load a simple
'config.txt' file directly?

I know that these issue are tiny annoyances, but I can fix these in my
free time if you want.

Have a good day!

[1]: https://bugzilla.xfce.org/show_bug.cgi?id=15983
[2]: https://bugzilla.xfce.org/show_bug.cgi?id=14559
[3]:
https://git.xfce.org/apps/xfce4-panel-profiles/tree/xfce4-panel-profiles/xfce4-panel-profiles.py#n375
[4]:
https://git.xfce.org/apps/xfce4-panel-profiles/tree/xfce4-panel-profiles/xfce4-panel-profiles.py#n362
[5]:
https://git.xfce.org/apps/xfce4-panel-profiles/tree/xfce4-panel-profiles/panelconfig.py#n76

-- 
Emanuele Petriglia (ema-pe)


More information about the Xfce4-dev mailing list