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

Fix make error related to pcre #545

Merged
merged 1 commit into from
Jul 24, 2022
Merged

Conversation

tobil4sk
Copy link
Member

@tobil4sk tobil4sk commented May 8, 2022

The wrong variable was being used when building the regex part of the
standard library which was preventing compilation without a global
installation.

Only required if #515 is not merged first.

The wrong variable was being used when building the regex part of the
standard library which was preventing compilation without a global
installation.
@ncannasse
Copy link
Member

Please contribute to the PR instead so everything can be merged at once.

@ncannasse ncannasse closed this May 9, 2022
@tobil4sk
Copy link
Member Author

tobil4sk commented May 9, 2022

The other PR fixes this as well, but I thought it might be good to have this as a separate PR so that the current build works even if it takes a while for the other one to be merged.

@tobil4sk
Copy link
Member Author

@ncannasse A user came across this issue recently on the Haxe discord. This PR simply fixes a typo introduced in #520, and it would be useful to have by itself unless the PCRE2 PR is likely to be merged any time soon. Please could you merge this?

@ncannasse ncannasse reopened this Jul 24, 2022
@ncannasse ncannasse merged commit f261e9a into HaxeFoundation:master Jul 24, 2022
@tobil4sk
Copy link
Member Author

Thanks!

@tobil4sk tobil4sk deleted the fix-make branch July 24, 2022 14:18
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

Successfully merging this pull request may close these issues.

2 participants