Skip to content

Set libtool tag per command instead of global one #12350

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

Closed
wants to merge 1 commit into from

Conversation

jpalus
Copy link
Contributor

@jpalus jpalus commented Oct 2, 2023

Global --tag=CC defined in configure.ac is not correct in all cases. For example linking objects that were compiled from C++ sources needs to be done with C++ compiler, however for link mode libtool will prefer compiler indicated with --tag.

Fixes GH-12349

Global --tag=CC defined in configure.ac is not correct in all cases. For example
linking objects that were compiled from C++ sources needs to be done with C++
compiler, however for link mode libtool will prefer compiler indicated with
--tag.

Fixes phpGH-12349
@iluuu1994 iluuu1994 requested a review from petk October 3, 2023 09:20
pld-gitsync pushed a commit to pld-linux/php that referenced this pull request Oct 26, 2023
@petk
Copy link
Member

petk commented Jan 6, 2024

I'm onto this in the following days, thanks for your patience... Mold looks very promising and PHP definitely should support it also.

Copy link
Member

@petk petk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for PHP-8.2 branch. Those PHP_COMPILE and CXX_PHP_COMPILE variables aren't used in the code anymore but it's fine if they are changed also. Thanks @jpalus

@petk
Copy link
Member

petk commented Jan 10, 2024

Applied to PHP-8.2 branch and up via d57a776

@petk petk closed this Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants