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

Add initial support for build.zig #2414

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Add initial support for build.zig #2414

wants to merge 1 commit into from

Conversation

sagehane
Copy link

Proposal

I was wondering if this project would be willing to look into the Zig build system.

The idea is that, theoretically, a person can install Zig and the relevant dependencies and build this project for any of the target supported by Zig.

It also enables stuff such as UBSAN by default, which seems to have spotted some undefined behaviors for this project already.

TODO:

  • Get the build to work on all relevant platforms. (Currently only supports x86_64-linux)
  • Resolve what to do with irrlicht headers.
    I renamed irrlicht.h -> irrlicht/irrlicht.h for example. Curious why it currently is that way.
  • Get the build working on Debian-based distros that seem to expect lua-c++ instead of lua.
  • Figure out how to handle the files described in https://github.com/Fluorohydride/ygopro/wiki#running.

@mercury233
Copy link
Collaborator

Zig seems cool but I'm afraid I can't provide too much help.

For Irrlicht, the release from Irrlicht official organize files like irrlicht/include/*.h and irrlicht/source/Irrlicht/*.cpp so we added irrlicht/include to include in premake.

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.

2 participants