WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
91986
[CMake] Define constants in cmakeconfig.h instead of passing them directly via add_definitions()
https://bugs.webkit.org/show_bug.cgi?id=91986
Summary
[CMake] Define constants in cmakeconfig.h instead of passing them directly vi...
Raphael Kubo da Costa (:rakuco)
Reported
2012-07-23 06:06:04 PDT
The amount of constants for paths and other values we are passing via ADD_DEFINITIONS(-DFOO=bar) in our build system files is increasing. This is not the correct approach -- a cleaner way of passing values from the build system to source files is by using #cmakedefine, #cmakedefine01 or #define and have CMake replace the desired values via CONFIGURE_FILE(). We need to extend cmakeconfig.h.cmake and have it perhaps #include a port-specific config.h file which should be responsible for having these definitions.
Attachments
Add attachment
proposed patch, testcase, etc.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug