You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I won't pretend to have the firmest grasp of this, but nixpkgs reviewers are leaning heavily on PR authors to use the mkDerivation (finalAttrs: ...) pattern first added in:
Since resholve's mkDerivation will need work to support this and it doesn't look immediately tractable, there'll probably be an increasing number of people getting asked to implement this (as in NixOS/nixpkgs#224881 (comment)) only to run into a dead end for now.
The text was updated successfully, but these errors were encountered:
I won't pretend to have the firmest grasp of this, but nixpkgs reviewers are leaning heavily on PR authors to use the
mkDerivation (finalAttrs: ...)
pattern first added in:After skimming related PRs, my best guess is that I'll want to wait to tackle this until the following PR implementing
lib.extendMkDerivation
lands:Since resholve's mkDerivation will need work to support this and it doesn't look immediately tractable, there'll probably be an increasing number of people getting asked to implement this (as in NixOS/nixpkgs#224881 (comment)) only to run into a dead end for now.
The text was updated successfully, but these errors were encountered: