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

Add MacOS specific linker flags to meson tests build step for MacOS #100

Merged
merged 3 commits into from
Jan 22, 2024

Conversation

artemgavrilov
Copy link
Collaborator

@artemgavrilov artemgavrilov commented Jan 17, 2024

Fixes #25

Unlike linux on MacOS shared library and loadable module(bundle) are different things. So linker was miss -bundle flag for testing executables build.

https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/LoadingCode/Concepts/AboutLoadableBundles.html

Copy link

github-actions bot commented Jan 17, 2024

Performance test results:
Normal queries: 9122
TDE queries: 8151
Percentage: 89%

@artemgavrilov artemgavrilov marked this pull request as ready for review January 17, 2024 17:44
Copy link
Member

@dAdAbird dAdAbird left a comment

Choose a reason for hiding this comment

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

LGTM

@dutow
Copy link
Collaborator

dutow commented Jan 22, 2024

@artemgavrilov could you also try to add a macos based github action? It can be a separate PR, but it would help with keeping the build working in the future.

@artemgavrilov
Copy link
Collaborator Author

@artemgavrilov could you also try to add a macos based github action? It can be a separate PR, but it would help with keeping the build working in the future.

Sure, will do it in separate PR

Copy link
Collaborator

@codeforall codeforall left a comment

Choose a reason for hiding this comment

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

Looks good

@codeforall codeforall merged commit 8508771 into main Jan 22, 2024
5 checks passed
@artemgavrilov artemgavrilov deleted the fix-meson-build-macos branch January 22, 2024 16:25
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.

Broken meson builds on Mac
4 participants