After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 760008 - orc/orcdebug.c:39:25: fatal error: android/log.h: No such file or directory
orc/orcdebug.c:39:25: fatal error: android/log.h: No such file or directory
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: orc
unspecified
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-12-30 18:28 UTC by Igor Filakhtov
Modified: 2015-12-30 22:57 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to make android liblog optional (751 bytes, patch)
2015-12-30 18:48 UTC, Igor Filakhtov
rejected Details | Review
configure: Only check for Android's liblog on Android targets (1.01 KB, patch)
2015-12-30 18:54 UTC, Sebastian Dröge (slomo)
committed Details | Review

Description Igor Filakhtov 2015-12-30 18:28:53 UTC
orc fails to build because of automatic dependency detection. It detects Android llog on my system somehow, but then fails to compile of course, since I'm running Gentoo and not an android...

Would be nice to have a configuration option --disable-android or --enable-android for configure.

Reproducible: Always

Steps to Reproduce:
1. just trying to compile orc with llog library provided on your system

Actual Results:  
Build fails with missing header error

Expected Results:  
Build succeeded
Comment 1 Sebastian Dröge (slomo) 2015-12-30 18:42:57 UTC
Where does this liblog come from on your system, what is it? It must also contain the symbol __android_log_print, otherwise configure wouldn't detect it.

It would probably make sense to limit the usage of that to systems that have an Android specific host triplet.
Comment 2 Igor Filakhtov 2015-12-30 18:48:41 UTC
Created attachment 318049 [details] [review]
Patch to make android liblog optional

Added --enable-android-log option to enable detection of android liblog.
Comment 3 Sebastian Dröge (slomo) 2015-12-30 18:54:29 UTC
Created attachment 318050 [details] [review]
configure: Only check for Android's liblog on Android targets
Comment 4 Sebastian Dröge (slomo) 2015-12-30 18:55:47 UTC
Comment on attachment 318049 [details] [review]
Patch to make android liblog optional

This should come with some auto-detection if anything. I think it's better to just only ever use it for Android targets for now.
Comment 5 Sebastian Dröge (slomo) 2015-12-30 18:57:39 UTC
Attachment 318050 [details] pushed as e0d39f6 - configure: Only check for Android's liblog on Android targets
Comment 6 Igor Filakhtov 2015-12-30 18:58:37 UTC
Sebastian, target-based detection is much better, I'm agree with you!

In my case these symbols come from android ndk (I think) or some related tools for Android ROM manipulation. Can not tell more right now, but can dig for it later if it is important.
Comment 7 Sebastian Dröge (slomo) 2015-12-30 19:10:39 UTC
Well, it's mostly important for yourself :) That seems like a messed up system if you have random NDK libraries in your library path. It will probably cause problems elsewhere too.