-
-
Notifications
You must be signed in to change notification settings - Fork 13.7k
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
qt5: remove overrideScope' #245905
qt5: remove overrideScope' #245905
Conversation
Result of |
f1b691a
to
f6a944e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This issue indeed solves the problem. I think this should be backported to 23.05, so I'm adding the appropriate backport label.
@@ -329,9 +329,6 @@ let | |||
propagatedBuildInputs = [ self.qtbase.dev buildPackages.makeBinaryWrapper ] | |||
++ lib.optional stdenv.isLinux self.qtwayland.dev; | |||
} ../hooks/wrap-qt-apps-hook.sh; | |||
} // lib.optionalAttrs config.allowAliases { | |||
# remove before 23.11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
According to git blame
@jtojnar added this message.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In #222928 .
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added the alias and the comment when i converted the set to use makeScopeWithSplicing. The purpose was to remove the alias before 23.11 :P
Successfully created backport PR for |
Description of changes
Throw error when accessing
qt5.overrideScope'
, previously issued a warning and redirected toqt5.overrideScope
.Fixes: #245679
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/
)