Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 509698 - =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'
Summary: =media-libs/gstreamer-0.10.36 with =sys-devel/clang-3.4-r100 - In file includ...
Status: RESOLVED OBSOLETE
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: GStreamer package maintainers
URL:
Whiteboard:
Keywords: Inclusion, PATCH
Depends on:
Blocks: systemwide-clang
  Show dependency tree
 
Reported: 2014-05-06 16:37 UTC by Jauhien Piatlicki (RETIRED)
Modified: 2016-06-07 18:38 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---

Attachments
build.log (build.log,40.34 KB, text/plain)
2014-05-06 16:37 UTC, Jauhien Piatlicki (RETIRED) 		Details
Include epatch_user in the ebuild. (gstreamer-0.10.36-r2-with-epatch_user.patch,410 bytes, patch)
2014-08-07 11:00 UTC, Fabio Scaccabarozzi 		Details | Diff
Patch to swap header order for clang compilation (resort-headers-for-clang.patch,452 bytes, patch)
2014-08-07 11:00 UTC, Fabio Scaccabarozzi 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.