Added meson build files and manifests for dlls. #167

Open
hanetzer wants to merge 1 commits from hanetzer/mesonbuild into master
hanetzer commented 2017-07-18 11:29:25 +00:00 (Migrated from github.com)

Works, builds both native linux and cross-compiles mingw-w64 variants.
Need to work out copying appropriate dlls into the mkxp runtime folder.

Follow up to #35

Signed-off-by: Marty Plummer ntzrmtthihu777@gmail.com

Works, builds both native linux and cross-compiles mingw-w64 variants. Need to work out copying appropriate dlls into the mkxp runtime folder. Follow up to #35 Signed-off-by: Marty Plummer <ntzrmtthihu777@gmail.com>
khkramer commented 2017-07-18 13:00:44 +00:00 (Migrated from github.com)

Could this potentially make a static build as well? I haven't been able to make a build with cmake sofar that doesn't force me to have like 20 dll's in my game folder.

Could this potentially make a static build as well? I haven't been able to make a build with cmake sofar that doesn't force me to have like 20 dll's in my game folder.
hanetzer commented 2017-07-18 13:08:56 +00:00 (Migrated from github.com)

@khkramer yes, but would require a static build of openal, which you can't easily do at the same time as a shared (dll) build. However, I've actually added the ability to use executable manifests, which allow you to basically subdir the dlls. In the current setup, you'd copy the dlls and runtime.manifest into a subdir next to the game executable.

Depending on how the libs are compiled you may need to adjust runtime.manifest (simple xml file, easy to understand).

@khkramer yes, but would require a static build of openal, which you can't easily do at the same time as a shared (dll) build. However, I've actually added the ability to use executable manifests, which allow you to basically subdir the dlls. In the current setup, you'd copy the dlls and runtime.manifest into a subdir next to the game executable. Depending on how the libs are compiled you may need to adjust runtime.manifest (simple xml file, easy to understand).
hanetzer commented 2017-07-28 05:10:25 +00:00 (Migrated from github.com)

Actually, if you are going to merge this, do not do so until #168 is merged (if it is going to be merged).
I'll rebase this pr if/once that happens.

Actually, if you are going to merge this, do not do so until #168 is merged (if it is going to be merged). I'll rebase this pr if/once that happens.
Ancurio (Migrated from github.com) reviewed 2017-08-10 20:16:26 +00:00
@ -0,0 +24,4 @@
dependencies : [ ruby, sdl2, sigcxx ],
include_directories : incdirs,
target_type : 'static_library')
Ancurio (Migrated from github.com) commented 2017-08-10 20:16:26 +00:00

Does meson require to have its build files in every source directory, or is that just your convention?

Does meson require to have its build files in every source directory, or is that just your convention?
hanetzer (Migrated from github.com) reviewed 2017-08-10 20:19:01 +00:00
@ -0,0 +24,4 @@
dependencies : [ ruby, sdl2, sigcxx ],
include_directories : incdirs,
target_type : 'static_library')
hanetzer (Migrated from github.com) commented 2017-08-10 20:19:01 +00:00

its 'a' convention. not necessarily mine but its been the best way I've found to go about it for now.

its 'a' convention. not necessarily mine but its been the best way I've found to go about it for now.
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b hanetzer/mesonbuild master
git pull origin hanetzer/mesonbuild

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff hanetzer/mesonbuild
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MapleShrine/mkxp#167
No description provided.