[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#750017: perl-policy: All packages using Perl vendorarch directory need a perlapi-* dependency



On Sun, 01 Jun 2014 09:53:11 -0700, Russ Allbery wrote:

> Niko Tyni <ntyni@debian.org> writes:
> > I now realize that this wording unintentionally removes the perlapi
> > requirement for binary modules outside vendorarch (see for instance
> > the amanda-common package, which uses /usr/lib/amanda/perl/).
> 
> > Revised patch attached. The new wording feels a bit awkward but
> > I guess it will do. Improvements and eyeballs welcome, of course.
> 
> Ah, good catch.  The new wording seems fine to me.  Seconded.

Improved wording seconded as well.

Cheers,
gregor
 
> > From: Niko Tyni <ntyni@debian.org>
> > Date: Fri, 30 May 2014 14:10:22 +0300
> > Subject: [PATCH] All packages using Perl vendorarch directory need a perlapi-*
> >  dependency
> 
> > Having $Config{vendorarch} change between Perl major versions (for
> > multiarch or other reasons) implies that packages using it need a strict
> > dependency on those versions of perl-base that have a compatible search
> > path (@INC).
> 
> > The vast majority of these packages are binary ("XS") modules, where we
> > already require a dependency on the virtual package perlapi-*, provided
> > by perl-base. The name of this virtual package will change when the
> > interface between the Perl interpreter and the modules changes.
> 
> > If we consider @INC part of that interface, we can use the existing
> > mechanism also for the few nonbinary modules using $Config{vendorarch}.
> > This is a new requirement that currently affects six packages in
> > the archive.
> > ---
> >  perl-policy.sgml | 9 +++++----
> >  1 file changed, 5 insertions(+), 4 deletions(-)
> 
> > diff --git a/perl-policy.sgml b/perl-policy.sgml
> > index c23f7c3..12cd82c 100644
> > --- a/perl-policy.sgml
> > +++ b/perl-policy.sgml
> > @@ -388,14 +388,15 @@ $(MAKE) install DESTDIR=$(CURDIR)/debian/&lt;tmp&gt;
> >  	</sect1>
> >  
> >  	<sect1 id="binary_modules">
> > -	  <heading>Binary Modules</heading>
> > +	  <heading>Binary and Other Architecture Dependent Modules</heading>
> >  	  <p>
> >  	    Binary modules must specify a dependency on either
> >  	    <package>perl</package> or <package>perl-base</package> with
> >  	    a minimum version of the <package>perl</package> package
> > -	    used to build the module, and must additionally depend on
> > -	    the expansion of
> > -	    <package>perlapi-$Config{debian_abi}</package> using
> > +	    used to build the module. Additionally, all binary modules
> > +	    (regardless of their installation directory) and any other modules
> > +	    installed into <tt>$Config{vendorarch}</tt> must depend on the
> > +	    expansion of <package>perlapi-$Config{debian_abi}</package> using
> >  	    the <tt>Config</tt> module. If <tt>$Config{debian_abi}</tt>
> >  	    is empty or not set, <tt>$Config{version}</tt> must be used.
> >  	  </p>
> 

-- 
 .''`.  Homepage: http://info.comodo.priv.at/ - OpenPGP key 0xBB3A68018649AA06
 : :' : Debian GNU/Linux user, admin, and developer  -  http://www.debian.org/
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   NP: Ludwig Hirsch: Der Turm

Attachment: signature.asc
Description: Digital Signature


Reply to: