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

Bug#579457: debian-policy: finer granularity for perlapi-*



On Thu, May 06, 2010 at 09:39:42AM +1000, Brendan O'Dea wrote:
> On 6 May 2010 04:57, Niko Tyni <ntyni@debian.org> wrote:

> > I'm thinking the dependants could use something like
> >  "perlapi-" . ($Config{debian_abi} || $Config{version})

> Rather than "evaluates to false", I'd suggest wording it:
> 
>   If <tt>$Config{debian_abi}</tt> is not set or empty,
> <tt>$Config{version}</tt> must be used.
> 
> as this more clearly indicates the intent.

I was trying to cater for the degenerate case of $Config{debian_abi}=0
(which breaks the above short-circuit form) so dependants wouldn't have
to check for that.

If we need that at all, I suppose it should go in the perl-base requirements.

New try. I replaced "abiversion" with "abiname" to emphasize that it does
not need to look like a version number, and applied your suggestion with
 s/is not set or empty/is empty or not set/
to avoid a slight precedence ambiguity.

diff --git a/perl-policy.sgml b/perl-policy.sgml
index 1d26df7..a71dd21 100644
--- a/perl-policy.sgml
+++ b/perl-policy.sgml
@@ -89,8 +89,11 @@
 	</p>
 	<p>
 	  The <package>perl-base</package> package must provide
-	  <package>perlapi-<var>version</var></package> for all released
-	  versions it is compatible with.
+	  <package>perlapi-<var>abiname</var></package> for all released
+	  package versions it is compatible with. The choice of
+	  a nonzero <var>abiname</var> is arbitrary, but if it differs from
+	  <tt>$Config{version}</tt>, it must be specified in
+	  <tt>$Config{debian_abi}</tt>.
 	</p>
       </sect>
 
@@ -348,8 +351,9 @@ $(MAKE) install PREFIX=$(CURDIR)/debian/&lt;tmp&gt;/usr
 	    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{version}</package> using
-	    the <tt>Config</tt> module.
+	    <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>
 	</sect1>
 
-- 
Niko Tyni   ntyni@debian.org



Reply to: