On 12/01/12 04:23, [email protected] wrote:
> Hi,
> 
> Current cmake file (utils/CMakeLists.txt) adds libcms
> to library list to build pdftocairo, like,
> 
>   add_executable(pdftocairo ${pdftocairo_SOURCES})
>   target_link_libraries(pdftocairo ${CAIRO_LIBRARIES} ${FREETYPE_LIBRARIES} 
> ${common_libs})
>   if(LCMS_FOUND)
>     target_link_libraries(pdftocairo ${LCMS_LIBRARIES})
>   endif(LCMS_FOUND)
> 
> But utils/Makefile.am does not do such. If it's not my
> misunderstanding, please consider to add LCMS_{CFLAGS,LIBS}
> to PDFTOCAIRO_{CFLAGS,LIBS} when USE_CMS is defined.
> Patch is following.

Pushed to master and 0.18. Thanks for the patch.

> 
> Regards,
> mpsuzuki
> 
> diff --git a/utils/Makefile.am b/utils/Makefile.am
> index 951abdb..762598c 100644
> --- a/utils/Makefile.am
> +++ b/utils/Makefile.am
> @@ -30,6 +30,11 @@ pdftocairo_SOURCES =                               \
>       pdftocairo.cc                           \
>       $(common)
>  
> +if USE_CMS
> +PDFTOCAIRO_CFLAGS += $(LCMS_CFLAGS)
> +PDFTOCAIRO_LIBS += $(LCMS_LIBS)
> +endif
> +
>  pdftocairo_LDADD =                           \
>       $(top_builddir)/poppler/libpoppler-cairo.la \
>       $(LDADD) $(PDFTOCAIRO_LIBS)
> _______________________________________________
> poppler mailing list
> [email protected]
> http://lists.freedesktop.org/mailman/listinfo/poppler

_______________________________________________
poppler mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/poppler

Reply via email to