Skip to content

build: initialize meson build system#343

Open
mochaaP wants to merge 5 commits intohanickadot:mainfrom
mcha-forks:mochaa
Open

build: initialize meson build system#343
mochaaP wants to merge 5 commits intohanickadot:mainfrom
mcha-forks:mochaa

Conversation

@mochaaP
Copy link
Copy Markdown

@mochaaP mochaaP commented Apr 30, 2025

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

@hanickadot
Copy link
Copy Markdown
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
Copy Markdown

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.

Comment thread meson.build Outdated
Comment thread meson.build
@mochaaP
Copy link
Copy Markdown
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
Copy Markdown

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.

Comment thread include/meson.build
@mochaaP
Copy link
Copy Markdown
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