Roberto E. Vargas Caballero wrote:
> The patch, seems correct, but I am not sure why freetype was explicity added,
> if someone can add more information ...

Heyho,

commit 4682cc9596a5c82325882bdf1313728de95467d2 suggests portability as reason.
Also you can see that previously freetype2 was only an include and fontconfig
was only linked against.

--Markus


> author Christoph Lohmann <[email protected]> Fri Jan 4 20:52:22 2013 +0100
> committer Christoph Lohmann <[email protected]> Fri Jan 4 20:52:22 2013 +0100
> 
> Making the fontconfig and freetype paths portable in config.mk.
> 
> 
> diff --git a/config.mk b/config.mk
> index f1757ed..ecc5c83 100644
> --- a/config.mk
> +++ b/config.mk
> @@ -11,8 +11,12 @@ X11INC = /usr/X11R6/include
>  X11LIB = /usr/X11R6/lib
>  
>  # includes and libs
> -INCS = -I. -I/usr/include -I${X11INC} -I/usr/include/freetype2
> -LIBS = -L/usr/lib -lc -L${X11LIB} -lX11 -lutil -lXext -lXft -lfontconfig
> +INCS = -I. -I/usr/include -I${X11INC} \
> +       $(shell pkg-config --cflags fontconfig) \
> +       $(shell pkg-config --cflags freetype2)
> +LIBS = -L/usr/lib -lc -L${X11LIB} -lX11 -lutil -lXext -lXft \
> +       $(shell pkg-config --libs fontconfig)  \
> +       $(shell pkg-config --libs freetype2)
>  
>  # flags
>  CPPFLAGS = -DVERSION=\"${VERSION}\"

Reply via email to