Skip to content

{clang,gcc}_multi: remove manual throws#426902

Merged
Ericson2314 merged 1 commit intoNixOS:masterfrom
wolfgangwalther:ci-eval-asserts-multi
Jul 21, 2025
Merged

{clang,gcc}_multi: remove manual throws#426902
Ericson2314 merged 1 commit intoNixOS:masterfrom
wolfgangwalther:ci-eval-asserts-multi

Conversation

@wolfgangwalther
Copy link
Contributor

@wolfgangwalther wolfgangwalther commented Jul 20, 2025

This avoids hacky workarounds in CI when generating attrpaths for non-x86_64-linux platforms.

Evaluation for these two attributes will not succeed on platforms other than Linux anyway, because:

  • The unsupported system error from glibc-nolibgcc is propagated.
  • pkgsi686Linux.gcc triggers an assert anyway.

Related: #426629.

Similar to #426645.

Things done


Add a 👍 reaction to pull requests you find important.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Jul 20, 2025
@wolfgangwalther

This comment was marked as outdated.

@wolfgangwalther wolfgangwalther marked this pull request as draft July 20, 2025 12:33
@wolfgangwalther wolfgangwalther force-pushed the ci-eval-asserts-multi branch from 262d15d to 52b5960 Compare July 21, 2025 17:40
@wolfgangwalther wolfgangwalther requested a review from a team July 21, 2025 17:40
@wolfgangwalther wolfgangwalther marked this pull request as ready for review July 21, 2025 17:40
@wolfgangwalther wolfgangwalther changed the title {clang,gcc}_multi: only throw with aliases {clang,gcc}_multi: set meta.platforms instead of throwing Jul 21, 2025
@wolfgangwalther wolfgangwalther force-pushed the ci-eval-asserts-multi branch from 52b5960 to 7ee6939 Compare July 21, 2025 17:44
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. and removed 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. labels Jul 21, 2025
@wolfgangwalther wolfgangwalther force-pushed the ci-eval-asserts-multi branch from 7ee6939 to 48c858e Compare July 21, 2025 17:56
@wolfgangwalther wolfgangwalther changed the title {clang,gcc}_multi: set meta.platforms instead of throwing {clang,gcc}_multi: remove manual throws Jul 21, 2025
This avoids hacky workarounds in CI when generating attrpaths for
non-x86_64-linux platforms.

Evaluation for these two attributes will not succeed on platforms other
than Linux anyway, because:
- The unsupported system error from `glibc-nolibgcc` is propagated.
- `pkgsi686Linux.gcc` triggers an assert anyway.
@wolfgangwalther wolfgangwalther force-pushed the ci-eval-asserts-multi branch from 48c858e to 63e69e4 Compare July 21, 2025 17:57
@Ericson2314
Copy link
Member

This makes sense to me. GCC NG will also improve this situation.

@Ericson2314 Ericson2314 merged commit 524ced7 into NixOS:master Jul 21, 2025
23 of 27 checks passed
@wolfgangwalther wolfgangwalther deleted the ci-eval-asserts-multi branch July 21, 2025 18:44
@nixpkgs-ci
Copy link
Contributor

nixpkgs-ci bot commented Jul 22, 2025

Successfully created backport PR for release-25.05:

@github-actions github-actions bot added the 8.has: port to stable This PR already has a backport to the stable release. label Jul 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

8.has: port to stable This PR already has a backport to the stable release. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants