-
Notifications
You must be signed in to change notification settings - Fork 701
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
s/nix style/v2-build/ in the documentation #6105
Comments
An example of confusion: Nix store can (and does) mean |
Agreed. |
+1, but it should also mention the other temporary names (nix-style local builds, new-*) so that at least it shows up in searches |
But do we want to keep referring to I for one intend to keep referring to "Nix-style {store,hash,build}" (with an emphasis on "-style") where I refer to the concept related to the Nix-style store/hashes, as well as start to slowly phase out my use of "new-build" and "v2-build" in favor of just saying |
I think @hvr did veto this. Nix-style it is. For the future: don't use number, nor new, not foo-like "relative" references. Find some fancy word, the first thing you'll pick will stick. |
Actually no. https://cabal.readthedocs.io/en/latest/nix-local-build-overview.html starts with
That needs to be changed in the way or another. Either
Having consistent language would make documentation more polished almost for free. |
well, note that
was changed into that wording (back then still for
-``cabal new-build``, also known as Nix-style local builds, is a new
-command inspired by Nix that comes with cabal-install 1.24. Nix-style
-local builds combine the best of non-sandboxed and sandboxed Cabal:
+Nix-style local builds are a new build system implementation inspired by Nix.
+The Nix-style local build system is commonly called "new-build" for short after the ``cabal new-*`` family of commands that control it.
+However those names are only temporary until Nix-style local builds becomes the default. and also note specifically
|
Sorry, let me rephrase: Nix-style local builds have become the default, so some more editing is needed there |
Adding my 2c as requested by #haskell (and my apologies, it really is 2c or less, I'm not here to work on this). While they arose for understandable reasons, nowadays both "nix-style" and "v2-build" just create confusion for non-experts:
IMHO it would be good for cabal if they were both discontinued ASAP (this issue was mainly about removing the "nix style" term). It would take 1. sustained consistent effort across all docs/uis/fora, and 2. a new replacement term that is sticky and liked by both the devs and new users. |
I think "v1-style build" covers two build/package management strategies:
And "v2/nix style" covers:
|
i was suggesting neutral terms might be "packagedb-style builds" (for v1) and "store-style builds" (for v2) since this refers to where the packages live, rather than any descriptors of how stuff works. |
packagedb vs store is informative to cabal developers, but not to users. Ultimately it needs to be three easy words for the three cases. Currently I'd favour
|
Agreed about getting rind of "nix". I think the "sandboxed" style is completely removed, right? No longer possible. So, nothing wrong with the term, but it should probably never appear in docs nor codebase (outside old release notes). The "v1 style", "v2 style" naming has the advantage that there still are, and will be for at least a few months, "v1-" and "v2-" prefixes in command names. There is also the pair "" and "new-", but the first element of the pair is ambiguous. There is a slight ambiguity with "global", because we are actively working on bringing it back using v2- machinery, because some use cases require it (the tool |
I conjecture that mentioning Nix only adds confusion. Almost all people I know use
v2-build
(well actually they usenew-build
, but maybev2-build
is a bit better).E.g.
v2-build
local builds is the distinction between local packages, which users edit and recompile and must be built per-project, versus external packages, which can be cached across projects.The text was updated successfully, but these errors were encountered: