littleblank / mp4v2

Automatically exported from code.google.com/p/mp4v2
Other
0 stars 0 forks source link

mingw build issue #143

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
Build failure on mingw.
Tried with mingw-w64 cross compilation from Linux host, but others will be more 
or less the same.
Patch is attached.

1) libplatform/platform_win32.cpp is missing from GNUmakefile.am

2) In platform.h, currently mingw goes along the same path as MSVC and requires 
proper definition of MP4V2_EXPORTS or MP4V2_USE_STATIC_LIB so that MP4V2_EXPORT 
macro is properly evaluated to __declspec().
However, currently none of symbols are provided via configure/makefile. As a 
result, MP4V2_EXPORT is always evaluated to __declspec(dllimport), and results 
in  build error.
In the patch, I simply fixed this condition so that mingw goes to the same path 
as other GCC environments.
Since mingw gcc (version 4) happily accepts 
__atribute__((visibility("default"))), no need to use MSVC way.

3) By the previous fix, mingw DLL compilation can now be enabled. So, I fixed 
configure.ac to enable it.

Original issue reported on code.google.com by honeycom...@gmail.com on 29 Oct 2012 at 5:01

Attachments:

GoogleCodeExporter commented 8 years ago
Thanks!  I have a bit of a backlog going on, so I'll get this as soon as I can.

Original comment by kid...@gmail.com on 29 Oct 2012 at 1:11

GoogleCodeExporter commented 8 years ago
If I add commit rights, could you apply this patch?  I'm super swamped at 
work/in life, but I agree this should be applied.  Let me know if that's okay 
with you.

Original comment by kid...@gmail.com on 28 Dec 2012 at 4:55

GoogleCodeExporter commented 8 years ago
Well, to tell the truth, I hesitate to touch official repo (I'm not familiar 
with subversion).
However, if you want to me do so, I will try it. Setting up svn client is not 
problem at all (it's a simple matter of apt-get or something), and maybe I can 
use git-svn (actually I'm using it for my private copy of mp4v2).

Original comment by honeycom...@gmail.com on 29 Dec 2012 at 6:04