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

Allow system mpc #281

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Apteryks
Copy link
Contributor

Hi,

This enables building OpenBoardView using a system-provided MPC library.

Thanks!

* src/CMakeLists.txt: Only fetch the src/mpc git submodule and define
the mpc library when no system MPC is found (via pkg-config).
* src/CMakeLists.txt [!MPC_FOUND]: Add the 'mpc' sub-directory to
include directory.
* src/openboardview/FileFormats/GenCADFile.h: Change "mpc/mpc.h" to "mpc.h".
@piernov
Copy link
Member

piernov commented Apr 1, 2023

Due to the name conflict with libmpc, which is vastly more common and usually installed in most build environments, I'm not sure it's a good idea to rely on this.
If the renaming of mpc (Micro Parser Combinators) to pcq actually became the default we could use that instead, but it seems like it's abandoned.

@Apteryks
Copy link
Contributor Author

Apteryks commented Apr 1, 2023

The name conflict with GNU mpc is annoying, but given that GNU mpc doesn't ship any pkg-config file, one can be sure that 'mpc' found via pkg-config would be the orangeduck-mpc (that's how I named the package in GNU Guix to avoid the name clash).

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.

None yet

2 participants