From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 13:07:58
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 Summary: [PATCH] Build issues on Windows/MSVC Product: GStreamer Version: 0.10.14 Platform: Other OS/Version: All Status: UNCONFIRMED Severity: normal Priority: Normal Component: gstreamer (core) AssignedTo: gst...@li... ReportedBy: ole...@ta... QAContact: gst...@li... GNOME version: Unspecified GNOME milestone: Unspecified Please describe the problem: GStreamer doesn't build properly with Windows/MSVC (MSVS2005) and GLib 2.14.x. There are also some symbols missing in the .def file that need to be exported. Steps to reproduce: 1. 2. 3. Actual results: Expected results: Does this happen every time? Other information: -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 14:45:44
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 Summary: [PATCH] Build issues on Windows/MSVC Product: GStreamer Version: 0.10.14 Platform: Other OS/Version: All Status: UNCONFIRMED Severity: normal Priority: Normal Component: gst-plugins-base AssignedTo: gst...@li... ReportedBy: ole...@ta... QAContact: gst...@li... GNOME version: Unspecified GNOME milestone: Unspecified Please describe the problem: gst-plugins-base doesn't build properly with Windows/MSVC (MSVS2005). There are also some symbols missing in the .def file that need to be exported. Steps to reproduce: 1. 2. 3. Actual results: Expected results: Does this happen every time? Other information: -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 14:47:22
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 ------- Comment #1 from Ole André Vadla Ravnås 2007-10-31 14:46 UTC ------- Created an attachment (id=98250) --> (http://bugzilla.gnome.org/attachment.cgi?id=98250&action=view) Some additional symbols need to be exported. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 14:49:11
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 ------- Comment #2 from Ole André Vadla Ravnås 2007-10-31 14:47 UTC ------- Created an attachment (id=98251) --> (http://bugzilla.gnome.org/attachment.cgi?id=98251&action=view) Added two casts needed to avoid MSVC warnings when compiling GstRingBuffer. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 14:52:28
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 ------- Comment #3 from Ole André Vadla Ravnås 2007-10-31 14:52 UTC ------- Created an attachment (id=98252) --> (http://bugzilla.gnome.org/attachment.cgi?id=98252&action=view) Fixed a critical GstRTPHeader packing issue with MSVC. A colleague (Haakon Sporsheim) might already have submitted this patch. Including it just in case. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 14:57:27
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 ------- Comment #4 from Ole André Vadla Ravnås 2007-10-31 14:57 UTC ------- Created an attachment (id=98253) --> (http://bugzilla.gnome.org/attachment.cgi?id=98253&action=view) Fixes a MSVC runtime error when using the debug CRT. MSVC requires you to mask when data loss is intentional. It compiles in run-time checks to catch this when building with the debug C runtime, causing run-time checks failing with videotestsrc. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 19:32:19
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 ------- Comment #5 from Ole André Vadla Ravnås 2007-10-31 19:32 UTC ------- Created an attachment (id=98266) --> (http://bugzilla.gnome.org/attachment.cgi?id=98266&action=view) Fixed some #undefs to shut up MSVC. MSVC doesn't like "#undef FOO 1", unlike gcc, so this results in warnings. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-11-01 12:09:58
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|gstreamer- |t....@ze... |bu...@li... | Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-11-01 12:55:06
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492114 GStreamer | gst-plugins-base | Ver: 0.10.14 Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #98250|none |committed Flag| | Attachment #98251|none |reviewed Flag| | Attachment #98252|0 |1 is obsolete| | Attachment #98253|none |committed Flag| | Attachment #98266|none |committed Flag| | AssignedTo|t....@ze... |gstreamer- | |bu...@li... Status|ASSIGNED |RESOLVED OS/Version|All |Windows Resolution| |FIXED Summary|[PATCH] Build issues on |Build issues on Windows/MSVC |Windows/MSVC | Target Milestone|HEAD |0.10.15 ------- Comment #6 from Tim-Philipp Müller 2007-11-01 12:54 UTC ------- Thanks for the patches. The ringbuffer warning fix I've done slightly differently (just return NULL). The rtp header packing has already been fixed, as you said. The config.h patch should really be against config.h.in :) 2007-11-01 Tim-Philipp Müller <tim at centricular dot net> Patch by: Ole André Vadla Ravnås <ole...@ta...> * gst-libs/gst/audio/gstringbuffer.c: (build_linear_format), (gst_ring_buffer_parse_caps): Return NULL instead of an enum that happens to be 0, fixes warning on MSVC (#492114). * gst-libs/gst/audio/gstringbuffer.h: No trailing commas in enum list (for gcc-2.9x). * gst/videotestsrc/videotestsrc.c: (random_char): Make information loss explicit instead of implicitly truncating to eight bits via the return value. Fixes runtime error on MSVC when using the debug CRT (#492114). * win32/common/config.h.in: Fix a bunch of '#undef FOO bar', which MSVC doesn't like (#492114). * win32/common/libgstinterfaces.def: * win32/common/libgstrtp.def: Export a few more symbols (#492114). -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492114. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 13:10:10
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 ------- Comment #1 from Ole André Vadla Ravnås 2007-10-31 13:09 UTC ------- Created an attachment (id=98238) --> (http://bugzilla.gnome.org/attachment.cgi?id=98238&action=view) Debug category should be exported, not imported, when building plugins. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 13:11:23
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 ------- Comment #2 from Ole André Vadla Ravnås 2007-10-31 13:10 UTC ------- Created an attachment (id=98239) --> (http://bugzilla.gnome.org/attachment.cgi?id=98239&action=view) Some additional symbols need to be exported. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 13:12:28
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 ------- Comment #3 from Ole André Vadla Ravnås 2007-10-31 13:11 UTC ------- Created an attachment (id=98241) --> (http://bugzilla.gnome.org/attachment.cgi?id=98241&action=view) dirent needs a couple of casts to build warning-free with MSVC. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 13:15:53
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 ------- Comment #4 from Ole André Vadla Ravnås 2007-10-31 13:13 UTC ------- Created an attachment (id=98242) --> (http://bugzilla.gnome.org/attachment.cgi?id=98242&action=view) Fixed a couple of missing includes and a C99 issue (MSVC doesn't like C99). -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 13:16:56
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 ------- Comment #5 from Ole André Vadla Ravnås 2007-10-31 13:15 UTC ------- Created an attachment (id=98243) --> (http://bugzilla.gnome.org/attachment.cgi?id=98243&action=view) GLib doesn't have a pipe() macro anymore starting with 2.14.0. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 20:50:40
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|gstreamer- |t....@ze... |bu...@li... | Status|UNCONFIRMED |ASSIGNED Ever Confirmed|0 |1 -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-10-31 22:01:31
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #98238|none |reviewed Flag| | Attachment #98239|none |committed Flag| | Attachment #98241|none |committed Flag| | Attachment #98242|none |committed Flag| | Attachment #98243|none |committed Flag| | Status|ASSIGNED |NEEDINFO Summary|[PATCH] Build issues on |Build issues on Windows/MSVC |Windows/MSVC | Target Milestone|HEAD |0.10.15 ------- Comment #6 from Tim-Philipp Müller 2007-10-31 22:01 UTC ------- Thanks for the patches. I had to do the lib*.def ones manually, since most of the missing symbols were already added a few weeks ago (and your patch changed the alphabetical order of things too for some reason). Hope I didn't miss anything, please double-check. I have changed the guard around the unistd.h include to #ifdef HAVE_UNISTD_H, that seems more correct to me. Could you expand on patch #98238 a bit for non-win32 like me? Who defines GST_PLUGIN_EXPORTS? I can't find a reference to that anywhere within GStreamer. Also, what does the change actually do? Is it really needed? (It seems to have worked fine so far without on MSVC) -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-11-01 01:29:32
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 Ole André Vadla Ravnås changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |NEW ------- Comment #7 from Ole André Vadla Ravnås 2007-11-01 01:29 UTC ------- Sorry about the hassle with the .def stuff, my bad for not getting around to pushing the patches earlier. The change of alphabetical order was clearly not intentional, I realize that this must've been done rather hastily. I'm planning on going over this properly with the same code built with autotools on Linux, run nm, sort the symbols and compare to an export dump on Windows. There's probably a few left out, the ones added by the patch were discovered because some plugin relied on it, and given that we only build a subset of the plugins there could be a few still left out. Maybe this is something that could be integrated with the continuous build system, after building it could dump the symbols and compare to the .def files. Great catch regarding the HAVE_UNISTD_H fix, that's definitely more correct. Patch #98238 is in retrospect a bit confusing and needs some explaining. When exporting symbols you need to explicitly say which should be exported, or you won't get any exported at all. There are two ways to do this. 1. You have an entry like the following in the header-file: int __declspec(dllexport) gst_foo_do_bar (void); (Obviously the _user_ of the library needs "dllimport" instead of "dllexport", so you want to use a macro for the whole __declspec(dllimport/dllexport).) 2. Tell the linker to use a .def file that you create with a list of symbols, like: gst_foo_do_bar A common convention is to have a define called "LIBRARY_NAME_EXPORTS" that you only define when building LIBRARY_NAME, and in the header-file you do: #ifndef __GNUC__ # define __DLL_IMPORT__ __declspec(dllimport) # define __DLL_EXPORT__ __declspec(dllexport) #else # define __DLL_IMPORT__ __attribute__((dllimport)) extern # define __DLL_EXPORT__ __attribute__((dllexport)) extern #endif #if defined(__WIN32__) || defined(_WIN32) || defined(WIN32) # ifdef LIBRARY_NAME_EXPORTS # define LIBRARY_NAME_API __DLL_EXPORT__ # else # define LIBRARY_NAME_API __DLL_IMPORT__ # endif #else # define LIBRARY_NAME_API extern #endif And use the LIBRARY_NAME_API like this: int LIBRARY_NAME_API gst_foo_do_bar (void); GStreamer has, as you can see from the patch, a mixture of these two (but the majority is of the latter kind). The upside to the .def approach is that you don't need to change all the header-files if you're porting something to Windows, whilst the flip side is obviously that you need to maintain the .def files. The patch changes the macro GST_DEBUG_CATEGORY_EXTERN so that it declares the symbol as exported when used by a plugin (built with GST_PLUGIN_EXPORTS defined) to export a debug category, and imported otherwise. Without this patch the linker generates warnings about conflicting linkage, but presumably it figures out what your intention is and does the right thing. When building GStreamer on Windows we have a bunch of MSVS2005 project files of which those for plugins define GST_PLUGIN_EXPORTS. I didn't add this to the project files in the win32 subdirectory because I wouldn't be able to test these changes. The reason that we're not using these project files is another (rather long) story, but basically it's because we have a different approach. MSVS has the notion of a solution containing projects, where each project is typically a program or a library. The solution also knows about dependencies between the projects. In our team we have one big solution that contains projects for everything from GLib to GStreamer (as this is the most comfortable to work with for a Windows developer, given that dependencies etc. are taken care of), and by using something called "property sheets" we can share common things among the projects. All projects inherit the property sheet that we've called "Common" which sets output directories, common compiler flags etc., and the projects that are GStreamer plugins also inherit "GStreamerPlugin", which overrides the output directory to be $(OutDir)/lib/gstreamer-0.10 instead of the default $(OutDir)/bin (set by the Common property sheet), adds this GST_PLUGIN_EXPORTS define, etc. We also have a python tool that parses a given Makefile.am and installs header-files in $(OutDir)/include, so that we don't have to duplicate the list of header-files in our build system. So bottom line is that we end up with just the list of files in the project files, they end up being really slim thanks to property sheets, and with one toplevel solution instead of one solution per component, so that means we have very little in common with the projects in the win32 subdir. I'm hoping that this is something that can be made suitable for upstream eventually, but that's a different issue (something that I'm planning on trying to address in http://projects.collabora.co.uk/~oleavr/OABuild/). Sorry about the lengthy explanation. Coming from a Linux-background I have to say that the more I learn about Windows the more it beats me that it is not a very developer-friendly platform. :) -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |
From: GStreamer (bugzilla.gnome.org) <bug...@bu...> - 2007-11-01 23:52:48
|
If you have any questions why you received this email, please see the text at the end of this email. Replies to this email are NOT read, please see the text at the end of this email. You can add comments to this bug at: http://bugzilla.gnome.org/show_bug.cgi?id=492077 GStreamer | gstreamer (core) | Ver: 0.10.14 Tim-Philipp Müller changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #98238|0 |1 is obsolete| | Attachment #98238|reviewed |obsolete Flag| | AssignedTo|t....@ze... |gstreamer- | |bu...@li... Status|NEW |RESOLVED Resolution| |FIXED ------- Comment #8 from Tim-Philipp Müller 2007-11-01 23:52 UTC ------- After further discussion on IRC: Nov 01 17:52:45 <__tim> oleavr: can I bug you again about the export declaration for a second? Nov 01 17:54:20 <__tim> oleavr: why does it need to be exported at all? exported debug category in plugins usually just means that other .c files within the same plugin should be able to use the symbol, but it doesn't need to be visible to code outside the plugin afaics Nov 01 17:55:12 <oleavr> __tim: ahh.. that explain a few things, I was wondering what the dllimport was doing there in the first place Nov 01 17:56:16 <oleavr> __tim: let me try removing it here and see if MSVC goes on a pony killing spree :) Nov 01 19:49:00 <oleavr> __tim: I'm puzzled about the whole __declspec(dllimport) for the GST_DEBUG_CATEGORY_EXTERN, can't see why it was there in the first place, it should clearly be extern for all cases.. and I just verified this with MSVC, builds just fine without any warnings Nov 01 19:50:44 <oleavr> __tim: so it'd rock if you could fix it by removing the MSVC special-casing and mark the remaining patch on #492077 as obsolete :) Nov 01 19:56:34 <__tim> oleavr: okay, I'll have another look later; maybe there's a hint in the history why it was added in the first place :) Nov 01 20:24:49 <oleavr>__tim: tracked it down: http://webcvs.freedesktop.org/gstreamer/gst-plugins-good/gst/avi/gstavidemux.c?r1=1.181&r2=1.182 http://webcvs.freedesktop.org/gstreamer/gst-plugins-good/gst/avi/gstavidemux.c?r1=1.186&r2=1.187 http://webcvs.freedesktop.org/gstreamer/gstreamer/gst/gstinfo.h?r1=1.94&r2=1.95 http://webcvs.freedesktop.org/gstreamer/gstreamer/gst/gstinfo.h?r1=1.99&r2=1.100 http://webcvs.freedesktop.org/gstreamer/gstreamer/gst/gstinfo.h?r1=1.100&r2=1.101 looks to me like the initial fix couldn't possibly have been tested given that it has a syntax error (_declspec instead of __declspec) Nov 01 20:26:22 <__tim> oh THAT's where it comes from, it was just copy'n'pasted from a plugin Nov 01 20:26:25 <__tim> excellent 2007-11-01 Tim-Philipp Müller <tim at centricular dot net> * gst/gstinfo.h: (GST_DEBUG_CATEGORY_EXTERN): Remove __declspec(dllimport) for MSVC that was copied over into core from a plugin, obviously without ever having been tested (note the single underscore in _declspec in the initial commit), and that doesn't really make sense. See #492077. -- See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received this email, why you can't respond via email, how to stop receiving emails (or reduce the number you receive), and how to contact someone if you are having problems with the system. You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=492077. |