Skip to content
This repository was archived by the owner on Feb 25, 2021. It is now read-only.

Import GTK in a backward compatible way #12

Merged
merged 3 commits into from
Oct 29, 2018
Merged

Import GTK in a backward compatible way #12

merged 3 commits into from
Oct 29, 2018

Conversation

landmaj
Copy link
Contributor

@landmaj landmaj commented Oct 28, 2018

This fixes issue #11 without breaking backward compatibility for older versions of MATE. The program will check MATE version and import appropriate GTK. Why is this important? Ubuntu Mate 16.04 is supported until 2021 and that version still requires older version of GTK. A lot of people are stuck at that version (including me) for example at work.

I took the liberty of fixing setup.py a little bit, since I had to add a new file there anyway. It used tabs instead of spaces (which are used in all other files - mixed indentation is not allowed in Python 3), there were unnecessary semicolons. I also changed maintainer to author (it makes more sense in your case) and bumped version to 2.0.1.

Copy link
Owner

@city41 city41 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for doing this! Just looks like one issue with the version checking. Could you also make it version 2.1.0 instead?

@city41 city41 merged commit bc931a6 into city41:master Oct 29, 2018
@city41
Copy link
Owner

city41 commented Oct 29, 2018

Thanks @landmaj!

@landmaj
Copy link
Contributor Author

landmaj commented Oct 29, 2018

I removed by accident gi.require_version("MatePanelApplet", "4.0"). It's not really required but will throw a warning, so I added it back.

Bumped version to 2.1.0, as you asked.

Also updated README to reflect the changes, but I kept notice about not addressing GTK2 issues.

@landmaj
Copy link
Contributor Author

landmaj commented Oct 29, 2018

Tested on Ubuntu Mate 18.04 and 16.04, works without issues.

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

Successfully merging this pull request may close these issues.

2 participants