Skip to content

qt6.qtbase: prepare for structuredAttrs#472656

Merged
philiptaron merged 1 commit intoNixOS:stagingfrom
SFrijters:qt6-base-replacevars-structuredattrs
Jan 31, 2026
Merged

qt6.qtbase: prepare for structuredAttrs#472656
philiptaron merged 1 commit intoNixOS:stagingfrom
SFrijters:qt6-base-replacevars-structuredattrs

Conversation

@SFrijters
Copy link
Member

@SFrijters SFrijters commented Dec 20, 2025

#237216

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

@SFrijters SFrijters marked this pull request as draft December 20, 2025 10:00
@SFrijters SFrijters force-pushed the qt6-base-replacevars-structuredattrs branch from 048db42 to 2174d8d Compare December 20, 2025 10:07
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 1001-2500 This PR causes many rebuilds on Linux and should target the staging branches. labels Dec 20, 2025
@SFrijters SFrijters force-pushed the qt6-base-replacevars-structuredattrs branch from 2174d8d to 3875954 Compare December 23, 2025 00:00
@SFrijters
Copy link
Member Author

Also requires #473906 .

@SFrijters SFrijters marked this pull request as ready for review December 24, 2025 23:27
@SFrijters SFrijters force-pushed the qt6-base-replacevars-structuredattrs branch from 3875954 to e69dd2e Compare December 24, 2025 23:29
@SFrijters SFrijters force-pushed the qt6-base-replacevars-structuredattrs branch from e69dd2e to ceec85b Compare December 24, 2025 23:33
@SFrijters SFrijters changed the title qt6.qtbase: use replaceVars for setup hook qt6.qtbase: prepare for structuredAttrs Dec 24, 2025
@SFrijters SFrijters requested a review from philiptaron January 30, 2026 11:26
Copy link
Contributor

@philiptaron philiptaron left a comment

Choose a reason for hiding this comment

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

Change LGTM, doing a build

@nixpkgs-ci nixpkgs-ci bot added the 12.approvals: 1 This PR was reviewed and approved by one person. label Jan 30, 2026
@K900
Copy link
Contributor

K900 commented Jan 30, 2026

Diff LGTM, how the hell did this ever work actually.

@philiptaron philiptaron added this pull request to the merge queue Jan 31, 2026
Merged via the queue into NixOS:staging with commit 38945fd Jan 31, 2026
33 checks passed
@SFrijters SFrijters deleted the qt6-base-replacevars-structuredattrs branch February 1, 2026 20:24
@Stebalien Stebalien mentioned this pull request Feb 25, 2026
3 tasks
Stebalien added a commit to Stebalien/nixpkgs that referenced this pull request Feb 28, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
wrvsrx pushed a commit to wrvsrx/nixpkgs that referenced this pull request Feb 28, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
Stebalien added a commit to Stebalien/nixpkgs that referenced this pull request Mar 1, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
wrvsrx pushed a commit to wrvsrx/nixpkgs that referenced this pull request Mar 1, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
Stebalien added a commit to Stebalien/nixpkgs that referenced this pull request Mar 2, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
Stebalien added a commit to Stebalien/nixpkgs that referenced this pull request Mar 2, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
wrvsrx pushed a commit to wrvsrx/nixpkgs that referenced this pull request Mar 2, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
Stebalien added a commit to Stebalien/nixpkgs that referenced this pull request Mar 2, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
wrvsrx pushed a commit to wrvsrx/nixpkgs that referenced this pull request Mar 3, 2026
This applies a workaround suggested by @wrvsrv. Unfortunately, the
correct fix (NixOS#472656) requires a mass rebuild so it'll take a while to
pass through staging.

fixes NixOS#493843
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 101-500 This PR causes between 101 and 500 packages to rebuild on Darwin. 10.rebuild-linux: 501+ This PR causes many rebuilds on Linux and should normally target the staging branches. 10.rebuild-linux: 1001-2500 This PR causes many rebuilds on Linux and should target the staging branches. 12.approvals: 1 This PR was reviewed and approved by one person.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants