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

Test failure: #214: Changing linker flags #244

Closed
Mikolaj opened this issue Sep 8, 2014 · 4 comments
Closed

Test failure: #214: Changing linker flags #244

Mikolaj opened this issue Sep 8, 2014 · 4 comments
Milestone

Comments

@Mikolaj
Copy link
Collaborator

Mikolaj commented Sep 8, 2014

PATH=/bin:/usr/bin dist/build/TestSuite/TestSuite \
  --extra-paths-74 ~/env/fpco-patched-7.4/local/bin:~/env/fpco-patched-7.4/dot-cabal/bin:~/env/ghc742/dot-cabal/bin \
  --package-db-74 ~/env/fpco-patched-7.4/dot-ghc/snippet-db \
  --extra-paths-78 ~/env/fpco-patched-7.8/local/bin:~/env/fpco-patched-7.8/dot-cabal/bin:~/env/ghc742/dot-cabal/bin \
  --package-db-78 ~/env/fpco-patched-7.8/dot-ghc/snippet-db \
  --test-78 \
  -j1 \
  -p 214
 #214: Changing linker flags:                                                                          FAIL
    expected: RunOk
     but got: RunGhcException "External exception: Server killed. Local exception: user error (demandInput: not enough bytes)"
@Mikolaj Mikolaj added this to the Version 0.9 milestone Sep 8, 2014
@edsko
Copy link
Collaborator

edsko commented Sep 8, 2014

@Mikolaj are you sure you are running the latest server? I thought I fixed this in #214 -- and I'm not getting failures here. That said, #214 is turning out to be a tricky issue that I thought was fixed in the past when it wasn't, so entirely possible there's still a problem somewhere.

@edsko
Copy link
Collaborator

edsko commented Sep 12, 2014

This is only happening on Linux, and not on OSX. I have reproduced this on a VM. Gdb tells me that this is crashing with SIGSEGV in fputs.

edsko added a commit that referenced this issue Sep 12, 2014
This closes #244, I think, pending review by Duncan.
@edsko
Copy link
Collaborator

edsko commented Sep 12, 2014

@dcoutts, @Mikolaj : I think I found the source of the problem: we were not generating position independent code. I have now modified the code that so that always (unconditionally) generate position independent code; @dcoutts , it would be great if you could review the patch (820f980) -- it's a one line change.

@edsko
Copy link
Collaborator

edsko commented Sep 12, 2014

(Note to self: http://stackoverflow.com/a/2422063/742991 is useful when debugging generated assembly code under gdb)

@edsko edsko closed this as completed Sep 13, 2014
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

No branches or pull requests

2 participants