Skip to content

Comments

Meson: init at 0.60.2#150486

Merged
AndersonTorres merged 6 commits intoNixOS:masterfrom
AndersonTorres:new-meson
Dec 18, 2021
Merged

Meson: init at 0.60.2#150486
AndersonTorres merged 6 commits intoNixOS:masterfrom
AndersonTorres:new-meson

Conversation

@AndersonTorres
Copy link
Member

@AndersonTorres AndersonTorres commented Dec 12, 2021

Motivation for this change

I found a new Wayland window manager that requires an updated Meson.

But I am a bit afraid of removing the old Meson release. We never know how many bugs and mass rebuilds an innocent new release can generate.

Therefore, I created a new Meson in parallel with the older one.

Things done
  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • 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/)
  • 22.05 Release Notes (or backporting 21.11 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
    • (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes
  • Fits CONTRIBUTING.md.

@ofborg ofborg bot added the 8.has: package (new) This PR adds a new package label Dec 12, 2021
@ofborg ofborg bot requested a review from jtojnar December 12, 2021 23:00
@ofborg ofborg bot added 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Dec 12, 2021
Copy link
Member

@jtojnar jtojnar left a comment

Choose a reason for hiding this comment

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

Not really a fan of such megaforks, they tend to be a maintenance nightmare. It would be much cleaner to just overrideAttrs since the majority of the expression remains the same.

@AndersonTorres
Copy link
Member Author

AndersonTorres commented Dec 13, 2021

I don´t think so. There are patches that needed to be rewritten.

Any case, as I have suggested, it will be a matter of "remove old Meson", possibly in a future staging merge.

@ofborg ofborg bot requested a review from jtojnar December 13, 2021 01:46
@ofborg ofborg bot requested a review from jtojnar December 14, 2021 16:09
Meson is a delicate package. It needs some patches to properly work in Nixpkgs
and many packages have it as a dependency, from many small applications to
systemd (yay, mass rebuilds in sight!). Updating it is not a trivial and
harmless task.

Therefore, I took the paranoid way and encapsulate the current version in a
subdir, instead of the regular write-over procedure. This way, at the best we
can just remove the whole dir in the future, and at the worse we just maintain
duplicated code.
A very ugly decision: in order to register the diff between 0.57 and 0.60
releases of Meson in Nixpkgs, I copied the whole directory before actually write
it by hand.
Concatenation of lists in Python is done by the `+` operator.
The comment should be self-explaining now.
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Dec 14, 2021
@alyssais
Copy link
Member

I've marked #144779 as closing this PR. If that's incorrect feel free to remove the "Closes" line from that PR's body.

@AndersonTorres
Copy link
Member Author

I feel this partially closes #144779 instead. That being said, I will merge this first.

@AndersonTorres AndersonTorres merged commit a860b94 into NixOS:master Dec 18, 2021
@primeos primeos mentioned this pull request Dec 18, 2021
13 tasks
@alyssais
Copy link
Member

I wish you'd just merged the other PR — there was nothing more to do, and you asked if anybody objected to it being merged, and nobody said anything. Now merging the other one is even harder because there's a big mess of merge conflicts.

@AndersonTorres
Copy link
Member Author

The other PR was more confusing: more than 20 files changed.
On the other hand, this PR enables other updates more easily.

@alyssais alyssais mentioned this pull request Jan 4, 2022
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: package (new) This PR adds a new package 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants