Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gtkIndicator missing gi import? #15

Open
thomas-phillips-nz opened this issue Sep 26, 2023 · 2 comments
Open

gtkIndicator missing gi import? #15

thomas-phillips-nz opened this issue Sep 26, 2023 · 2 comments

Comments

@thomas-phillips-nz
Copy link
Contributor

I noticed we don't have any import in the requirements.txt file for the gtkIndicator gi package (pgi?). I guess we don't want to force this package onto users who might just be using the command-line tool.

Should we add instructions on installing this package and its dependencies before being able to run tenma-applet or ./gtkIndicator.py? I have had trouble trying to get pgi installed on my Windows machine, and I am not actually sure which OSes tenma-applet supports.

@kxtells
Copy link
Owner

kxtells commented Sep 27, 2023

I'd say it's simply missing documentation stating which environments are supported.

gtkIndicator was an afterthought and a nice to have, it uses GTK alongside its python bindings via PyGObject to have a systray indicator of your power unit.

This should mostly work on Linux systems with Desktop environments like GNOME and XFCE. I imagine in other environments (KDE) will crash as soon as it starts, either because it does not find the gi import, or after installing pyGObject it might still miss system dependencies.

@kxtells
Copy link
Owner

kxtells commented Sep 28, 2023

We could even treat it as an extras too.

Where the default application installs the command line alongside the lib, but we can set an specific install of the gtk applet 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants