Skip to content

Conversation

ruben-ayrapetyan
Copy link
Contributor

Adding support to specify include interfaces for more than one external library (for external builds).

CMakeLists.txt Outdated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if more than one external library provides stdlib.h? Maybe we should signal an error message in this case?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok. I'll add the check that would terminate build in the case: it seems that if more than one external library provides stdlib.h, we would have include path conflicts anyway.

@ruben-ayrapetyan ruben-ayrapetyan force-pushed the several_external_libraries branch from 2c535f4 to 1431251 Compare May 8, 2015 17:49
@ruben-ayrapetyan ruben-ayrapetyan changed the title Adding support to specify include interfaces for more than one external library Preparations for support of non-nuttx externally built targets May 8, 2015
…n libc.

JerryScript-DCO-1.0-Signed-off-by: Ruben Ayrapetyan [email protected]
…from Nuttx OS.

JerryScript-DCO-1.0-Signed-off-by: Ruben Ayrapetyan [email protected]
@ruben-ayrapetyan ruben-ayrapetyan force-pushed the several_external_libraries branch from 1431251 to 2852d93 Compare May 8, 2015 18:03
@sand1k
Copy link
Contributor

sand1k commented May 8, 2015

LGTM.

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