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
Remove composableDerivation, closes #18763 #44116
Conversation
Should we provide an explanation (with assert) if someone references it? |
The helper functions |
Thanks for checking. Fixed. |
lib/default.nix
Outdated
@@ -123,14 +123,14 @@ let | |||
traceShowValMarked showVal traceCall traceCall2 traceCall3 | |||
traceValIfNot runTests testAllTrue traceCallXml attrNamesToStr; | |||
inherit (misc) maybeEnv defaultMergeArg defaultMerge foldArgs | |||
defaultOverridableDelayableArgs composedArgsAndFun | |||
composedArgsAndFun |
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.
composedArgsAndFun
is removed from lib/deprecated.nix
below.
lib/deprecated.nix
Outdated
@@ -285,7 +217,7 @@ rec { | |||
# }; | |||
# will result in | |||
# { mergeAttrsBy = [...]; buildInputs = [ a b c d ]; } | |||
# is used by prepareDerivationArgs, defaultOverridableDelayableArgs and can be used when composing using | |||
# is used by prepareDerivationArgs and can be used when composing using |
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.
prepareDerivationArgs
is removed as well
maybeAttrNullable maybeAttr ifEnable checkFlag getValue | ||
checkReqs uniqList uniqListExt condConcat lazyGenericClosure | ||
innerModifySumArgs modifySumArgs innerClosePropagation | ||
closePropagation mapAttrsFlatten nvs setAttr setAttrMerge | ||
mergeAttrsWithFunc mergeAttrsConcatenateValues | ||
mergeAttrsNoOverride mergeAttrByFunc mergeAttrsByFuncDefaults | ||
mergeAttrsByFuncDefaultsClean mergeAttrBy prepareDerivationArgs | ||
nixType imap overridableDelayableArgs; |
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.
prepareDerivationArgs
(L133) is removed
4ff449c
to
11e91d2
Compare
Pushed an update. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)