Bug#893377: RFS: taptempo/1.2.1-1 [ITP]

[François Mazen <francois@mzf.fr>]

Hi,

I've uploaded a new release for my package taptempo (1.4.1-1), and I'm 
looking for a sponsor:

  Package: sponsorship-requests
  Severity: wishlist

  Dear mentors,

  I am looking for a sponsor for my package "taptempo"

 * Package name    : taptempo
   Version         : 1.4.1-1
   Upstream Author : Francois Mazen <francois@mzf.fr>
 * URL             : https://taptempo.tuxfamily.org/
 * License         : GPL-3.0
   Section         : sound

  It builds those binary packages:

    taptempo   - command line tap tempo

  To access further information about this package, please visit the
following URL:

  https://mentors.debian.net/package/taptempo


  Alternatively, one can download the package with dget using this
command:

    dget -x https://mentors.debian.net/debian/pool/main/t/taptempo/tapt
empo_1.4.1-1.dsc

  More information about TapTempo can be obtained from https://taptempo
.tuxfamily.org/.

  Changes since the last upload (1.3.0):

  - Move debian package code to a separate repository.
  - Add unit tests using Catch framework.
  - Add gaming mode with --game switch.
  - Fix Options class when calling multiple times getopt_long
  - Change "check" target to "test" target, to match debian packaging
system.


  Regards,
   Francois Mazen



Le mardi 10 avril 2018 à 05:11 +0000, Lumin a écrit :
> Hi François,
> 
> I'm trying to apply for DD so I'm helping newcomers to review their
> packages. As agreed by Gianfranco, once we've finished polising
> the packaging, he will check and sponsor the package.
> 
> Now your package is in good shape, and there is nothing left to
> be dealt with except for waiting for the sponsorship.
> Let's hope Gianfranco will find a time doing the sponsorship.
> 
> Regards,
> 
> On 9 April 2018 at 17:43, François Mazen <francois@mzf.fr> wrote:
> > Hi Lumin,
> > 
> > I just want to know if I must do something to go on with the
> > integration of TapTempo into Debian.
> > Should I submit again a RFS for the new package version?
> > 
> > Thanks a lot for your help,
> > François
> > 
> > 
> > Le mardi 03 avril 2018 à 23:13 +0200, François Mazen a écrit :
> > > Hi Lumin,
> > > 
> > > upload to mentors done, please check:
> > > https://mentors.debian.net/package/taptempo
> > > 
> > > Regards,
> > > 
> > > François
> > > 
> > > 
> > > 
> > > Le mardi 03 avril 2018 à 06:17 +0000, Lumin a écrit :
> > > > Hi François,
> > > > 
> > > > On 31 March 2018 at 21:59, François Mazen <francois@mzf.fr>
> > > > wrote:
> > > > > 
> > > > > This program is useful to quickly find the tempo of a song.
> > > > > The idea is to type "taptempo" in a terminal, then hit enter
> > > > > key
> > > > > at
> > > > > each beat while hearing a song, and display the tempo.
> > > > > 
> > > > > The targeted people are mainly musicians who need to
> > > > > transcribe
> > > > > music
> > > > > or play the song at the exact original tempo. The typical
> > > > > situation
> > > > > to
> > > > > use this software is when you are in a hurry and you don't
> > > > > have
> > > > > time to
> > > > > launch a big workstation like Ardour or Lmms in order to find
> > > > > the
> > > > > tempo.
> > > > 
> > > > Got it. Thank you for this explanation.
> > > > 
> > > > > 
> > > > > > 8. When you have built the latest version of the modified
> > > > > > package,
> > > > > >     you could run lintian against it:
> > > > > > 
> > > > > >     lintian -EviI --pedantic xxxx.changes
> > > > > > 
> > > > > >     There generally shouldn't be any Error or Warning.
> > > > > 
> > > > > I've fixed all the error and the lintian output should be
> > > > > clean.
> > > > 
> > > > You have done quite a good job making the package in a good
> > > > shape,
> > > > and making the upstream very standard.
> > > > 
> > > > By the way I'm surprised that you have fixed all lintian
> > > > outputs,
> > > > including the pedantic stuff. The pedantic items are only
> > > > optional,
> > > > not what must be fixed. Errors and Warnings should be dealt
> > > > with,
> > > > and some lintian Info can even pass if the maintainer has a
> > > > good
> > > > reason.
> > > > 
> > > > In return everything's shining and in good shape :-)
> > > > 
> > > > > Let me know if it still require more work.
> > > > 
> > > > Nitpicking:
> > > > 
> > > > 1. Please collapse the two lines in changelog into one. They
> > > > refer
> > > > to
> > > >     the same thing.
> > > > 
> > > > -  * Initial debian package.
> > > > -  * Closes: #893306
> > > > +  * Initial debian package. (Closes: #893306)
> > > > 
> > > > 2. there is still an autpkgtest problem:
> > > > 
> > > > autopkgtest [07:01:02]: test version: [-----------------------
> > > > spawn taptempo --version
> > > > couldn't execute "taptempo": no such file or directory
> > > >     while executing
> > > > "spawn taptempo --version"
> > > >     (file
> > > > "/tmp/autopkgtest.C3pEq9/build.uWo/src/debian/tests/version"
> > > > line
> > > > 6)
> > > > autopkgtest [07:01:03]: test version: -----------------------]
> > > > autopkgtest [07:01:03]: test version:  - - - - - - - - - -
> > > > results
> > > > -
> > > > -
> > > > - - - - - - - -
> > > > version              FAIL non-zero exit status 1
> > > > autopkgtest [07:01:03]: test version:  - - - - - - - - - -
> > > > stderr -
> > > > -
> > > > - - - - - - - -
> > > > couldn't execute "taptempo": no such file or directory
> > > >     while executing
> > > > "spawn taptempo --version"
> > > >     (file
> > > > "/tmp/autopkgtest.C3pEq9/build.uWo/src/debian/tests/version"
> > > > line
> > > > 6)
> > > > 
> > > > this can be fixed by the patch. It looks somewhat wired but we
> > > > need
> > > > it.
> > > > 
> > > > --- a/debian/tests/control
> > > > +++ b/debian/tests/control
> > > > @@ -1,2 +1,2 @@
> > > >  Tests: version, help, tempo
> > > > -Depends: expect
> > > > +Depends: expect, taptempo
> > > > 
> > > > The autopkgtest result after patched:
> > > > 
> > > > http://debomatic-amd64.debian.net/distribution#unstable/taptemp
> > > > o/1.
> > > > 3.
> > > > 0-1/autopkgtest
> > > > 
> > > > build result:
> > > > 
> > > > http://debomatic-amd64.debian.net/distribution#unstable/taptemp
> > > > o/1.
> > > > 3.
> > > > 0-1/buildlog
> > > > 
> > > > > Should I update this new package to the mentors website?
> > > > 
> > > > Yes, please fix the two problem mentioned above, and upload to
> > > > mentors.
> > > > 
> > > > Thank you for you contribution to Debian, and have a good day.
> 
> 
>