[Ffmpeg-devel] [PATCH] rewrite vhook/drawtext.c

[Gustavo Sverzut Barbieri]

On 9/5/06, Diego Biurrun <diego at biurrun.de> wrote:
> On Mon, Sep 04, 2006 at 09:26:14PM -0300, Gustavo Sverzut Barbieri wrote:
> > On 9/4/06, Diego Biurrun <diego at biurrun.de> wrote:
> > >On Mon, Sep 04, 2006 at 10:19:39AM -0300, Gustavo Sverzut Barbieri wrote:
> > >>
> > >> Comments are always welcome!
> > >>
> > >> --- Makefile  (revision 6159)
> > >> +++ Makefile  (working copy)
> > >> @@ -15,12 +15,18 @@
> > >>
> > >>  ifeq ($(HAVE_FREETYPE2),yes)
> > >>      HOOKS += drawtext$(SLIBSUF)
> > >> -    CFLAGS += `freetype-config --cflags`
> > >> -    LDFLAGS += `freetype-config --libs`
> > >> +
> > >> +drawtext.o: drawtext.c
> > >> +     $(CC) $(CFLAGS) -Wall -Werror -g -c -o $@ $< $(VHOOKCFLAGS)
> > >`freetype-config --cflags`
> > >
> > >Leave this like it was before, this is redundant.
> >
> > So, should I add $(freetype-config --cflags) and $(freetype-config
> > --libs) to flags so every hook will like to it?
>
> freetype-config --cflags only adds another include directory, this is
> harmless.

Ok, but take a look at my mail about undefined symbols due linker flag
--as-need. You can see that imlib2 also doesn't work, at least here it
doesn't.


-- 
Gustavo Sverzut Barbieri
--------------------------------------
Jabber: barbieri at gmail.com
   MSN: barbieri at gmail.com
  ICQ#: 17249123
 Skype: gsbarbieri
Mobile: +55 (81) 9927 0010
 Phone:  +1 (347) 624 6296; 08122692 at sip.stanaphone.com
   GPG: 0xB640E1A2 @ wwwkeys.pgp.net