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.

Sign up for GitHub

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

Jump to bottom

Send output of building Setup.hs to stderr #1410

Closed
mgsloan opened this issue Nov 24, 2015 · 0 comments · Fixed by #1423
Closed

Send output of building Setup.hs to stderr #1410

mgsloan opened this issue Nov 24, 2015 · 0 comments · Fixed by #1423
Assignees
@rvion
Labels
awaiting pull request newcomer friendly type: bug
Milestone
P2: Should

Comments

@mgsloan
Copy link
Contributor

mgsloan commented Nov 24, 2015

Its usage of runIn causes it to inherit stdout and stderr. For example, here's some integration test output for a failing case:

       uncaught exception: TestFailure (Exited with ExitFailure 1

       stdout:
       [1 of 1] Compiling Main             ( /tmp/stack31370/Setup.hs, /tmp/stack31370/Setup.o )
       Linking /tmp/stack-integration-home31337/.stack/setup-exe-cache/tmp-setup-Simple-Cabal-1.22.4.0-x86_64-linux-ghc-7.10.2 ...


       stderr:
       Running: /home/mgsloan/fpco/stack/.stack-work/install/x86_64-linux/lts-3.14/7.10.2/bin/stack build
       Downloading lts-3.9 build plan ...
       Downloaded lts-3.9 build plan.
       Caching build plan
@mgsloan mgsloan added this to the P2: Should milestone Nov 24, 2015
@rvion rvion self-assigned this Nov 24, 2015
@rvion rvion mentioned this issue Nov 24, 2015
Merged
rvion added a commit to rvion/stack that referenced this issue Nov 24, 2015
@rvion
actually fixes commercialhaskell#1410
9363eff
rvion added a commit to rvion/stack that referenced this issue Nov 25, 2015
@rvion
actually fixes commercialhaskell#1410
514b66c
rvion added a commit to rvion/stack that referenced this issue Nov 25, 2015
@rvion
actually fixes commercialhaskell#1410
147ff23
rvion added a commit to rvion/stack that referenced this issue Nov 25, 2015
@rvion
actually fixes commercialhaskell#1410
4923a3f
rvion added a commit to rvion/stack that referenced this issue Nov 29, 2015
@rvion
fix commercialhaskell#1410 and refactor runIn, callProcess, etc. to t…
332636b 
…ake a Cmd arg

introduce CMD data type
add runCmd' following callProcess' model
actually fixes commercialhaskell#1410
rename CMD -> Cmd
move Cmd to System.Process.Run
clean redundant imports
remove test.dot and test.html
fix callProcess call on windows in Stack.Exec
mgsloan added a commit that referenced this issue Nov 30, 2015
@mgsloan
Merge pull request #1423 from rvion/Send-output-of-building-Setup-1410
8c02285 
[RFC] Send output of building setup to stderr (fixes #1410)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rvion rvion

Labels
awaiting pull request newcomer friendly type: bug
Projects
None yet
Milestone
P2: Should
Development

Successfully merging a pull request may close this issue.

[RFC] Send output of building setup to stderr (fixes #1410) rvion/stack
4 participants
@mgsloan @snoyberg @rvion @borsboom