[poppler] PATCH: Add includes to pdftocairo to compile in mingw

Albert Astals Cid aacid at kde.org
Sun Aug 27 10:52:37 UTC 2017


El divendres, 25 d’agost de 2017, a les 21:55:12 CEST, Adrian Johnson va 
escriure:
> On 25/08/17 03:54, Albert Astals Cid wrote:
> > Hi, i need this patch to be able to compile pdftocairo with mingw.
> > 
> > Any reason not to commit it?
> 
> I managed to get a cmake mingw cross compile working and also got the
> same error on pdftocairo-win32.cc. I compiled this file with verbose on
> with both autotools and cmake and compared the options.
> 
> The reason the cmake builds needs these includes is it is defining
> -DWIN32_LEAN_AND_MEAN which causes windows.h to include only a subset of
> the windows API. So adding these includes is fine.

According to someone that knows much more about windows than me 
WIN32_LEAN_AND_MEAN "is the way to go", so i'll commit the includes.

> I also noticed another difference between autotools and cmake. Autotools
> is compiling with -std=c++11. Cmake is using -std=gnu++11. I think
> -std=c++11 is better so as to avoid using anything that won't work with
> Visual Studio.

Fixed

> 
> There is also some additional mingw stuff in configure.ac:
>   mingw*)
>     os_win32=yes
>     win32_libs="-lgdi32 -lwinspool -lcomdlg32"
>     create_shared_lib="-no-undefined"
>     auto_import_flags="-Wl,--enable-auto-import"
> 
> I could not see this in cmake. It seems to compile without it. Not sure
> if it is needed for anything.

This seems like very old stuff, unless we have a reason to ad it, i'd rather 
not.

Cheers,
  Albert

> _______________________________________________
> poppler mailing list
> poppler at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/poppler




More information about the poppler mailing list