Skip to content

docs: Fix mistake in cabal instructions and update stack-to-nix usage #73

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

Merged
merged 2 commits into from
Feb 26, 2019

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Feb 26, 2019

Closes #68

@rvl rvl changed the title docs: Fix small mistake in cabal instructions docs: Fix mistake in cabal instructions and update stack-to-nix usage Feb 26, 2019
@rvl rvl merged commit fa7d436 into master Feb 26, 2019
@rvl rvl deleted the rvl/fix-68 branch February 26, 2019 23:09
{ pkgs ? import <nixpkgs> {} }:

let
haskell = import (builtins.fetchTarball https://github.com/input-output-hk/haskell.nix/archive/master.tar.gz) { inherit pkgs; };
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

doesn't this contradict with overrideWith described in basic user guide?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, actually we need update cabal-to-nix code generation to be more similar to stack-to-nix, and delete those big chunks of boilerplate code in the Setup section.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And the proper way to override is with hackageSourceJSON and stackageSourceJSON? If so shouldn't they appear here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be a section in the user guide for pinning and overriding pins. But for now, #77.

andreabedini pushed a commit to andreabedini/haskell.nix that referenced this pull request Sep 14, 2022
* Fix buildkite issues

* Use default nixpkgs truncateIndex test. Update hash
andreabedini pushed a commit to andreabedini/haskell.nix that referenced this pull request Sep 14, 2022
* Fix buildkite issues

* Use default nixpkgs truncateIndex test. Update hash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants