Re: [dev] [st][PATCH] minor: remove unnecessary/redundant libraries

From: Markus Teich <markus.teich_AT_stusta.mhn.de>
Date: Thu, 24 Jul 2014 19:31:03 +0200

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 <20h_AT_r-36.net> Fri Jan 4 20:52:22 2013 +0100
> committer Christoph Lohmann <20h_AT_r-36.net> 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
> _AT_@ -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}\"
Received on Thu Jul 24 2014 - 19:31:03 CEST