[Buildroot] [PATCH 1/1 v2] qdecoder : new package

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Fri Apr 25 07:18:06 UTC 2014


Dear Sagaert Johan,

On Thu, 24 Apr 2014 22:43:14 +0200, Sagaert Johan wrote:

> What should I do with the BR2_PREFER_STATIC_LIB ? 
> Is this a something I should add as depenency ?

Yes. You should add !BR2_PREFER_STATIC_LIB to the dependencies, and add
the corresponding comment (see the Buildroot manual for details).

> (I am not very familiar with libraries, I assume that qdecoder.a is a static library and the others (*.so.*) are shared)
> 
> What should I add to finalize this patch ?
> I added a comment line to the QDECODER_CONF_OPT line for my next version of the patch. 

You should get rid of the --prefix and --libdir options, by patching
the Makefile.in to add ${DESTDIR} in front of the installation paths in
the 'install' target. Then you can send this patch upstream :-)

Thanks a lot,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



More information about the buildroot mailing list