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

Ensure libffi.map is generated #321

Closed
wants to merge 1 commit into from
Closed

Ensure libffi.map is generated #321

wants to merge 1 commit into from

Conversation

@bgamari
Copy link

bgamari commented Oct 2, 2017

This file is necessary on Linux it seems.

@tromey

This comment has been minimized.

Copy link
Member

tromey commented Oct 9, 2017

I don't think libffi.map.in is meant to be processed by config.status. Instead I see a target in Makefile.am:

libffi.map: $(top_srcdir)/libffi.map.in
	$(COMPILE) -D$(TARGET) -E -x assembler-with-cpp -o $@ $<

Could you say what problem you are seeing? Maybe there is a dependency missing or an ordering problem or something like that.

bgamari added a commit to ghc/libffi-tarballs that referenced this pull request Oct 20, 2017
@atgreen

This comment has been minimized.

Copy link
Member

atgreen commented Oct 25, 2017

@bgamari what error makes you think you need this change? Is libffi_version_dep not set to libffi.map in your builds for some reason?

@atgreen

This comment has been minimized.

Copy link
Member

atgreen commented Mar 17, 2018

I'm going to close this because, as @tromey notes, libffi.map is created by the build process, not the configure process. @bgamari - please reopen this ticket if you disagree, or if there's some other bug that is preventing libffi.map from being created at build time.

Thanks!

@atgreen atgreen closed this Mar 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.