Skip to content

Conversation

@snoyberg
Copy link
Contributor

@snoyberg snoyberg commented Aug 8, 2017

Note: Documentation fixes for https://docs.haskellstack.org/en/stable/ should target the "stable" branch, not master.

Please include the following checklist in your PR:

  • Any changes that could be relevant to users have been recorded in the ChangeLog.md
  • The documentation has been updated, if necessary.

Please also shortly describe how you tested your change. Bonus points for added tests!

@snoyberg
Copy link
Contributor Author

snoyberg commented Aug 8, 2017

@mgsloan @borsboom Will you review the comments I've put in the changelog in this PR before I merge? I think this is the right call, but would appreciate a second (and third) opinion.

@snoyberg snoyberg requested review from borsboom and mgsloan August 8, 2017 14:11
@borsboom
Copy link
Contributor

borsboom commented Aug 8, 2017

So if I read this correctly, this doesn't implement the local-only option mentioned in #849 (comment).

In a way, this seems similar to setting flags on a package, but IIRC that doesn't automatically promote a package to local (instead, it requires you to make the package an extra-dep). I'm wondering if there's a justification for that difference in behaviour or if it's just by accident.

@snoyberg
Copy link
Contributor Author

snoyberg commented Aug 9, 2017

Just to unify the discussion from here and the issue: I think we're in agreement that this is a fair first step, and are considering some additional changes. Let me document them here:

  • Some way to apply flags in a stack.yaml file to all local packages. @mgsloan mentioned a $locals key. I like that, and would move to replace * with $everything to be more explicit (with a warning printed for the old syntax).
  • A broader question about implicit snapshots. We could add them in under @mgsloan's idea of only having project packages and things depending on them end up in the local database. I'm open to revisiting that. And once that's in place, yes, it would make sense to promote all packages to a custom snapshot.

My recommendation is, in the interest of keeping the changesets manageable, we consider if this change is an improvement. If so, let's merge and open new issues for these enhancements.

@mgsloan
Copy link
Contributor

mgsloan commented Aug 9, 2017

@snoyberg Yep, that seems reasonable to me!

@snoyberg
Copy link
Contributor Author

snoyberg commented Aug 9, 2017

OK, two new issues opened. I'm going to merge this so I can get started on the other two.

@snoyberg snoyberg merged commit f3ae87f into master Aug 9, 2017
@snoyberg snoyberg deleted the 849-unify-flags-ghc-options branch August 9, 2017 06:20
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.

4 participants