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
all-packages.nix: move defaults to package files continues^2 #56026
all-packages.nix: move defaults to package files continues^2 #56026
Conversation
These are already inherited in the parent derivation.
, cudaSupport ? stdenv.isLinux, cudatoolkit | ||
, cudnnSupport ? false, cudnn ? null | ||
, cudaSupport ? config.cudaSupport or false, cudatoolkit | ||
, cudnnSupport ? cudaSupport, cudnn ? null |
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.
Do you want to change the default on Linux here? Given it is Deep Learning, running it CPU-only might be too painful to be useful…
}: | ||
, firmware ? config.pcmciaUtils.firmware or [] # Special pcmcia cards. | ||
, configOpts ? config.pcmciaUtils.config or null # Special hardware (map memory & port & irq) | ||
}: # used to generate postInstall script. |
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 wonder how to find out how many users will be hit by such an incompatible rename.
Do you want to change the default on Linux here? Given it is Deep Learning, running it CPU-only might be too painful to be useful…
The default doesn't actually change, see the related piece in `all-packages.nix`.
I wonder how to find out how many users will be hit by such an incompatible rename.
`config.pcmciaUtils` option name does not change, package derivation option name change does not really matter because if somebody has it set via override they'll just get a evaluation error.
|
What? Why?
A continuation of #55129 and #55732. Another noop.
git log
A bunch of these.
xmonad_log_applet: move defaults to package file, cleanup
These are already inherited in the parent derivation.
nix-instantiate
environmentnix-env -qaP
diffs/cc @7c6f434c from #55129 and @bendlas, @volth, @samueldr from #55732