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 710984 - misc patch to improve MSVC compatibility and CMake build script
misc patch to improve MSVC compatibility and CMake build script
Status: RESOLVED OBSOLETE
Product: GStreamer
Classification: Platform
Component: gst-plugins-gl
1.2.0
Other Windows
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-10-28 02:11 UTC by comicfans44
Modified: 2014-05-30 05:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-improve-CMake-build-system (18.69 KB, patch)
2013-10-28 02:12 UTC, comicfans44
needs-work Details | Review
0002-improve-MSVC-vc10-compatibility (7.48 KB, patch)
2013-10-28 02:12 UTC, comicfans44
needs-work Details | Review
0003-improve-cmake-build-script-add-install-target (6.06 KB, patch)
2013-10-28 02:13 UTC, comicfans44
needs-work Details | Review

Description comicfans44 2013-10-28 02:11:13 UTC
I'm trying to implement some element based on opengl plugin on win32, but I found the CMake build script incomplete ,VS project out of date and some headers not compatible with msvc , so I made some patch to complete the CMake build script, hopes this can help others working with msvc. this script can also build in Linux ,but not behave exactly the same as autoconf. it lacks automatic config header generation , use pre-defined header instead. tested on win7 32 bit.
the FindGstreamer.cmake file included in patch can also be used to found Gstreamer gl plugin with cmake.

these patch not the same as I posted in mail list. since I found some small bug in it, so I did some fix and re-format it.
Comment 1 comicfans44 2013-10-28 02:12:20 UTC
Created attachment 258263 [details] [review]
0001-improve-CMake-build-system
Comment 2 comicfans44 2013-10-28 02:12:47 UTC
Created attachment 258264 [details] [review]
0002-improve-MSVC-vc10-compatibility
Comment 3 comicfans44 2013-10-28 02:13:19 UTC
Created attachment 258265 [details] [review]
0003-improve-cmake-build-script-add-install-target
Comment 4 Sebastian Dröge (slomo) 2013-10-31 22:04:59 UTC
Review of attachment 258264 [details] [review]:

::: gst-libs/gst/gl/gstglapi.h
@@ +47,3 @@
 #  include <GL/gl.h>
 #  include <GL/glu.h>
+# if defined(__WIN32__) || defined(_WIN32)

Just use #if defined(G_OS_WIN32) here

::: gst/gl/gstglmosaic.c
@@ +263,2 @@
+	/* *INDENT-OFF* */
+	GLfloat temp[]=

Just move GLFloat *v_vertices here and initialize it with {} directly instead of introducing a new variable
Comment 5 Sebastian Dröge (slomo) 2013-10-31 22:05:31 UTC
Comment on attachment 258263 [details] [review]
0001-improve-CMake-build-system

Please provide your real name for the patch
Comment 6 Sebastian Dröge (slomo) 2013-10-31 22:05:34 UTC
Comment on attachment 258265 [details] [review]
0003-improve-cmake-build-script-add-install-target

Please provide your real name for the patch
Comment 7 comicfans44 2013-11-28 08:36:27 UTC
(In reply to comment #6)
> (From update of attachment 258265 [details] [review])
> Please provide your real name for the patch


Sorry for my slow reponse, I'm busy working on my office work...I'm 王昕宇(Wang Xinyu). Thank you for reviewing.
Comment 8 comicfans44 2013-11-29 00:53:16 UTC
plus: I found the compiled dll does not export necessary functions (needs some additional steps),so it can not be linked to , but the opengl elements in dll can work with no problem. hopes further improve
Comment 9 Matthew Waters (ystreet00) 2014-05-30 05:15:32 UTC
Hi, the opengl library/plugins are now included in the larger set of gst-plugins-bad, whose build system does not support cmake files.  The recommended way of building on windows os's is through the cerbero build system (http://cgit.freedesktop.org/gstreamer/cerbero).
Let us know of any other bugs you find :).