Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FindGStreamer_1_0: fix build failure for new gstreamer versions #2910

Merged
merged 1 commit into from Oct 16, 2015

Conversation

dupondje
Copy link
Contributor

@dupondje dupondje commented Oct 4, 2015

On newer GStreamer versions FreeRDP fails to build with the following error.
/usr/include/gstreamer-1.0/gst/gstelement.h:55:27: fatal error: gst/gstconfig.h: Bestand of map bestaat niet

The patch was taken from:
WebKit/WebKit@c5a29d1#diff-277559c7222bb7abe4c5ef38647cce71

I tested this on Ubuntu 15.10 and solves my build error.

@freerdp-bot
Copy link

Refer to this link for build results (access rights to CI server needed):
https://ci.freerdp.com//job/PullRequestTester/741/

@bmiklautz
Copy link
Member

@freerdp-bot add @dupondje to whitelist

PATH_SUFFIXES gstreamer-1.0
)
string(REGEX MATCH "(.*)>=(.*)" _dummy "${_pkgconfig_name}")
if ("${CMAKE_MATCH_2}" STREQUAL "")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As we never pass a extra version comparison here this check regex isn't required for us. But it also doesn't hurt.

@bmiklautz
Copy link
Member

If all gstreamer pkg-config files exists this works like a charm.
+1

@yan12125
Copy link

Bump. Tested on Arch Linux with gstreamer 1.6.0 and commit d277346. Any progress on this PR?

@bmiklautz
Copy link
Member

@yan12125 generally no objection but I'm still waiting on the answer on the second comment.

@yan12125
Copy link

Thanks for the information. Hope it merged soon :)

@bmiklautz
Copy link
Member

We should sort out the GStreamer_FIND_VERSION at some point but as the version is checked specially afterwards it's ok for now.
@dupondje thanks for your efforts.

bmiklautz added a commit that referenced this pull request Oct 16, 2015
FindGStreamer_1_0: fix build failure for new gstreamer versions
@bmiklautz bmiklautz merged commit 7aca1e0 into FreeRDP:master Oct 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants