• Request for review / upload for libintl-perl

    From Ken Ibbotson@21:1/5 to All on Mon Jan 11 11:20:02 2021
    Please review and upload if acceptable.

    dpt checkout libintl-perl

    2 patches have been raised to handle spelling and POD markup corrections.

    Lintian does show a couple more items - please advise on the same if not
    ready for upload.

    I did have to increment the dot version in changelog as the 3000 build
    action currently happening updated to version, and the 'dpt takeover' used
    the same version number '1.26-2.1`.
    Not sure if this may be a bug with the takeover script.

    Thanks in advance
    --
    Ken Ibbotson
    E: keni@computer.org

    *"Reality is merely an illusion, albeit a very persistent one."*
    - Albert Einstein (1879-1955)

    <div dir="ltr"><div>Please review and upload if acceptable.</div><div><br></div><div>dpt checkout libintl-perl</div><div><br></div><div>2 patches have been raised to handle spelling and POD markup corrections.</div><div><br></div><div>Lintian does show a
    couple more items - please advise on the same if not ready for upload.</div><div><br></div><div>I did have to increment the dot version in changelog as the 3000 build action currently happening updated to version, and the &#39;dpt takeover&#39; used the
    same version number &#39;1.26-2.1`.</div><div>Not sure if this may be a bug with the takeover script.</div><div></div><div><br></div><div>Thanks in advance<br></div><div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div
    dir="ltr"><div><div>--<br>Ken Ibbotson</div>E: <a href="mailto:keni@computer.org" target="_blank">keni@computer.org</a><br><div><br></div><div><i>&quot;Reality is merely an illusion, albeit a very persistent one.&quot;</i></div>    - Albert Einstein (
    1879-1955)<br></div></div></div></div></div></div>

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From gregor herrmann@21:1/5 to Ken Ibbotson on Sun Jan 17 16:00:03 2021
    On Mon, 11 Jan 2021 20:48:27 +1030, Ken Ibbotson wrote:

    Please review and upload if acceptable.
    dpt checkout libintl-perl

    Thanks for your work on this package!

    Some remarks:

    - d/changelog:
    + "Peter Eisentraut <petere@debian.org> moved to Uploaders." should
    be removed, as you have later correctly removed them from
    Uploaders.
    + Maybe document some more of your changes (e.g. the conversion of
    d/copyright is missing).

    - d/copyright:
    + including old copyright holders for debian/* might be nice,
    cf. `dh-make-perl refresh --only copyright'
    + you used "Artistic or GPL-1+" for debian/*, which is of course
    fine; in general we use the superset of upstream license and perl
    license, aka "GPL-3+ or Artistic or GPL-1+"

    - d/control:
    + "perl (>= 5.8.0-7)" -> the version can be dropped, as even oldoldstable
    has 5.20; cf. `cme fix dpkg-control'
    + this also removes the version from "dpkg-dev (>= 1.16.1~)", and
    an unversioned dpkg-dev can be dropped completely

    - debian/.gbp.conf
    does this work? gbp.conf(5) mentions debian/gbp.conf.

    - lintian overrides: they make me curious, especially because they
    don't have a comment, explaining what they do :)
    Let's see:
    I: libintl-xs-perl: unused-override unusual-interpreter
    So this one can be removed
    O: libintl-perl: unusual-interpreter usr/share/perl5/Locale/Messages.pm #!/bin/false
    Hm, ok weird but well …

    - debian/rules:
    + PERL_MM_USE_DEFAULT=1 can be removed (I believe it's set by the
    debhelper perl makemaker buildsystem)
    + The old d/rules had some none-default commands which are missing
    now. Let's see what they do …
    Test 1: building twice in a row. This fails with
    dpkg-source: info: local changes detected, the modified files are:
    libintl-perl-1.26/ReleaseNotes
    libintl-perl-1.26/config.log
    So these two files should be put into debian/clean

    Then we had "dh_install -i --exclude=xs", which surprised me but
    explains the lintian warning
    W: libintl-perl source: binaries-have-file-conflict libintl-perl libintl-xs-perl usr/share/man/man3/Locale::gettext_xs.3pm.gz
    i.e. this file is in both packages which is prevented by the
    --exclude. - Hm, how is this written nowadays?
    The following seems to work:

    override_dh_install:
    dh_install -a
    dh_install -i --exclude=xs

    - autopkgtests fail or are skipped … We need
    + debian/tests/pkg-perl/smoke-files with test.pl and tests/
    and then it still fails *sigh*
    cannot open './xs_disabled': No such file or directory at ./test.pl line 59.
    So we need a smoke-setup as well … and here we go.
    + debian/tests/pkg-perl/use-name with the name of one perl module,
    maybe Locale::Recode
    + I've committed and pushed the autopkgtests changes as I already
    have them here :)

    2 patches have been raised to handle spelling and POD markup corrections.

    Thanks.

    I did have to increment the dot version in changelog as the 3000 build
    action currently happening updated to version, and the 'dpt takeover' used the same version number '1.26-2.1`.

    This (-2.2) is not correct. -X.Y are for non-maintainer uploads by
    convention (as the -2.1 from Holger) but since we are doing a regular maintainer upload now, -3 is the next "regular" debian revision.

    See also lintian's
    W: libintl-perl source: maintainer-upload-has-incorrect-version-number 1.26-2.2

    Not sure if this may be a bug with the takeover script.

    Well, what dpt-takeover did was
    +libintl-perl (1.26-2.1ubuntu1) UNRELEASED; urgency=medium
    which points to a different enironment :)

    (In general, dpt-takeover just calls `dch' which picks the next
    version.)

    Lintian does show a couple more items - please advise on the same if not ready for upload.

    Let's see what else we have besides the messages already discussed:

    W: libintl-perl: national-encoding usr/share/doc/libintl-perl/examples/simplecal/po/ar.po
    Well, yes. Might warrant an override.

    I: libintl-perl source: duplicate-short-description libintl-perl libintl-xs-perl
    Hm, yes, the short description is identical. If there is some space
    in <80 characters, libintl-xs-perl could add something about XS
    there.
    If not, no big deal.

    I: libintl-perl: example-wrong-path-for-interpreter usr/share/doc/libintl-perl/examples/simplecal/Makefile.PL (#!/usr/local/bin/perl != /usr/bin/perl)
    I: libintl-perl: example-wrong-path-for-interpreter usr/share/doc/libintl-perl/examples/simplecal/bin/simplecal.pl (#!/usr/local/bin/perl != /usr/bin/
    P: libintl-perl: example-unusual-interpreter usr/share/doc/libintl-perl/examples/simplecal/lib/SimpleCal.pm #!/bin/false

    I think these should be fixed (at least the first two). https://perl-team.pages.debian.net/debhelper.html has some recipes,
    cf. the "Note on paths" and "Fixing Interpreter Shebang Lines"
    sections

    I: libintl-perl source: patch-not-forwarded-upstream debian/patches/dot-inc.patch
    Might be worth taking a look at (if it's already fixed upstream or
    needs forwarding etc.)


    Ok, I think that's all for now :)


    Cheers,
    gregor


    --
    .''`. https://info.comodo.priv.at -- Debian Developer https://www.debian.org
    : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D 85FA BB3A 6801 8649 AA06
    `. `' Member VIBE!AT & SPI Inc. -- Supporter Free Software Foundation Europe
    `- NP: Mark Knopfler: Irish Boy

    -----BEGIN PGP SIGNATURE-----

    iQKTBAEBCgB9FiEE0eExbpOnYKgQTYX6uzpoAYZJqgYFAmAEUKNfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEQx RTEzMTZFOTNBNzYwQTgxMDREODVGQUJCM0E2ODAxODY0OUFBMDYACgkQuzpoAYZJ qgaEHhAAnwCYJNTpEGKlK07DynYmLi6xtNl98+9fAL8942Nmnr0wEkqpl3hBBmn0 yrW1UJA8dl3iTnnXvrU7Y5XuUk3QXlDtR3UbFCgjpQ5TVnVI6NlygIKmjJTKMRJ1 2DQT/Qf2ledMIwiPOXqbqy/SrAz1reYWD0lZuPiHTENzYaIQaXYxbmQO338w/da+ Jpxe1x5IhIDGPMt5a4Z8koByNn2j1JZbsR0snN4HwWY36mZdkkU4MTyUL31jK5Ra E3aY+7pC2aHt9ShrabmAhT1JdOPw9WokBAB1o+hiBdkbfIQ/9vAxbVGcaGShPH9W /bMQ767WdG0iLywXWibsMdhYLzfxxueP5ppvBZp/5akW5HGCTeF08KvOUM/zlJwZ fwOiW7X/8OrFNpkggPxcVqOWl68pxaXNnBUyP1cJRw3remkKdqkm6SC2cliEOt3N cGoBXkAe9CBK+W40bD5QjJ9Z9lZHK+Zp3I/eBC2KcX2v73
  • From Ken Ibbotson@21:1/5 to gregor herrmann on Tue Feb 2 09:10:01 2021
    Hi Gregor, and anyone else who may be able to assist,

    I have resolved a number of the below Lintian results, and pushed to origin.

    Having trouble with the shebang changes needed by Lintian, and the scripts indicated.
    I can run the command locally as:
    sed -Ei -e '1s|^#!\s*/usr/local/bin/perl|#!/usr/bin/perl|' ./sample/simplecal/bin/simplecal.pl
    gdh sample/simplecal/bin/simplecal.pl
    diff --git a/sample/simplecal/bin/simplecal.pl b/sample/simplecal/bin/ simplecal.pl
    index ec184ba..8dee002 100644
    --- a/sample/simplecal/bin/simplecal.pl
    +++ b/sample/simplecal/bin/simplecal.pl
    @@ -1,4 +1,4 @@
    -#! /usr/local/bin/perl -w
    +#!/usr/bin/perl -w
    # vim: tabstop=4

    use strict;
    and the file is updated properly.

    However, it fails to update the files under a build with the following in
    the `debian/rules` file:
    override_dh_installexamples:
    dh_installexamples
    find $(TMP)/usr/share/doc/$(PACKAGE)/examples -type f -print0 | \
    xargs -r0 sed -E -i -e
    '1s|^#!\s*/usr/local/bin/perl|#!/usr/bin/perl|'

    The first observation is that there are no `TMP` or `PACKAGE` environment variables defined, so the `find` would be akin to:
    override_dh_installexamples:
    dh_installexamples
    find /usr/share/doc//examples -type f -print0 | \
    xargs -r0 sed -E -i -e
    '1s|^#!\s*/usr/local/bin/perl|#!/usr/bin/perl|'

    The sample file used above to prove the command works manually, is located under: ./debian/libintl-perl/usr/share/doc/libintl-perl/examples/simplecal/lib/SimpleCal.pm
    and
  • From gregor herrmann@21:1/5 to Ken Ibbotson on Tue Feb 2 11:50:01 2021
    On Tue, 02 Feb 2021 18:39:04 +1030, Ken Ibbotson wrote:

    The first observation is that there are no `TMP` or `PACKAGE` environment variables defined, so the `find` would be akin to:

    I can't look at the package right now but maybe these thoughts help:

    From my previous mail:

    | https://perl-team.pages.debian.net/debhelper.html has some recipes,
    | cf. the "Note on paths" and "Fixing Interpreter Shebang Lines"
    | sections

    "Note on paths" means this section:

    https://perl-team.pages.debian.net/debhelper.html#Note_on_Paths

    which recommends to add

    | PACKAGE = $(shell dh_listpackages)
    | TMP = $(CURDIR)/debian/$(PACKAGE)

    to the top of d/rules.

    Maybe that's already all that's missing :)


    Cheerts,
    gregor

    --
    .''`. https://info.comodo.priv.at -- Debian Developer https://www.debian.org
    : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D 85FA BB3A 6801 8649 AA06
    `. `' Member VIBE!AT & SPI Inc. -- Supporter Free Software Foundation Europe
    `- BOFH excuse #163: no "any" key on keyboard

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Ken Ibbotson@21:1/5 to gregor herrmann on Sun Feb 7 06:30:01 2021
    I have marked this package 'unstable' to signify that I believe it is ready
    to upload.

    All suggested changes have been completed, with the exception of one:

    W: libintl-perl: national-encoding usr/share/doc/libintl-perl/examples/simplecal/po/ar.po
    Well, yes. Might warrant an override.

    I tried to run 'iconv' against this as a possible override, but alas it
    baulked at the line 18 - the very first converted value in Arabic.
    Added a Lintian override for 'national-encoding'.

    Please advise if not suitable, as I can always revert the Lintain override.

    --
    Ken Ibbotson
    E: keni@computer.org

    *"Reality is merely an illusion, albeit a very persistent one."*
    - Albert Einstein (1879-1955)


    On Mon, 18 Jan 2021 at 01:29, gregor herrmann <gregoa@debian.org> wrote:

    On Mon, 11 Jan 2021 20:48:27 +1030, Ken Ibbotson wrote:

    Please review and upload if acceptable.
    dpt checkout libintl-perl

    Thanks for your work on this package!

    Some remarks:

    - d/changelog:
    + "Peter Eisentraut <petere@debian.org> moved to Uploaders." should
    be removed, as you have later correctly removed them from
    Uploaders.
    + Maybe document some more of your changes (e.g. the conversion of
    d/copyright is missing).

    - d/copyright:
    + including old copyright holders for debian/* might be nice,
    cf. `dh-make-perl refresh --only copyright'
    + you used "Artistic or GPL-1+" for debian/*, which is of course
    fine; in general we use the superset of upstream license and perl
    license, aka "GPL-3+ or Artistic or GPL-1+"

    - d/control:
    + "perl (>= 5.8.0-7)" -> the version can be dropped, as even oldoldstable
    has 5.20; cf. `cme fix dpkg-control'
    + this also removes the version from "dpkg-dev (>= 1.16.1~)", and
    an unversioned dpkg-dev can be dropped completely

    - debian/.gbp.conf
    does this work? gbp.conf(5) mentions debian/gbp.conf.

    - lintian overrides: they make me curious, especially because they
    don't have a comment, explaining what they do :)
    Let's see:
    I: libintl-xs-perl: unused-override unusual-interpreter
    So this one can be removed
    O: libintl-perl: unusual-interpreter usr/share/perl5/Locale/Messages.pm #!/bin/false
    Hm, ok weird but well …

    - debian/rules:
    + PERL_MM_USE_DEFAULT=1 can be removed (I believe it's set by the
    debhelper perl makemaker buildsystem)
    + The old d/rules had some none-default commands which are missing
    now. Let's see what they do …
    Test 1: building twice in a row. This fails with
    dpkg-source: info: local changes detected, the modified files are:
    libintl-perl-1.26/ReleaseNotes
    libintl-perl-1.26/config.log
    So these two files should be put into debian/clean

    Then we had "dh_install -i --exclude=xs", which surprised me but
    explains the lintian warning
    W: libintl-perl source: binaries-have-file-conflict libintl-perl libintl-xs-perl usr/share/man/man3/Locale::gettext_xs.3pm.gz
    i.e. this file is in both packages which is prevented by the
    --exclude. - Hm, how is this written nowadays?
    The following seems to work:

    override_dh_install:
    dh_install -a
    dh_install -i --exclude=xs

    - autopkgtests fail or are skipped … We need
    + debian/tests/pkg-perl/smoke-files with test.pl and tests/
    and then it still fails *sigh*
    cannot open './xs_disabled': No such file or directory at ./test.pl
    line 59.
    So we need a smoke-setup as well … and here we go.
    + debian/tests/pkg-perl/use-name with the name of one perl module,
    maybe Locale::Recode
    + I've committed and pushed the autopkgtests changes as I already
    have them here :)

    2 patches have been raised to handle spelling and POD markup corrections.

    Thanks.

    I did have to increment the dot version in changelog as the 3000 build action currently happening updated to version, and the 'dpt takeover'
    used
    the same version number '1.26-2.1`.

    This (-2.2) is not correct. -X.Y are for non-maintainer uploads by
    convention (as the -2.1 from Holger) but since we are doing a regular maintainer upload now, -3 is the next "regular" debian revision.

    See also lintian's
    W: libintl-perl source: maintainer-upload-has-incorrect-version-number 1.26-2.2

    Not sure if this may be a bug with the takeover script.

    Well, what dpt-takeover did was
    +libintl-perl (1.26-2.1ubuntu1) UNRELEASED; urgency=medium
    which points to a different enironment :)

    (In general, dpt-takeover just calls `dch' which picks the next
    version.)

    Lintian does show a couple more items - please advise on the same if not ready for upload.

    Let's see what else we have besides the messages already discussed:

    W: libintl-perl: national-encoding usr/share/doc/libintl-perl/examples/simplecal/po/ar.po
    Well, yes. Might warrant an override.

    I: libintl-perl source: duplicate-short-description libintl-perl libintl-xs-perl
    Hm, yes, the short description is identical. If there is some space
    in <80 characters, libintl-xs-perl could add something about XS
    there.
    If not, no big deal.

    I: libintl-perl: example-wrong-path-for-interpreter usr/share/doc/libintl-perl/examples/simplecal/Makefile.PL (#!/usr/local/bin/perl != /usr/bin/perl)
    I: libintl-perl: example-wrong-path-for-interpreter usr/share/doc/libintl-perl/examples/simplecal/bin/simplecal.pl (#!/usr/local/bin/perl != /usr/bin/
    P: libintl-perl: example-unusual-interpreter usr/share/doc/libintl-perl/examples/simplecal/lib/SimpleCal.pm #!/bin/false

    I think these should be fixed (at least the first two). https://perl-team.pages.debian.net/debhelper.html has some recipes,
    cf. the "Note on paths" and "Fixing Interpreter Shebang Lines"
    sections

    I: libintl-perl source: patch-not-forwarded-upstream debian/patches/dot-inc.patch
    Might be worth taking a look at (if it's already fixed upstream or
    needs forwarding etc.)


    Ok, I think that's all for now :)


    Cheers,
    gregor


    --
    .''`. https://info.comodo.priv.at -- Debian Developer https://www.debian.org
    : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D 85FA BB3A 6801 8649 AA06
    `. `' Member VIBE!AT & SPI Inc. -- Supporter Free Software Foundation Europe
    `- NP: Mark Knopfler: Irish Boy


    <div dir="ltr"><div dir="ltr"><div>I have marked this package &#39;unstable&#39; to signify that I believe it is ready to upload.</div><div><br></div><div>All suggested changes have been completed, with the exception of one:</div><div><br></div><div
    style="margin-left:40px">W: libintl-perl: national-encoding usr/share/doc/libintl-perl/examples/simplecal/po/ar.po<br>
    Well, yes. Might warrant an override.</div><div><br></div><div>I tried to run &#39;iconv&#39; against this as a possible override, but alas it baulked at the line 18 - the very first converted value in Arabic.</div><div>Added a Lintian override for &#39;
    national-encoding&#39;.</div><div><br></div><div>Please advise if not suitable, as I can always revert the Lintain override.<br></div><div><br></div><div></div><div><div><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="
    ltr"><div><div>--<br>Ken Ibbotson</div>E: <a href="mailto:keni@computer.org" target="_blank">keni@computer.org</a><br><div><br></div><div><i>&quot;Reality is merely an illusion, albeit a very persistent one.&quot;</i></div>    - Albert Einstein (1879-
    1955)<br></div></div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 18 Jan 2021 at 01:29, gregor herrmann &lt;<a href="mailto:gregoa@debian.org">gregoa@debian.org</a>&gt; wrote:<br></div><blockquote
    class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Mon, 11 Jan 2021 20:48:27 +1030, Ken Ibbotson wrote:<br>

    &gt; Please review and upload if acceptable.<br>
    &gt; dpt checkout libintl-perl<br>

    Thanks for your work on this package!<br>

    Some remarks:<br>

    - d/changelog:<br>
      + &quot;Peter Eisentraut &lt;<a href="mailto:petere@debian.org" target="_blank">petere@debian.org</a>&gt; moved to Uploaders.&quot; should<br>
        be removed, as you have later correctly removed them from<br>
        Uploaders.<br>
      + Maybe document some more of your changes (e.g. the conversion of<br>
        d/copyright is missing).<br>

    - d/copyright:<br>
      + including old copyright holders for debian/* might be nice,<br>
        cf. `dh-make-perl refresh --only copyright&#39;<br>
      + you used &quot;Artistic or GPL-1+&quot; for debian/*, which is of course<br>
        fine; in general we use the superset of upstream license and perl<br>
        license, aka &quot;GPL-3+ or Artistic or GPL-1+&quot;<br>

    - d/control:<br>
      + &quot;perl (&gt;= 5.8.0-7)&quot; -&gt; the version can be dropped, as even oldoldstable<br>
        has 5.20; cf. `cme fix dpkg-control&#39;<br>
      + this also removes the version from &quot;dpkg-dev (&gt;= 1.16.1~)&quot;, and<br>
        an unversioned dpkg-dev can be dropped completely<br>

    - debian/.gbp.conf<br>
      does this work? gbp.conf(5) mentions debian/gbp.conf.<br>

    - lintian overrides: they make me curious, especially because they<br>
      don&#39;t have a comment, explaining what they do :)<br>
      Let&#39;s see:<br>
      I: libintl-xs-perl: unused-override unusual-interpreter<br>
      So this one can be removed<br>
      O: libintl-perl: unusual-interpreter usr/share/perl5/Locale/Messages.pm #!/bin/false<br>
      Hm, ok weird but well …<br>

    - debian/rules:<br>
      + PERL_MM_USE_DEFAULT=1 can be removed (I believe it&#39;s set by the<br>
        debhelper perl makemaker buildsystem)<br>
      + The old d/rules had some none-default commands which are missing<br>
        now. Let&#39;s see what they do …<br>
        Test 1: building twice in a row. This fails with<br>
        dpkg-source: info: local changes detected, the modified files are:<br>
         libintl-perl-1.26/ReleaseNotes<br>
         libintl-perl-1.26/config.log<br>
        So these two files should be put into debian/clean<br>

        Then we had &quot;dh_install -i --exclude=xs&quot;, which surprised me but<br>
        explains the lintian warning<br>
        W: libintl-perl source: binaries-have-file-conflict libintl-perl libintl-xs-perl usr/share/man/man3/Locale::gettext_xs.3pm.gz<br>
        i.e. this file is in both packages which is prevented by the<br>
        --exclude. - Hm, how is this written nowadays?<br>
        The following seems to work:<br>

        override_dh_install:<br>
            dh_install -a<br>
            dh_install -i --exclude=xs<br>

    - autopkgtests fail or are skipped … We need<br>
      + debian/tests/pkg-perl/smoke-files with <a href="http://test.pl" rel="noreferrer" target="_blank">test.pl</a> and tests/<br>
        and then it still fails *sigh*<br>
        cannot open &#39;./xs_disabled&#39;: No such file or directory at ./<a href="http://test.pl" rel="noreferrer" target="_blank">test.pl</a> line 59.<br>
        So we need a smoke-setup as well … and here we go.<br>
      + debian/tests/pkg-perl/use-name with the name of one perl module,<br>
        maybe Locale::Recode<br>
      + I&#39;ve committed and pushed the autopkgtests changes as I already<br>
        have them here :)<br>

    &gt; 2 patches have been raised to handle spelling and POD markup corrections.<br>

    Thanks.<br>

    &gt; I did have to increment the dot version in changelog as the 3000 build<br> &gt; action currently happening updated to version, and the &#39;dpt takeover&#39; used<br>
    &gt; the same version number &#39;1.26-2.1`.<br>

    This (-2.2) is not correct. -X.Y are for non-maintainer uploads by<br> convention (as the -2.1 from Holger) but since we are doing a regular<br> maintainer upload now, -3 is the next &quot;regular&quot; debian revision.<br>

    See also lintian&#39;s<br>
    W: libintl-perl source: maintainer-upload-has-incorrect-version-number 1.26-2.2<br>

    &gt; Not sure if this may be a bug with the takeover script.<br>

    Well, what dpt-takeover did was<br>
    +libintl-perl (1.26-2.1ubuntu1) UNRELEASED; urgency=medium<br>
    which points to a different enironment :)<br>

    (In general, dpt-takeover just calls `dch&#39; which picks the next<br> version.)<br>

    &gt; Lintian does show a couple more items - please advise on the same if not<br>
    &gt; ready for upload.<br>

    Let&#39;s see what else we have besides the messages already discussed:<br>

    W: libintl-perl: national-encoding usr/share/doc/libintl-perl/examples/simplecal/po/ar.po<br>
    Well, yes. Might warrant an override.<br>

    I: libintl-perl source: duplicate-short-description libintl-perl libintl-xs-perl<br>
    Hm, yes, the short description is identical. If there is some space<br>
    in &lt;80 characters, libintl-xs-perl could add something about XS<br> there.<br>
    If not, no big deal.<br>

    I: libintl-perl: example-wrong-path-for-interpreter usr/share/doc/libintl-perl/examples/simplecal/Makefile.PL (#!/usr/local/bin/perl != /usr/bin/perl)<br>
    I: libintl-perl: example-wrong-path-for-interpreter usr/share/doc/libintl-perl/examples/simplecal/bin/<a href="http://simplecal.pl" rel="noreferrer" target="_blank">simplecal.pl</a> (#!/usr/local/bin/perl != /usr/bin/<br>
    P: libintl-perl: example-unusual-interpreter usr/share/doc/libintl-perl/examples/simplecal/lib/SimpleCal.pm #!/bin/false<br>

    I think these should be fixed (at least the first two).<br>
    <a href="https://perl-team.pages.debian.net/debhelper.html" rel="noreferrer" target="_blank">https://perl-team.pages.debian.net/debhelper.html</a> has some recipes,<br>
    cf. the &quot;Note on paths&quot; and &quot;Fixing Interpreter Shebang Lines&quot;<br>
    sections<br>

    I: libintl-perl source: patch-not-forwarded-upstream debian/patches/dot-inc.patch<br>
    Might be worth taking a look at (if it&#39;s already fixed upstream or<br> needs forwarding etc.)<br>


    Ok, I think that&#39;s all for now :)<br>


    Cheers,<br>
    gregor<br>


    -- <br>
     .&#39;&#39;`.  <a href="https://info.comodo.priv.at" rel="noreferrer" target="_blank">https://info.comodo.priv.at</a> -- Debian Developer <a href="https://www.debian.org" rel="noreferrer" target="_blank">https://www.debian.org</a><br>
     : :&#39; : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D  85FA BB3A 6801 8649 AA06<br>
     `. `&#39;  Member VIBE!AT &amp; SPI Inc. -- Supporter Free Software Foundation Europe<br>
       `-   NP: Mark Knopfler: Irish Boy<br>
    </blockquote></div></div>

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From gregor herrmann@21:1/5 to Ken Ibbotson on Sat Feb 20 17:00:02 2021
    On Sun, 07 Feb 2021 15:53:47 +1030, Ken Ibbotson wrote:

    I have marked this package 'unstable' to signify that I believe it is ready to upload.

    Thanks, and sorry for missing your mail about it!

    I now took a last look at the package, tweaked 2 or 3 details and
    uploaded it.

    Thanks again for all your work on this non-trivial takeover.


    Cheers,
    gregor

    --
    .''`. https://info.comodo.priv.at -- Debian Developer https://www.debian.org
    : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D 85FA BB3A 6801 8649 AA06
    `. `' Member VIBE!AT & SPI Inc. -- Supporter Free Software Foundation Europe
    `- NP: James Taylor: Country Road

    -----BEGIN PGP SIGNATURE-----

    iQKTBAEBCgB9FiEE0eExbpOnYKgQTYX6uzpoAYZJqgYFAmAxMXpfFIAAAAAALgAo aXNzdWVyLWZwckBub3RhdGlvbnMub3BlbnBncC5maWZ0aGhvcnNlbWFuLm5ldEQx RTEzMTZFOTNBNzYwQTgxMDREODVGQUJCM0E2ODAxODY0OUFBMDYACgkQuzpoAYZJ qgbQZg//SGlUj90/Li2t5GHUz+VAviMV6pGDKVsGCu8PhXC87wRGVKzMY9d8tHma rSLjmZXqOCnPXzdr3lBQ0YVqHKOQNztCaRubGXgOUBVYyxoziXB9d3R8dRGg5b9R Bw+fi038WRW2MU+NqV4XxnXyIq9uU85YxRE/zXQcM19yc4s8/nPNl8SOrryFuvK9 RPJRf2jFI6T5HwXjbB1i3dE3Klf4GdlmAYv2M0ky74mgD9ZDZfzwKzBCjAIqLKrE v0ER7r1vx1miki+dUrqZW3k4tcbgjzQ//EIUBI4w1n3lYooY6cHKC6AQatMA+SYC 6Kpvf7wpQ4uxmaccz05YvPP4SBG+jzsm73262U7rqzmQNkTtnJVs7BaR1NEfN7/C BerPk4KXLSul+vzB5nLbQtpAI070vo0D58oBQtGCYSTV3pi0NWHADpX+fb5elIBK DB1THkultw+lgY+awUoJyt0aCeuiC/JvNtnYTxpV+KaOQB8e0wPxZiXg9ikyS3Pf 6EAUbNq3RkYpLnPyyHSFd9KYpWtp4sBjAclHuskP0N9NFV0IiS7fXcPQF7y7nmIB HtLmH2RIRVqrYjYrh42THXTma9OlOKr02D47LOmF7XTdeHyKBvTF5z58jYWSKX9t x6VPnpSOVOlhqH/RIZtaBfNvHItv5qgYW16a0keuz1ZOE1gt2zE=
    =1OMH
    -----END PGP SIGNATURE-----

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)