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-pomodoro 0.24.1 -> 0.26.0 #345165

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

Conversation

Herschenglime
Copy link

Description of changes

This PR brings gnome-pomodoro up to the latest published version which is compatible with both Gnome 46 and 47. Both the application portion and the extension are functional.

The fix-schema-path.patch had to be changed, as the way that schemas were compiled upstream changed after v0.24 which broke the existing build steps. I also added a preFixup patch to compile these schemas manually.

Fixes #314999.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
    • NixOS test(s) (look inside nixos/tests)
    • and/or package tests
    • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
    • made sure NixOS tests are linked to the relevant packages
    • As far as I'm aware, there were not existing tests for this package and I did not add any. However, it was verified to function locally.
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@Aleksanaa
Copy link
Member

Just found out that I'm maintaining three pomodoro packages 😅 please also consider add yourself to maintainers

Copy link
Member

@Bot-wxt1221 Bot-wxt1221 left a comment

Choose a reason for hiding this comment

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

LGTM

@Herschenglime
Copy link
Author

Just found out that I'm maintaining three pomodoro packages 😅 please also consider add yourself to maintainers

Sure, I've added a commit making myself a maintainer on this package. Thanks for your review!

Herschenglime and others added 3 commits September 30, 2024 11:57
This commit adds myself as a maintainer to the gnome-pomodoro package in
addition to adding myself to the overall list.
Compile schemas manually to follow Nix conventions.

The gnome.post_install(glib_compile_schemas: true) param tries to
compile schemas in `<prefix>/<datadir>/glib-2.0/schemas`, which is not
compatible with the Nix way of handling gsettings schemas.

This commit patches that option out and manually adds a compilation step
to the preFixup hook, which fixes the issue and results in a fully
working local install of the program.
@Aleksanaa Aleksanaa changed the title gnome-pomodoro 0.24.1 → 0.26.0 gnome-pomodoro 0.24.1 -> 0.26.0 Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update request: gnome.pomodoro 0.24.1 → 0.25.1
4 participants