Skip to content
This repository has been archived by the owner on Jan 15, 2021. It is now read-only.

Add module name to preprocessor defines #833

Closed
wants to merge 3 commits into from
Closed

Conversation

thegecko
Copy link
Member

@bogdanm, this addition was suggested by @jacobrosenthal in #774 to fix #773.

Do you see any problem (specifically with symbol name clashes) with this change?

@autopulated
Copy link
Contributor

It's already defined as YOTTA_MODULE_NAME: http://docs.yottabuild.org/reference/buildsystem.html /

# make YOTTA_MODULE_NAME available as a preprocessor symbol when
# compiling this module:
target_compile_definitions({{ component.getName() }} PRIVATE "-DYOTTA_MODULE_NAME={{ component.getName() }}")

@thegecko
Copy link
Member Author

Thanks @autopulated! I'll close this, then. @jacobrosenthal please reopen if there is any issue.

@thegecko thegecko closed this Feb 18, 2018
@thegecko thegecko deleted the f/dependency-name branch February 18, 2018 10:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Compiler define of module.json's name field
2 participants