The usage was previously guarded by HAVE_DLFCN. This breaks on Android with
LLVM_BUILD_STATIC as the platform does not provide a static version of libdl.
Using HAVE_DLOPEN fixes it as the code will only get used if we are actually able
to link an executable using dlopen.
Details
Details
Diff Detail
Diff Detail
- Build Status
Buildable 1676 Build 1676: arc lint + arc unit
Event Timeline
Comment Actions
Do you have any objections to this going in? (If I chose the wrong reviewers, let me know).
Comment Actions
Please use #if defined(HAVE_DLFCN_H) && defined(HAVE_DLOPEN) in both places, otherwise LGTM.
Comment Actions
Thank you for checking this out.
HAVE_DLOPEN is defined only if HAVE_DLFCN_H is detected, but I suppose it doesn't hurt to be explicit.