On Thu, May 06, 2010 at 09:39:42AM +1000, Brendan O'Dea wrote:
> On 6 May 2010 04:57, Niko Tyni <[email protected]> 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   [email protected]



--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]

Reply via email to