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

CMake: Compile with system libraries for xxhash, Catch2, and fmt, if available #4042

Merged
merged 3 commits into from
Sep 6, 2024

Conversation

neobrain
Copy link
Member

@neobrain neobrain commented Sep 6, 2024

Instead of compiling these from bundled sources, check if the corresponding system libraries are installed first. This should help FEX packaging for Linux distributions (see #2996).

@neobrain neobrain force-pushed the refactor_system_libs branch 2 times, most recently from ccfa61c to 43c45cf Compare September 6, 2024 08:23
@neobrain neobrain force-pushed the refactor_system_libs branch from 43c45cf to ab5d3ab Compare September 6, 2024 08:31
Copy link
Collaborator

@alyssarosenzweig alyssarosenzweig left a comment

Choose a reason for hiding this comment

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

as a follow on it would be nice to make these not submodules anymore so our forks don't get pulled unless you actually need them. meson has "subproject wraps" for this, I'm not sure the cmake way.

@neobrain
Copy link
Member Author

neobrain commented Sep 6, 2024

as a follow on it would be nice to make these not submodules anymore so our forks don't get pulled unless you actually need them. meson has "subproject wraps" for this, I'm not sure the cmake way.

CMake has FetchContent for this. I played around with it today and it works in principle, but I'm not sure if the change would come with unexpected side effects (by sole virtue of being a change, as opposed to some specific weakness I've observed in FetchContent).

Copy link
Member

@Sonicadvance1 Sonicadvance1 left a comment

Choose a reason for hiding this comment

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

Doesn't seem to break anything directly that I could see.

@Sonicadvance1 Sonicadvance1 merged commit a4acd64 into FEX-Emu:main Sep 6, 2024
12 checks passed
@neobrain neobrain deleted the refactor_system_libs branch September 7, 2024 12:32
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.

3 participants