Skip to content

Comments

[Backport release-21.11] irods: Don't use builtins.nixVersion#156552

Merged
thiagokokada merged 1 commit intorelease-21.11from
backport-156527-to-release-21.11
Jan 24, 2022
Merged

[Backport release-21.11] irods: Don't use builtins.nixVersion#156552
thiagokokada merged 1 commit intorelease-21.11from
backport-156527-to-release-21.11

Conversation

@github-actions
Copy link
Contributor

Bot-based backport to release-21.11, triggered by a label in #156527.

  • Before merging, ensure that this backport complies with the Criteria for Backporting.
    • Even as a non-commiter, if you find that it does not comply, leave a comment.

The value of builtins.nixVersion should never be used except to bail
out if it's too old. It causes the evaluation result to depend on the
version of Nix, so e.g. the binary cache doesn't work.

(cherry picked from commit 5762b8c)
@ofborg ofborg bot requested a review from bzizou January 24, 2022 16:56
@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Jan 24, 2022
@thiagokokada thiagokokada merged commit b1f2623 into release-21.11 Jan 24, 2022
@edolstra edolstra deleted the backport-156527-to-release-21.11 branch January 24, 2022 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants