WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
184973
[OpenVR] CMake options exposed to users
https://bugs.webkit.org/show_bug.cgi?id=184973
Summary
[OpenVR] CMake options exposed to users
Michael Catanzaro
Reported
2018-04-25 09:17:40 PDT
Splitting this out from
bug #184972
: The OpenVR build options BUILD_SHARED, USE_LIBCXX, and USE_CUSTOM_LIBCXX are all currently exposed to users building WebKit. I don't think any openvr options should be exposed, because they look like normal WebKit options and there's not really any way to know that they're not. So I would mark these all as advanced. But then there's not really any point of keeping the options at all, right? That seems to indicate that the options should be removed. Sergio, what do you think we should do here? At the least, I would mark them all as advanced.
Attachments
Add attachment
proposed patch, testcase, etc.
Konstantin Tokarev
Comment 1
2018-04-25 09:44:00 PDT
In case you are wondering how to avoid exporting these options, I know 2 ways: * Remove option() calls in 3rd-party cmake files * Build 3rd party cmake projects via ExternalProject instead of including their project files into our build system (which may result in other side effects as well, e.g. changing global variables)
Michael Catanzaro
Comment 2
2018-04-25 12:53:49 PDT
(In reply to Konstantin Tokarev from
comment #1
)
> * Build 3rd party cmake projects via ExternalProject instead of including > their project files into our build system (which may result in other side > effects as well, e.g. changing global variables)
That sounds intriguing. At least it would reduce the need to patch OpenVR so much.
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