-
Notifications
You must be signed in to change notification settings - Fork 18
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
Linux (GNOME) #7
Comments
From what (little) I've found online there doesn't seem to be a consistent way to detect this on Linux, but it seems like calling:
and looking for Happy to open a PR to |
Seems to me the ecosystem is a little too vast to get this consistently right across all the Linux platforms. Also, I am not thrilled by a Nevertheless, I am not against a PR on this topic, including one that uses |
Indeed I imagine it could fail in all sorts of ways. Hopefully a try/except might be good enough for a first version. I'll open a quick PR and we can see if it looks reasonable. |
Could you use PyGObject instead? |
I would rather not, as this would violate the "philosophy" of this package of avoiding extra dependencies
|
You could nest the import though, so only people who really need it can install the package. |
As I mentioned before, for the moment I would prefer to go with the way addressed in #8 as I think is leaner, though more brittle. Medium term, it could be worthy to investigate how PyGObject is implemented and get the relevant parts needed for just this feature to include just them in Darkdetect. |
Nice project! Do you have any plans to support GNOME (or KDE) as well?
The text was updated successfully, but these errors were encountered: