Skip to content

build: initialize meson build system #343

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

mochaaP
Copy link

@mochaaP mochaaP commented Apr 30, 2025

I'm adding this to meson's package registry: https://github.com/mesonbuild/wrapdb.

@hanickadot
Copy link
Owner

can this use only single-header one? I'm worried I will forget to add files to the meson file and will break someone, as I'm not using meson

@bgilbert
Copy link

Probably a good idea to add a Meson build to CI here, since otherwise any future problems will only show up in WrapDB's CI, after a release is tagged.

@mochaaP
Copy link
Author

mochaaP commented May 1, 2025

@hanickadot

can this use only single-header one? I'm worried I will forget to add files to the meson file and will break someone, as I'm not using meson

Since this is an interface-only library, the files list doesn't actually matter - it's just a hint for IDE file lists and compilers that don't support dependency tracking.

@bgilbert
Copy link

bgilbert commented May 1, 2025

Since this is an interface-only library, the files list doesn't actually matter - it's just a hint for IDE file lists and compilers that don't support dependency tracking.

It does matter because those headers are being installed, along with a pkg-config file that other software might use to build with those installed headers.

@mochaaP
Copy link
Author

mochaaP commented May 1, 2025

I will add a CI task that installs the library with meson and manually compiles the demos.

@mochaaP mochaaP force-pushed the mochaa branch 3 times, most recently from 67feb42 to 36014f3 Compare May 1, 2025 02:12
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