-
-
Notifications
You must be signed in to change notification settings - Fork 13k
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
flatpak-builder: 1.2.3 -> 1.4.2 #280152
flatpak-builder: 1.2.3 -> 1.4.2 #280152
Conversation
26172be
to
b623ca4
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review/3032/3265 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I cancelled the nixpkgs-review at 400 successful builds, These changes looks very good to me.
Would you be interested in maintaining this package? It's okay to say no
Yeah, I can maintain it. |
b623ca4
to
d1ff0ac
Compare
d1ff0ac
to
4fa5330
Compare
The tarball provided is using an old/incomplete version of the libglnx, I'm talking with upstream about this to see what can be done. Edit: upstream provided a fixed tarball. |
4fa5330
to
5f82728
Compare
Result of 7 packages marked as broken and skipped:
1 package blacklisted:
8 packages failed to build:
765 packages built:
All failing packages also fail on hydra except:
LGTM |
Could you retarget the PR to staging? Make sure to mark it as draft first in case your master is ahead of staging, so lots of people don't get pinged for review by ofborg |
5f82728
to
c0aefce
Compare
Done. |
Description of changes
Changelog: https://github.com/flatpak/flatpak-builder/releases/tag/1.4.2
Closes: #279784
I replaced the old build tool with meson, as recommended by upstream. Needed to adapt appstream to build with compose support, so flatpak-builder can use it. Also, updated the patch to fix paths and modified the test one, and also removed unused parts.
Btw should I set a withCompose in appstream and make it optional?
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.