[packages/rpm-pld-macros] - big cleanup of the main macros - removed macros identical with upstream - added new hotness for bu

Jan Rękorajski baggins at pld-linux.org
Fri Feb 7 12:56:42 CET 2025


On Thu, 06 Feb 2025, Jakub Bogusz wrote:

> On Wed, Feb 05, 2025 at 07:39:12PM +0100, Jakub Bogusz wrote:
> > On Wed, Feb 05, 2025 at 02:04:02AM +0100, baggins wrote:
> > >  
> > > -%configure {./configure \
> > > -	LDFLAGS="${LDFLAGS:-%rpmldflags}" \
> > > -	CFLAGS="${CFLAGS:-%rpmcflags}" \
> > > -	CXXFLAGS="${CXXFLAGS:-%rpmcxxflags}" \
> > > -	FFLAGS="${FFLAGS:-%rpmcflags}" \
> > > -	FCFLAGS="${FCFLAGS:-%rpmcflags}" \
> > > -	CPPFLAGS="${CPPFLAGS:-%rpmcppflags}" \
> > > -	%{?__cc:CC="%{__cc}"} \
> > > -	%{?__cxx:CXX="%{__cxx}"} \
> > > +%configure { \
> > > +  %{set_build_flags}; \
> > > +  ./configure \
> > >  	--host=%{_target_platform} \
> > >  	--build=%{_target_platform} \
> > >  	--prefix=%{_prefix} \
> > 
> > 
> > This change breaks invocation like:
> > 
> > bash %configure
> > 
> > if given configure is too bashish (now shebang change would be
> > required) - e.g. which.spec
> 
> And more cases:
> 
> 2)
> 
> VAR=val %configure
> 
> (now needs to be changed to export before %configure)
> 
> 3)
> 
> ../%configure ...
> 
> (see e.g. apache.spec)
> 
> This has no obvious replacement.
> In some cases maybe configure symlink would work, depending on source
> directory detection (relative to $0 or relative to $0 realpath)

I addressed shell and dir in 2.043 macros with configuredir and
configureshell macros. configure2_13 already had parametrized dir override, btw.

The requirement to export vars is actually good side effect, this was
inconsistent with all other macros and confusing.

-- 
Jan Rękorajski                    | PLD/Linux
SysAdm | baggins<at>pld-linux.org | http://www.pld-linux.org/


More information about the pld-devel-en mailing list