-
-
Notifications
You must be signed in to change notification settings - Fork 14.5k
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
Pantheon: prepare for meson 0.61 #154837
Pantheon: prepare for meson 0.61 #154837
Conversation
With one last patch applied on elementary-screenshot, the test passed. |
430367a
to
b8cf273
Compare
@ofborg test pantheon |
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
i18n.merge_file has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61
b8cf273
to
d277f13
Compare
Result of 1 package marked as broken and skipped:
33 packages built:
|
Motivation for this change
I am not sure whether upstream will make some release for these fixes before elementary OS 7 so I opened this just to make sure if someone wants to bump meson, it won't break Pantheon. To build Pantheon with meson 0.61, apply this patch on top of this PR.
Packages mostly fails with "Function does not take positional arguments" when using meson 0.61. As what upstream change says,
i18n.merge_file
has been ignoring positional arguments for a time and explicitly rejects with error since meson 0.61, and actually this has been marked as deprecated in 0.60. So it is still a good idea to fetch these fixes in advance.To make sure at least nixosTests.pantheon passes:
Things done
sandbox = true
set innix.conf
? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)nixos/doc/manual/md-to-db.sh
to update generated release notes