Skip to content

Safer installation from source in INSTALL.md - #2294

Merged
paf31 merged 2 commits into
purescript:masterfrom
hdgarrood:dont-suggest-nightly
Sep 2, 2016
Merged

Safer installation from source in INSTALL.md#2294
paf31 merged 2 commits into
purescript:masterfrom
hdgarrood:dont-suggest-nightly

Conversation

@hdgarrood

Copy link
Copy Markdown
Contributor

This commit ensures that the stackage snapshot used when building from
source following the instructions in INSTALL.md will be the same as the
one used in development and in binary bundles.

This is safer (because you'll end up with a set of dependencies which
have been tested together) and it also means you'll always get the
latest version according to Hackage. In comparison, the Stackage nightly
is often behind because of waiting for purescript or other libraries to
make changes to allow all packages in the nightly to build together.

See also discussion in #2278.

@hdgarrood

Copy link
Copy Markdown
Contributor Author

Note also that this brings INSTALL.md in line with what's already on the website.

@kritzcreek

Copy link
Copy Markdown
Member

Not that it needs to happen in this PR, but the CONTRIBUTING.md file is also out of date (it still lists cabal --enable-tests ...).

@hdgarrood

Copy link
Copy Markdown
Contributor Author

Oh thanks for the heads up, might as well take care of that now too.

@hdgarrood

Copy link
Copy Markdown
Contributor Author

Oops, accidentally pushed this most recent to commit to master (but reverted again).

This commit ensures that the stackage snapshot used when building from
source following the instructions in INSTALL.md will be the same as the
one used in development and in binary bundles.

This is safer (because you'll end up with a set of dependencies which
have been tested together) and it also means you'll always get the
latest version according to Hackage. In comparison, the Stackage nightly
is often behind because of waiting for purescript or other libraries to
make changes to allow all packages in the nightly to build together.
@paf31
paf31 merged commit ab5f139 into purescript:master Sep 2, 2016
@paf31

paf31 commented Sep 2, 2016

Copy link
Copy Markdown
Contributor

Thanks for this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants