-
Notifications
You must be signed in to change notification settings - Fork 710
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
Forward declare open_temp_exec_file #764
Forward declare open_temp_exec_file #764
Conversation
It's defined in closures.c and used in tramp.c. Also declare it as an hidden symbol, as it should be.
aead3c5
to
cbfb9b4
Compare
Fixes #760. Thank you! |
Thanks for the quick merge \o |
We’re also running into this. Would be great to see a new library release containing this. 🙏 |
Please make a new release containing this fix. The current version of libffi has not been buildable with the current version of clang for nearly a year now, and you can see from the linked issues (both here and in #760) that this has affected many people. |
I will prepare a new release very soon. Thanks for the feedback. |
When using the new version of GCC, it is unable to compile libffi. Ref: libffi/libffi#764 Thanks @serge-sans-paille for your patch :) Signed-off-by: Kevin Cui <bh@bugs.cc>
When using the new version of GCC, it is unable to compile libffi. Ref: libffi/libffi#764 Thanks @serge-sans-paille for your patch :) Signed-off-by: Kevin Cui <bh@bugs.cc>
* external/libffi/implicit-function-declaration.patch was obsolete after libffi/libffi#764 * Backport external/python3/0001-bpo-45898-Remove-duplicate-symbols-from-_ctypes-cfie.patch.1 from upstream to fix build errors on Windows. Change-Id: I6d7cd70dd29ea0ce4b22738637afa0e7eb76a93f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167337 Tested-by: Jenkins Reviewed-by: Taichi Haradaguchi <20001722@ymail.ne.jp>
It's defined in closures.c and used in tramp.c.
Also declare it as an hidden symbol, as it should be.