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

Workaround for config_types.h.in to use CMake and configure side-by-side #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

botanegg
Copy link

@botanegg botanegg commented Oct 9, 2021

Init INCLUDE_INTTYPES_H, INCLUDE_STDINT_H, INCLUDE_SYS_TYPES_H, CMake variables as 0

It is need to prevent generate broken code in config_types.h
So next code was genarated before (e.g. for Android SDK)

#define INCLUDE_INTTYPES_H
...
#if INCLUDE_INTTYPES_H
...
#endif

And got error: expected value in expression (in third line)

After this changes code will be

#define INCLUDE_INTTYPES_H 0
...
#if INCLUDE_INTTYPES_H
...
#endif

This changes seems like a smelt code, but it need to compatibility with configure script
To avoid this workaround we can use cmake-specific directive #cmakedefine01 in config_types.h.in configure_file man

@botanegg botanegg changed the title Initialized CMake variables for configure as 0 WIP (seems like erroneously): Initialized CMake variables for configure as 0 Oct 10, 2021
@botanegg botanegg changed the title WIP (seems like erroneously): Initialized CMake variables for configure as 0 Workaround for config_types.h.in to use CMake and configure side-by-side Oct 10, 2021
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.

1 participant