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

Build-mode #60

Closed
wants to merge 2 commits into from
Closed

Build-mode #60

wants to merge 2 commits into from

Conversation

dcharkes
Copy link
Collaborator

@dcharkes dcharkes commented Jun 7, 2023

Introduces BuildMode with release and debug values.

This is not an enum, but a collection with two opaque objects. (Similar to Target.) This way adding more values later will not be a breaking change.

Package c_compiler picks up the configuration and sets either the DEBUG or RELEASE macro to 1.

We should probably do the mapping from BuildMode to defines in build.dart files, because it might be project specific. The defines should be added to the CBuilder constructors or run function.

Bug:

@dcharkes
Copy link
Collaborator Author

dcharkes commented Jun 7, 2023

(new commits are not showing up somehow)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant