-
Notifications
You must be signed in to change notification settings - Fork 45
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
WIP: Add Meson support #254
base: master
Are you sure you want to change the base?
Conversation
I cannot get the unit tests to work on Meson for some reason, would anyone be able to help me out with that? Or would omitting the unit test be fine for Meson builds? |
IMO it would be fine to omit as long as it doesn't prevent testing Meson. In other words, can we still get a CI in place that builds some working executable ? What error are you seeing ? |
@HenkKalkwater Hi! I am interested in Meson too Why we use |
Very likely because I want to generate a pkgconfig file for this library (it is used further below) and it seems to work. If I'm not mistaken, this is needed so that other projects can find this library to set the correct linker flags and so on. |
Ok, I just not sure at 100% in that tradition It looks like other Meson projects can use any other Meson project directly (as subproject) without additional 3-rd party formats like pkgconfig, cmake, etc |
Yes, Meson is able to do so. Generating pkg-config files enables Meson and other build systems to find it when it is installed on the system on Unix-like systems and perhaps Windows, but I'm not sure about that. |
Ok, I added wrap files into my project and tried to setup Meson and catched: subprojects/taggedalgebraic/meson.build:19:0: ERROR: Recursive include of subprojects: vibe_d => vibe-core => eventcore => taggedalgebraic => eventcore. I think we must carefully rewrite from scratch every meson.build of vibe.d dependencies with using of all Meson modern changes and without unnecessary directives before we can proceed. Also I started to make For now it isn't able to convert whole vibe.d, but vibe.d is very important to build with Meson because from vibe.d depend about 1/3 of all packages. This is why I want to propose to (re)write meson.build files for Vibe.d and all of its dependencies. |
I just messed-up URLs |
When vibe-core was split off vibe.d itself, Meson support was broken. This adds it back in, based of the original Meson code when it was in the vibe.d project.
Should be a step towards resolving vibe-d/vibe.d#2531
Depends on: vibe-d/eventcore#171
Right now, some of the unit tests seem to be broken. I have no idea what causes it, which is why this is marked as WIP: