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

Mark this as compatible with GNOME 3.38 and 40 #2

Merged
merged 1 commit into from
May 20, 2021

Conversation

michel-slm
Copy link

@michel-slm michel-slm commented Mar 26, 2021

Fedora 33 ships with GNOME 3.38 (but with version validation disabled)
and this extension works fine there. Fedora 34 beta has GNOME 40 (with
version validation enabled) and the extension works once the version
is added to the list of supported versions.

Signed-off-by: Michel Alexandre Salim michel@michel-slm.name


This change is Reviewable

Fedora 33 ships with GNOME 3.38 (but with version validation disabled)
and this extension works fine there. Fedora 34 beta has GNOME 40 (with
version validation enabled) and the extension works once the version
is added to the list of supported versions.

Signed-off-by: Michel Alexandre Salim <michel@michel-slm.name>
@heatray
Copy link

heatray commented Apr 23, 2021

Using with #1 on gnome 40 I am getting warnings: Usage of object.actor is deprecated for ArgosButton
because of wrong shellVersion compare
https://github.com/mwilck/argos/pull/1/files#diff-f5dff91cadb0390ae96cd7ae61116fcc4036fe8f060c21117d032dfadecf6f64R282

const shellVersion = Config.PACKAGE_VERSION.split(".").map(Number).reduce(
  function(a, x) {
    return 100 * a + x;
  });

3.36.1 become 33601, but 40.0 become 4000

@mwilck mwilck merged commit 0f3a80f into mwilck:GNOME-3.36-compat May 20, 2021
@mwilck
Copy link
Owner

mwilck commented May 20, 2021

The shellVersion issue is addressed by #3.

@michel-slm michel-slm deleted the GNOME-40-compat branch September 30, 2022 18:14
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.

3 participants