Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 509698

Summary: =media-libs/gstreamer-0.10.36 with =sys-devel/clang-3.4-r100 - In file included from gstdatetime.c:24: /usr/include/glib-2.0/glib/gmessages.h:200:11: error: use of undeclared identifier 'g_log_domain_gstreamer'
Product: Gentoo Linux Reporter: Jauhien Piatlicki (RETIRED) <jauhien>
Component: Current packagesAssignee: GStreamer package maintainers <gstreamer>
Status: RESOLVED OBSOLETE    
Severity: normal CC: fsvm88, wizardedit
Priority: Normal Keywords: Inclusion, PATCH
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Bug Depends on:    
Bug Blocks: 408963    
Attachments: build.log
Include epatch_user in the ebuild.
Patch to swap header order for clang compilation

Description Jauhien Piatlicki (RETIRED) gentoo-dev 2014-05-06 16:37:59 UTC
Created attachment 376488 [details]
build.log

Build log attached.
Comment 1 Fabio Scaccabarozzi 2014-08-07 11:00:01 UTC
Created attachment 382444 [details, diff]
Include epatch_user in the ebuild.
Comment 2 Fabio Scaccabarozzi 2014-08-07 11:00:39 UTC
Created attachment 382446 [details, diff]
Patch to swap header order for clang compilation
Comment 3 Fabio Scaccabarozzi 2014-08-07 11:01:32 UTC
I have bumped the ebuild to use epatch_user (diff attached).
Dropping the second patch in /etc/portage/patches/ fixes the compilation issue. The patch simply reverses the order in which two headers appear.
Both gcc-4.8.3 and clang-3.4.2 compile and pass tests just fine.
Comment 4 Fabio Scaccabarozzi 2015-01-10 20:40:05 UTC
The patch still applies for clang 3.5.0, tests pass fine both with clang (3.5.0) and GCC (4.8.4).
Please note that gstreamer 0.10.x is no longer mantained, see official site, news section, on 2013-03-15.
Comment 5 Pacho Ramos gentoo-dev 2016-06-06 19:13:29 UTC
@Austin, if you have tested this works for you, feel free to commit :)

Thanks
Comment 6 Austin English (RETIRED) gentoo-dev 2016-06-06 23:57:27 UTC
(In reply to Pacho Ramos from comment #5)
> @Austin, if you have tested this works for you, feel free to commit :)
> 
> Thanks

I hadn't yet, actually. I'm using 3.5.0, and gstreamer-0.10.36-r2 compiles out of the box for me with clang. Comment #4 looks like 3.5.0 previously had issues though, which is interesting.
Comment 7 Pacho Ramos gentoo-dev 2016-06-07 18:35:57 UTC
In that case, this looks like obsolete right?
Comment 8 Austin English (RETIRED) gentoo-dev 2016-06-07 18:38:53 UTC
(In reply to Pacho Ramos from comment #7)
> In that case, this looks like obsolete right?

Yeah, I think that's fine. If I can reproduce in some other version, I'll open a new issue.