-
Notifications
You must be signed in to change notification settings - Fork 841
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
stack ghci should be usable with minimal dependencies #817
Comments
Running it with |
Hm, or it's supposed to. It seems not to be working properly:
|
I made a spin-off issue here: #818 |
Makes sense. I don't agree with this default, though. (Has it always been this way?) I believe that people will be using stack as package end-users (e.g. for local installs of packages) more than as package developers, so I'd optimize for the former case while keeping the latter case in easy reach. Here's behavior I'd expect (I don't know how much of it is already implemented but I couldn't find in the documentation, or how much of it is even a good idea):
|
Okay, #818 is fixed. |
Actually, thinking about it,
Agree, but that's separate work from this ticket.
Not necessary if we implement as above. |
There is an ongoing discussion in #815 which will need to be resolved first. |
Okay.
Is that ticket already opened? Should I open it? |
This may be relevant for target syntax: On Tue, Aug 18, 2015, 11:44 PM Chris Done notifications@github.com wrote:
|
@snoyberg That was helpful. There doesn't seem to be a way to do something like |
Well, no. It could be added, and would be known to only work for the case On Tue, Aug 18, 2015, 11:56 PM Alex R notifications@github.com wrote:
|
Not that I know of, feel free to open separate issues for those things! |
I'm ambivalent; I think if you're loading up a package source tree into GHCi then you're developing on it and you probably want the test suite ready to go. If you just want to try out a package, like turtle, you can do
|
OTOH loading just the lib and exes would be consistent with |
Fixes regression caused by adding the build line.
Not so much "want to avoid building executables" as "the package doesn't provide executables"; I feel like even if it's redundant, it's good to have an explicit way to do the default behavior, especially if the default may change. |
Closing due to lack of activity. Also, it seems like the original issue was resolved. |
After upgrading to the newest version of stack (prompted by @chrisdone in the thread for #665), I ran
stack ghci
in theturtle
project directory. Rather than failing (which was the previous undesirable behavior), it started building the dependencies for the test and bench suites. While this is clearly an improvement, I expect to be able to run ghci without necessarily installing the dependencies for the test and bench suites, and I think that should be the default behavior.The text was updated successfully, but these errors were encountered: