Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Pass .o file paths to GHCi from c-sources (#756) #961

Merged
merged 1 commit into from
Sep 10, 2015

Conversation

chrisdone
Copy link
Member

This works with the hoogle codebase now.

Pinging @ndmitchell for merge.

@snoyberg want to give it a quick sanity check?

@ndmitchell
Copy link
Contributor

One for @snoyberg or someone else to merge. While I have merge access, it's more an accident than something I should ever really use!

snoyberg added a commit that referenced this pull request Sep 10, 2015
Pass .o file paths to GHCi from c-sources (#756)
@snoyberg snoyberg merged commit fee700e into master Sep 10, 2015
@chrisdone chrisdone deleted the 756-ghci-object-files branch September 10, 2015 07:32
@ndmitchell
Copy link
Contributor

Does this fix #756? I just tried after a stack upgrade --git, and Hoogle still fails with a stack ghci (after deleting the ghci.bat and .ghci files, to give it the best shot).

@chrisdone
Copy link
Member Author

@ndmitchell It's supposed to work, I tested it with hoogle. Can you lpaste the -v output?

@ndmitchell
Copy link
Contributor

Take a Hoogle repo (I have some changes, but I don't believe any are meaningful). Delete the .ghci and ghci.bat files. Windows. Output at https://gist.github.com/ndmitchell/a1cfa634c997684d3877

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants