search

ARMmbed / yotta

DEPRECATED: yotta build; better software
Apache License 2.0
164 stars 64 forks source link

Add module name to preprocessor defines #833

Closed thegecko closed 6 years ago

thegecko commented 6 years ago

@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 commented 6 years ago

It's already defined as YOTTA_MODULE_NAME: http://docs.yottabuild.org/reference/buildsystem.html / https://github.com/ARMmbed/yotta/blob/7cec9c6cb339ae44998ca9d7f3ceac2cb64c3e4f/yotta/lib/templates/base_CMakeLists.txt#L119-L121

thegecko commented 6 years ago

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