-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Triage and break up the global settings #5638
Copy link
Copy link
Open
Labels
UXThe way in which users interact with Nix. Higher level than UI.The way in which users interact with Nix. Higher level than UI.contributor-experienceDeveloper experience for Nix contributorsDeveloper experience for Nix contributorsnew-cliRelating to the "nix" commandRelating to the "nix" commandsettingsSettings, global flags, nix.confSettings, global flags, nix.conf
Milestone
Metadata
Metadata
Assignees
Labels
UXThe way in which users interact with Nix. Higher level than UI.The way in which users interact with Nix. Higher level than UI.contributor-experienceDeveloper experience for Nix contributorsDeveloper experience for Nix contributorsnew-cliRelating to the "nix" commandRelating to the "nix" commandsettingsSettings, global flags, nix.confSettings, global flags, nix.conf
Fields
Give feedbackNo fields configured for issues without a type.
Projects
Status
Defined work
Thinking through #5603, and the two
getDefaultSystemFeaturesreminded me of this:We have many global settings that should be per-store.
thisSytemandextraPlatformsare good examples. This works best after SplitStoreclass #5729 so that we don't pollute stores likeHttpBinaryCacheStorewith these new per-store options that don't make sense for them.We have too much ad-hoc logic around build remotes that should be fixed up to use per-store settings instead, as we did before with system features in 4720853
s3://...has.Separate stores for evaluation, building and storing the result #5025 (comment) remains a very good idea. Then many of the remaining settings which aren't per-store would make sense as an additional config parameter to this now-freestanding (not
Storemethod) function.Finally, there may be global settings that are for the libstore layer at all, and they can be moved to another downstream library.