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

Rewrite to support Gnome 45 #13

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

matthijskooijman
Copy link
Contributor

In gnome 45, the module system is significantly different, so the code
structure is changed significantly. This changes the code to this new
structure, without making any functional changes.

This change also removes support for older versions of Gnome.

This fixes #12.

I created this PR on top of #8, which is not merged yet, but keeping that history is probably a good idea.

AFAIU this version number is intended to be filled with a linear version
number by extensions.gnome.org, so any version in here is not preserved
when uploading to there. Also, for gnome-shell, it seems that the
presence of this version number is an indication that the extension
installation is managed by gnome-shell.

In practice, when using a symlink directly into the git checkout for
this extension, the presence of this version key causes gnome-shell to
(presumably) consider this extension invalid and (observed) replace it
with the latest version downloaded from extensions.gnome.org.
It is probably also supported by intermediate versions, but I have not
tested that.
In gnome 45, the module system is significantly different, so the code
structure is changed significantly. This changes the code to this new
structure, without making any functional changes.

This change also removes support for older versions of Gnome.

This fixes aXe1#12.
@ZimbiX ZimbiX mentioned this pull request Jul 27, 2024
No changes are needed.

This was tested on 47 and assumes it also works on 46.
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

Successfully merging this pull request may close these issues.

Support Gnome 45
1 participant