From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Korsgaard Date: Tue, 28 Jan 2014 22:21:01 +0100 Subject: [Buildroot] [git commit] minidlna: fix static linking In-Reply-To: <20140128221653.6fc8974b@skate> (Thomas Petazzoni's message of "Tue, 28 Jan 2014 22:16:53 +0100") References: <20140108082945.20CF58DE52@busybox.osuosl.org> <20140128221653.6fc8974b@skate> Message-ID: <87sis72476.fsf@dell.be.48ers.dk> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net >>>>> "Thomas" == Thomas Petazzoni writes: > Dear Peter Korsgaard, > On Wed, 8 Jan 2014 09:00:11 +0100, Peter Korsgaard wrote: >> -# static build is broken w.r.t libgcc_s >> +ifeq ($(BR2_PREFER_STATIC_LIB),y) >> +# the configure script / Makefile forgets to link with some of the dependent >> +# libraries breaking static linking, so help it along >> +MINIDLNA_CONF_ENV = \ >> + LIBS='-lavformat -lavcodec -lavutil -logg -lz -lpthread -lm' >> +else > I know we're doing this in a bunch of cases, but I believe we should > rather fix the package itself, rather than encoding into the > Buildroot .mk file the list of libraries to link with. Because if > libogg or libavutil becomes unnecessary in a future version of > minidlna, we may not notice, and still link needlessly against them. True. Patches are welcome ;) -- Bye, Peter Korsgaard