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

Gnome extensions tophat pathfixes #267160

Closed
wants to merge 3 commits into from

Conversation

lugeha
Copy link

@lugeha lugeha commented Nov 13, 2023

Description of changes

The tophat gnome extension needs a few more search path patches for gtop to get settings to work.

Things done

  • Built on platform(s)

    • [*] x86_64-linux
  • [*] Tested basic functionality of all binary files (usually in ./result/bin/)

Add path fixes to prefs.js and lib/shared.js

Options are now available again in the extension.

@github-actions github-actions bot added the 6.topic: GNOME GNOME desktop environment and its underlying platform label Nov 13, 2023
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 labels Nov 13, 2023
@piegamesde
Copy link
Member

(How) does this interact with the GNOME 45 update?

@lugeha
Copy link
Author

lugeha commented Nov 13, 2023

Looking at the porting guide. Most gnome extensions are going to require a rewrite anyway. I think, in that regard, the point is moot.

@piegamesde
Copy link
Member

Hi, GNOME 45 finally got merged. Could you please rebase and test again?

@lugeha
Copy link
Author

lugeha commented Nov 24, 2023

Version in nixpkgs isn't even compatible, so... no?

@piegamesde
Copy link
Member

As of last week, the extension supports Gnome 45 and got updated here in #270315.

@fernandocarletti
Copy link

Hi! I'm just onboarding on NixOS and had the issue as well. Is this ready to be merged? If not, is there an alternative to the fix? I'm not very familiar with Gnome Extensions but if there is a way I could help with some work on it I'd appreciate some guidance.

@wegank wegank added the 2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md label Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.status: stale https://github.com/NixOS/nixpkgs/blob/master/.github/STALE-BOT.md 6.topic: GNOME GNOME desktop environment and its underlying platform 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants