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 options to zig compile #3115

Merged
merged 5 commits into from
Jun 18, 2023
Merged

Add options to zig compile #3115

merged 5 commits into from
Jun 18, 2023

Conversation

dantecatalfamo
Copy link
Contributor

@dantecatalfamo dantecatalfamo commented Jun 17, 2023

Support for compiling with raygui, raymath, and physac.
Also outputs the required headers.

addRaylib gains an additional positional argument that takes an Options struct.

Raygui should be located ../raygui relative to the repo root
Physac should be located ../physac relative to the repo root

These locations match the options in the Makefile

One step generates src/raygui.c, src/raymath.c, or src/physac.c with the [lib]_IMPLEMENTATION macro defined depending on which are enabled

Support for compiling with raygui, raymath, and physac.
Also outputs the required headers.

Raygui should be located `../raygui` relative to the repo root
Physac should be located `../physac` relative to the repo root

This behavior matches options in the Makefile
@raysan5
Copy link
Owner

raysan5 commented Jun 17, 2023

@dantecatalfamo physac shouldn't be included, the library is mostly abbandoned and has issues.

EDIT: Also note that raymath is already compiled into raylib, adding it could generate issues.

@dantecatalfamo
Copy link
Contributor Author

dantecatalfamo commented Jun 17, 2023

Okay, I've removed physac.

EDIT: Also note that raymath is already compiled into raylib, adding it could generate issues.

Oh I didn't see it was pulled in by rcore, I've removed the generated file 🙂

I set it to always install rlgl.h and raymath.h because the Makefile also does that.

@raysan5 raysan5 merged commit 3a90acf into raysan5:master Jun 18, 2023
@raysan5
Copy link
Owner

raysan5 commented Jun 18, 2023

@dantecatalfamo Thanks for the improvement!

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