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

Review Makefile thoroughly, cleanup/docs #256

Open
ped7g opened this issue Jan 30, 2025 · 0 comments
Open

Review Makefile thoroughly, cleanup/docs #256

ped7g opened this issue Jan 30, 2025 · 0 comments
Assignees
Milestone

Comments

@ped7g
Copy link
Collaborator

ped7g commented Jan 30, 2025

For example there's ifndef KEEP_SYMBOLS while this should maybe also evaluate KEEP_SYMBOLS and check also value?

review all optional variables and their comments (f.e. bundled lua could describe both options)

re-check all the described build flavors (asan, ubsan. ...), maybe add more docs about how-to (you already forgot how to do them, so you will have to figure it out anyway), check other txt notes for valgrind/profiling and move it to makefile docs so it's all in one place.

c++17 vs gnu++17 - maybe detect mingw platform?

Check also how feasible it would be to use just one makefile (also for windows)

@ped7g ped7g added this to the v1.21.0 milestone Jan 30, 2025
@ped7g ped7g self-assigned this Jan 30, 2025
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

No branches or pull requests

1 participant