SPECS: gd.spec (HEAD)

Artur Frysiak wiget w pld.org.pl
Śro, 23 Kwi 2003, 18:40:15 CEST


W liście z ?ro, 23-04-2003, godz. 18:30, Tomasz Kłoczko pisze: 
> On Mon, 21 Apr 2003, qboosh wrote:
> 
> > Module name:	SPECS
> > Changes by:	qboosh	03/04/21 03:15:23
> > 
> > Modified files:
> > 	gd.spec
> > 
> > Log message:
> > - removed setting LDFLAGS to "-s" to remove -s from `gdlib-config --ldflags`
> > - release 2
> > 
> > Index: gd.spec
> > ===================================================================
> > RCS file: /cvsroot/SPECS/gd.spec,v
> > diff -d -u -r1.79 -r1.80
> > --- gd.spec	15 Apr 2003 11:21:43 -0000	1.79
> > +++ gd.spec	21 Apr 2003 01:15:14 -0000	1.80
> > @@ -9,7 +9,7 @@
> >  Summary(pt_BR):	Biblioteca para manipulaçăo de imagens
> >  Name:		gd
> >  Version:	2.0.12
> > -Release:	1
> > +Release:	2
> >  License:	BSD-like
> >  Group:		Libraries
> >  Source0:	http://www.boutell.com/gd/http/%{name}-%{version}.tar.gz
> > @@ -29,6 +29,8 @@
> >  %{!?_without_gif:Provides:	gd(gif) = %{version}}
> >  
> >  %define		shlibver	%(echo %{version} | cut -f-2 -d.)
> > +# we don't want "-s" here, because it would be added to `gdlib-config --ldflags`
> > +%define		rpmldflags	%{nil}
> 
> Wydaje mi się, że nie jest to jest poprawne rozwiaznie. Raczej w to co
> wpada w ten skrypt nie powinno trafiać opcje linkera uzywane przy
> linkowaniu samej biblioteki i to tu raczje wydaje mi się jest błąd że
> zamiast wstawić tylko biblioteki ktoś był nadgorliwy i wrzucił opcje
> linkera.
> Żaden skrypt config czy pkg-config nie dostarczaja opcji linkera.

Zwróć jednak uwagę że LDFLAGS mogą zawierać nie tylko -s ale także -lfoo
-L/usr/lib/bar, co jest istotne przy linkowaniu (-l statycznym, -L także
dynamicznym).
Mnie się nie podoba wtykanie w proces budowania -s bo binarki i tak są
przecież później stripowane więc -s jest redundantne.

-- 
Artur Frysiak <wiget w pld.org.pl>



Więcej informacji o liście dyskusyjnej pld-devel-pl