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

swig: Upgrade formula to v4.1.1 #860

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Conversation

sevan
Copy link
Collaborator

@sevan sevan commented May 31, 2023

Switch to using pcre2 as pcre is EoL

Switch to using pcre2 as pcre is EoL
Copy link
Owner

@mistydemeo mistydemeo left a comment

Choose a reason for hiding this comment

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

It looks like brew test swig is failing for me; do you know if it's an issue with the built formula, or with the rest being out of date?

@sevan
Copy link
Collaborator Author

sevan commented Aug 21, 2023

system ENV.cc, "-c", "test_wrap.c", "-I/System/Library/Frameworks/Ruby.framework/Headers/"

Issue is with the swig tests case. It tries to make use of ruby.h from Ruby.Framework but that didn't exist until Leopard. We could just the version included in Tigerbrew?

@mistydemeo
Copy link
Owner

Tigerbrew's bundled Ruby won't really work since it's statically linked, but I can look into finding something else for it to work with. Doesn't need to be a blocker on merging this though - sorry for the delay!

@mistydemeo mistydemeo merged commit f677f7a into mistydemeo:master Nov 27, 2023
@sevan sevan deleted the swig411 branch November 27, 2023 02:39
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