Skip to content
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

Assume verbosity as silent when invoked in interpreter mode #1472

Closed
harendra-kumar opened this issue Dec 5, 2015 · 3 comments · Fixed by #1487
Closed

Assume verbosity as silent when invoked in interpreter mode #1472

harendra-kumar opened this issue Dec 5, 2015 · 3 comments · Fixed by #1487

Comments

@harendra-kumar
Copy link
Collaborator

Please refer to #1394 for the original discussion.

  • Currently stack prints some output of its own before running the script:

Run from outside a project, using implicit global project config
Using resolver: lts-3.14 from implicit global project's config file: /var/home/harendra/.stack/global-project/stack.yaml

When used as interpreter no other output is desirable other than the script's output itself. In fact this should hold true for any stack exec command. That might interfere with the processing of the script output.

We can fix this by assuming --verbosity silent when invoked as interpreter.

@mgsloan
Copy link
Contributor

mgsloan commented Dec 6, 2015

Yup, makes sense to me! Thanks for opening an issue

@mgsloan mgsloan added this to the P2: Should milestone Dec 6, 2015
harendra-kumar added a commit to harendra-kumar/stack that referenced this issue Dec 11, 2015
After the fix for commercialhaskell#1472 the guidance about --verbosity silent is not needed.
@sjakobi
Copy link
Member

sjakobi commented Feb 6, 2016

This just bit me, when I tried to do some simple timings with a script and see what stack internally does.

I ended up running stack -v runghc script.hs but I'm not entirely sure it does the same thing.

Wouldn't it be enough to have --verbosity silent as a default for the interpreter mode but make it overrideable?

@harendra-kumar
Copy link
Collaborator Author

I am closing this. Let's keep track of fixing the override issue in #2343.

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

Successfully merging a pull request may close this issue.

3 participants