Re: Bug#917405: RFS: dhcpcd5/7.0.8-1 [ITS]

2019-01-15 Thread Scott Leggett
On 2018-12-27.23:06, Scott Leggett wrote:
> Package: sponsorship-requests
> Severity: normal
> 
> Dear mentors,
> 
> I am looking for a sponsor for my package "dhcpcd5"
> 
>   Package name: dhcpcd5
>   Version : 7.0.8-1
>   Upstream Author : Roy Marples 
>   URL : https://roy.marples.name/projects/dhcpcd
>   License : BSD-2
>   Section : net
> 

Hello,

I'm still looking for a sponsor for dhcpcd5 if anyone is interested in
this package.

-- 
Regards,
Scott Leggett.


signature.asc
Description: PGP signature


Bug#917405: RFS: dhcpcd5/7.0.8-1 [ITS]

2018-12-27 Thread Scott Leggett
Package: sponsorship-requests
Severity: normal

Dear mentors,

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

  Package name: dhcpcd5
  Version : 7.0.8-1
  Upstream Author : Roy Marples 
  URL : https://roy.marples.name/projects/dhcpcd
  License : BSD-2
  Section : net

It builds those binary packages:

  dhcpcd5- DHCPv4, IPv6RA and DHCPv6 client with IPv4LL support

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

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

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

  dget -x 
https://mentors.debian.net/debian/pool/main/d/dhcpcd5/dhcpcd5_7.0.8-1.dsc

Changes since the last upload:

  dhcpcd5 (7.0.8-1) unstable; urgency=low
  
* Remove unused lintian override
* Update watch URL to HTTPS
* Bump standards and compat versions
* Add patch for typo in manpage
* Revert to upstream binary naming
* Update maintainer scripts
* Update debian/copyright
* Update init script for new binary location
* Add systemd service file
* Update package maintainer (Closes: #914070)
* Run wrap-and-sort over the package
* Add VCS fields to control file
* Add hooks for chrony, ntp, systemd-timesyncd, and openntpd (Closes: 
#863462)
* Add autopkgtest-virt-ssh setup script
* Add autopkgtests for NTP hook scripts
* Disable built-in NTP hook script
* Update debian/copyright
  
   -- Scott Leggett   Thu, 27 Dec 2018 22:26:40 +1100

-- 
Regards,
Scott Leggett.


signature.asc
Description: PGP signature


Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-30 Thread Scott Leggett
On 2016-11-29.15:24, Vincent Bernat wrote:

> All good for me. It's uploaded!

Thanks again for your help!

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-29 Thread Scott Leggett
On 2016-11-26.15:48, Vincent Bernat wrote:
>  ❦ 22 novembre 2016 23:55 +1100, Scott Leggett <sc...@sl.id.au> :
> 
> > This is now ready, here are the changes since your last review:
> >
> > * Add automatic migration from -dbg to -dbgsym.
> > * Add users/groups unconditionally to simplify preinst.
> > * Create "quagga" metapackage.
> 
> I would have removed the recommends for quagga-core package as it would
> make more difficult for most people to only keep the daemons they need
> and do not bring anything sensible compared to quagga/quagga-core
> packages. Of course, that's quite subjective.
> 

You're right - I've changed this to make the daemon packages Suggests
rather than Recommends.

> > * Use debhelper integration with dpkg-maintscript-helper.
> > * Remove /etc/default/quagga.
> 
> Should be removed with dpkg-maintscript-helper to alert the user there
> was a change (and its modifications are now ignored).
> 

Actually this file is not in the current quagga package - I added it
during my adoption of the package but have now removed it again.

> You could also remove README.Maintainer if it doesn't contain any
> relevant information. There are also some bits of the README.Debian
> about SNMP that are not up-to-date (some sentences just need to be
> removed).
>

I updated README.Debian to remove references to old Debian releases.
I'd like to keep README.Maintainer though, just because I did actually
find it useful during adoption and I've added some more packaging notes
in there. It doesn't end up in any package anyway - just in the
repository.

> Other than that, all is OK.

Great! New packages are up at https://mentors.debian.net/package/quagga

Changes since your last review:

  * Make daemon packages Suggests rather than Recommends of quagga-core.
  * Remove reference to old Debian release in README.Debian.
  * Avoid VCS files in packages.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-22 Thread Scott Leggett
On 2016-11-21.13:56, Vincent Bernat wrote:
>  ❦ 21 novembre 2016 23:39 +1100, Scott Leggett <sc...@sl.id.au> :
> 
> >> >   * Use systemd .service files rather than init.d scripts (Closes: 
> >> > #678946,
> >> > #805840, #839819, #412483).
> >> 
> >> I am quite OK with that. You don't have time to sort out those
> >> problems. However, this is somewhat controversial.
> >>
> >
> > I'm open to accepting patches / co-maintainership from anyone who wishes
> > to test and use traditional init scripts with quagga. Given the
> > longevity of bugs like #678946, I'm not optimistic of such patches
> > materialising.
> >
> > I personally cannot test such init scripts since all my systems now use
> > systemd, and I can't in good faith include code that I can't test and
> > which has known bugs.
> >
> > If there is another approach you think I could take here, please
> > advise...
> 
> People who want other people to keep init scripts alive are asking to
> just leave them be, even if they are buggy. That's not something I agree
> with, so I am happy that you just removed them. But you could get some
> opposition.
> 

That's okay. I'm happy to work with anyone who wants to improve the
quagga package.

> >>  - default/quagga could be called "quagga.default" and would be
> >>installed automatically as /etc/default/quagga.
> >> 
> >
> > As this is now part of "quagga-core", I still have to install this
> > file manually as far as I can tell. Thanks for the tip on the debhelper
> > way to install it though.
> 
> dh_installinit should install it if you name it quagga-core.default.
> 

Instead I have just removed /etc/default/quagga from the package because
it wasn't much use anymore.

> >>  - the service files seem to use wrong paths. In Debian, chown is in
> >>/bin, not /usr/bin.
> >> 
> >
> > This is interesting.. I ran `which chmod` on my unstable test box, and
> > got "/usr/bin/chmod". /bin is a symlink to /usr/bin in unstable now, and
> > /usr/bin is before /bin in $PATH.
> 
> It's because you have installed usrmerge. I don't think this is like
> this on a standard installation.
>

I don't have usrmerge installed.. this is a fresh unstable install from
a netboot mini.iso.

> > The new package is available again at
> > https://mentors.debian.net/package/quagga
> 
> I'll review the changes for tomorrow.

This is now ready, here are the changes since your last review:

* Add automatic migration from -dbg to -dbgsym.
* Add users/groups unconditionally to simplify preinst.
* Create "quagga" metapackage.
* Remove informational lintian overrides.
* Rename quagga-dev package to libquagga-dev.
* Remove non-standard maintainer script debugging code.
* Fix paths in .service file patch.
* Use dpkg-statoverride in preference to manual chmod/chown.
* Add Vcs-* fields to control file.
* Update quagga.NEWS.
* Update changelog for 1.1.0-1 release
* Use debhelper integration with dpkg-maintscript-helper.
* Remove /etc/default/quagga.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-22 Thread Scott Leggett
On 2016-11-21.08:11, The Wanderer wrote:
> On 2016-11-21 at 07:56, Vincent Bernat wrote:
> 
> > ❦ 21 novembre 2016 23:39 +1100, Scott Leggett <sc...@sl.id.au> :
> >> I personally cannot test such init scripts since all my systems now
> >> use systemd, and I can't in good faith include code that I can't
> >> test and which has known bugs.
> 
> (That's an interesting usage; I would have used "good conscience". I'll
> have to keep an eye out for this one.)
> 

Perhaps that's what I meant.. you're probably right :)

> >> If there is another approach you think I could take here, please 
> >> advise...
> > 
> > People who want other people to keep init scripts alive are asking
> > to just leave them be, even if they are buggy. That's not something I
> > agree with, so I am happy that you just removed them. But you could
> > get some opposition.
> 
> My own position is roughly that init scripts should be left in place
> unless the maintainer sincerely believes that they are not just buggy,
> but so badly broken that either they provide no functionality at all, or
> to have the functionality which they provide is _worse_ than to not have it.
> 
> In other words: do you, as the package maintainer, believe that a
> reasonable person would prefer having no init-system functionality for
> this package at all over having what these init scripts provide?
> 

I think quagga is a bit of a special case in this regard:

The Debian package uses a Debian-specific "quagga" init script which
takes subcommand arguments to the usual start/stop/restart for managing
the multiple routing daemons. It also implements a priority system where
groups of daemons can be stopped and started with one command. All of
this init script behaviour is configured using two configuration files -
/etc/quagga/{debian.conf,daemons}. Yes, two configuration files to
configure the behaviour of this one init script _in addition_ to the
rather more common /etc/default/quagga.

Needless to say this complexity is inconsistent any other daemon within
Debian and with upstream (which implements a single init/service per
routing daemon rather than one that controls them all). It's also rather
error prone as illustrated by the number of bugs closed by switching to
systemd.

The fact that the init script is monolithic also leads to problems when
trying to keep it along with the new .service files because there is no
equivalent quagga.service. To keep /etc/init.d/quagga, I'd have to mask
quagga.service in systemd, but that can always be unmasked by the
administrator and cause issues with multiple services trying to start
the same daemons.

If all that wasn't enough, upstream also ship a "watchquagga" service
whose sole purpose is to do the job that systemd now does for all
daemons - restart services when they crash or hang. With systemd,
"watchquagga" can be removed completely.

I hope this doesn't come across as too critical of the existing init
scripts - they've clearly served their purpose well. I just think that
there's a better option now.

So my answer is: I believe that a reasonable person coming to quagga for
the first time would rather have an interface consistent with all their
other services, instead of having to learn the intricacies and quirks
specific to the quagga init scripts. However that person will have very
different perspective to the Debian old-timer who was running quagga for
a decade using the existing system and now has to learn something new
(new, but consistent with every other service in a default Debian
installation).

As I said before, I'm open to keeping the existing init scripts. But
only if a) someone else steps up to do the testing and ongoing
maintenance of them, and b) the experience under systemd is not
diminished.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Bug#845018: Fwd: Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-22 Thread Scott Leggett
- Forwarded message from Vincent Bernat <ber...@debian.org> -

> Date: Mon, 21 Nov 2016 13:56:00 +0100
> From: Vincent Bernat <ber...@debian.org>
> To: Scott Leggett <sc...@sl.id.au>
> Cc: debian-mentors@lists.debian.org
> Subject: Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons
> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux)
> 
>  ❦ 21 novembre 2016 23:39 +1100, Scott Leggett <sc...@sl.id.au> :
> 
> >> >   * Use systemd .service files rather than init.d scripts (Closes: 
> >> > #678946,
> >> > #805840, #839819, #412483).
> >> 
> >> I am quite OK with that. You don't have time to sort out those
> >> problems. However, this is somewhat controversial.
> >>
> >
> > I'm open to accepting patches / co-maintainership from anyone who wishes
> > to test and use traditional init scripts with quagga. Given the
> > longevity of bugs like #678946, I'm not optimistic of such patches
> > materialising.
> >
> > I personally cannot test such init scripts since all my systems now use
> > systemd, and I can't in good faith include code that I can't test and
> > which has known bugs.
> >
> > If there is another approach you think I could take here, please
> > advise...
> 
> People who want other people to keep init scripts alive are asking to
> just leave them be, even if they are buggy. That's not something I agree
> with, so I am happy that you just removed them. But you could get some
> opposition.
> 
> >>  - default/quagga could be called "quagga.default" and would be
> >>installed automatically as /etc/default/quagga.
> >> 
> >
> > As this is now part of "quagga-core", I still have to install this
> > file manually as far as I can tell. Thanks for the tip on the debhelper
> > way to install it though.
> 
> dh_installinit should install it if you name it quagga-core.default.
> 
> >>  - the service files seem to use wrong paths. In Debian, chown is in
> >>/bin, not /usr/bin.
> >> 
> >
> > This is interesting.. I ran `which chmod` on my unstable test box, and
> > got "/usr/bin/chmod". /bin is a symlink to /usr/bin in unstable now, and
> > /usr/bin is before /bin in $PATH.
> 
> It's because you have installed usrmerge. I don't think this is like
> this on a standard installation.
> 
> > The new package is available again at
> > https://mentors.debian.net/package/quagga
> 
> I'll review the changes for tomorrow.
> -- 
> Debian package sponsoring guidelines:
>  https://vincent.bernat.im/en/debian-package-sponsoring.html



- End forwarded message -

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Bug#845018: Fwd: Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-22 Thread Scott Leggett
Oops, forgot to CC the bug report..

- Forwarded message from Scott Leggett <sc...@sl.id.au> -

> Date: Mon, 21 Nov 2016 23:39:31 +1100
> From: Scott Leggett <sc...@sl.id.au>
> To: debian-mentors@lists.debian.org
> Subject: Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons
> 
> Hi Vincent,
> 
> Thanks for taking the time to do another thorough review of another of
> my packaging efforts! :)
> 
> On 2016-11-19.19:47, Vincent Bernat wrote:
> >  ❦ 20 novembre 2016 01:43 +1100, Scott Leggett <sc...@sl.id.au> :
> > 
> > >   * Rely on automatic -dbgsym package rather than deprecated -dbg.
> > 
> > You need to make a migration for people having a -dbg. This is done with
> > --dbgsym-migration argument of dh_strip.
> >
> 
> Done.
> 
> > >   * Add quagga user to quaggavty group in preinst.
> > 
> > I suppose this was in the original packaging, but this is better to use
> > adduser/addgroup unconditionally. Both commands will check if the
> > user/group already exists with the appropriate parameters. Notably, if
> > the user/group is not a system one, it will trigger an error.
> >
> 
> Thanks, that simplifies the script nicely. Done.
> 
> > >   * Use systemd .service files rather than init.d scripts (Closes: 
> > > #678946,
> > > #805840, #839819, #412483).
> > 
> > I am quite OK with that. You don't have time to sort out those
> > problems. However, this is somewhat controversial.
> >
> 
> I'm open to accepting patches / co-maintainership from anyone who wishes
> to test and use traditional init scripts with quagga. Given the
> longevity of bugs like #678946, I'm not optimistic of such patches
> materialising.
> 
> I personally cannot test such init scripts since all my systems now use
> systemd, and I can't in good faith include code that I can't test and
> which has known bugs.
> 
> If there is another approach you think I could take here, please
> advise...
> 
> > >   * Split quagga package out into multiple packages (Closes: #705306).
> > 
> > People upgrading will suddenly have all the daemons missing. You could
> > introduce a "base" or "minimal" package that wouldn't depend on the
> > daemons (and the daemons would depend on them). The "quagga" package
> > would still depend on all the daemons. However, the NEWS entry could be
> > sufficient too.
> > 
> 
> That is a much better solution, thank you for the suggestion! I have
> updated the package as you suggest: there is now a "quagga" metapackage,
> and a "quagga-core", which all the routing daemons depend on.
> 
> > 
> > Also :
> > 
> >  - the -dev package should be libquagga-dev
> > 
> 
> Fixed.
> 
> >  - default/quagga could be called "quagga.default" and would be
> >installed automatically as /etc/default/quagga.
> > 
> 
> As this is now part of "quagga-core", I still have to install this
> file manually as far as I can tell. Thanks for the tip on the debhelper
> way to install it though.
> 
> >  - the service files seem to use wrong paths. In Debian, chown is in
> >/bin, not /usr/bin.
> > 
> 
> This is interesting.. I ran `which chmod` on my unstable test box, and
> got "/usr/bin/chmod". /bin is a symlink to /usr/bin in unstable now, and
> /usr/bin is before /bin in $PATH.
> 
> Either way, I've changed the paths in the .service files to /bin/*.
> 
> >  - if you think that maintaining all those lintian overrides is too much
> >effort, I think that people usually don't override "informational"
> >messages.
> > 
> 
> Okay, I've removed the trivial overrides.
> 
> >  - in maintainers script, I would drop the non-standard debug stuff at
> >the top of each of them
> > 
> 
> Done.
> 
> >  - in postinst, changing permissions in /etc is frowned upon since it
> >could undo what a user has done. You should use dpkg-statoverride
> >which gives the user a chance to implement its own policy.
> 
> Thanks for introducing me to another interesting dpkg-* tool. Fixed.
> 
> The new package is available again at
> https://mentors.debian.net/package/quagga
> 
> -- 
> Regards,
> Scott.



- End forwarded message -

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-22 Thread Scott Leggett
On 2016-11-21.13:56, Vincent Bernat wrote:
>  ❦ 21 novembre 2016 23:39 +1100, Scott Leggett <sc...@sl.id.au> :
> 
> I'll review the changes for tomorrow.

Hi Vincent,

Sorry for this.. there are a couple more changes I'd like to make. If
you haven't begun review already, please wait.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Bug#845018: RFS: quagga/1.1.0-1 [ITA] -- network routing daemons

2016-11-19 Thread Scott Leggett
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "quagga":

 Package name: quagga
 Version : 1.1.0-1
 Upstream Author : Quagga Maintainers <maintain...@quagga.net>
 URL : http://www.quagga.net
 License : GPL-2+
 Section : net

It builds those binary packages:

 libquagga0 - network routing daemons (libraries)
 quagga - network routing daemons
 quagga-bgpd - BGP4/BGP4+ routing daemon
 quagga-dev - network routing daemons (development files)
 quagga-doc - network routing daemons (documentation)
 quagga-isisd - IS-IS routing daemon
 quagga-ospf6d - OSPF6 routing daemon
 quagga-ospfd - OSPF routing daemon
 quagga-pimd - PIM routing daemon
 quagga-ripd - RIPv1 routing daemon
 quagga-ripngd - RIPng routing daemon

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

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

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

 dget -x https://mentors.debian.net/debian/pool/main/q/quagga/quagga_1.1.0-1.dsc

More information about quagga can be obtained from
http://www.quagga.net.

Changes since the last upload:

  * New upstream release (Closes: #774760, #516226, #830515)
  * Import packaging from the last debian release 1.0.20160315-3.
  * Remove dump_fix.patch applied upstream.
  * Remove patch which is no longer relevant.
  * Remove patch for CVE-2016-1245 fixed upstream.
  * Rely on automatic -dbgsym package rather than deprecated -dbg.
  * Remove deprecated XS-testsuite header in debian/control.
  * Remove template comment from debian/watch.
  * Add patch to fix spelling and grammar errors.
  * Register quagga-doc with doc-base.
  * Add patch to fix ospfclient(8) manpage numbering.
  * Added patch to avoid debug print statements on vtysh startup.
  * Adopt package, set myself as maintainer (Closes: #836418).
  * Add quagga user to quaggavty group in preinst.
  * Drop patch for Debian pager default in vtysh (Closes: #788243).
  * Update debian/copyright.
  * Bump compat level to 10 (Closes: #534833).
  * Use systemd .service files rather than init.d scripts (Closes: #678946,
#805840, #839819, #412483).
  * Split quagga package out into multiple packages (Closes: #705306).
  * Remove debconf question which left packages in inconsistent state.
  * Add patch for manpage versioning.
  * Update README.Debian, README.Maintainer.
  * Removed patch on vtysh.conf.
  * Add NEWS.Debian about the major changes to the package.

--
Regards,
Scott Leggett.



Re: debexpo bug on upload

2016-11-12 Thread Scott Leggett
On 2016-11-12.23:59, gustavo panizzo (gfa) wrote:
> On Sun, Nov 13, 2016 at 01:20:46AM +1100, Scott Leggett wrote:
>  
> > Unfortunately, I spoke too soon.. the .buildinfo upload failed with 403
> > Forbidden again :(
> Remove the .buildinfo line from the .changes file, then sign and upload.
> 
> mentors.d.n does not accept .buildinfo files yet
> 
> > 
> > Would you be able to remove that file too? :)
> 
> AFAIK, there is a cronjob that removes the incomplete uploads after some time

Okay, I'll give this a shot. Thanks for your help.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: debexpo bug on upload

2016-11-12 Thread Scott Leggett
On 2016-11-12.14:25, Gianfranco Costamagna wrote:
> Hi,
> 
> >The package is "quagga".
> 
> please wait for mentors to be fixed and open an RFS bug.
> or upload somewhere else, but really open a bug.
> 
> otherwise probably nobody will look at it
> (and there is already a quagga in the archive)

Yes - I'm adopting it!

However I need to upload it somewhere first before I can file a RFS.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: debexpo bug on upload

2016-11-12 Thread Scott Leggett
On 2016-11-13.01:11, Scott Leggett wrote:
> On 2016-11-12.15:02, Nicolas Dandrimont wrote:
> > * Scott Leggett <sc...@sl.id.au> [2016-11-13 00:06:39 +1100]:
> > 
> > > Hi,
> > > 
> > > I've run into a bug[0] with debexpo making it impossible to upload my
> > > package. I suspect there's a partially uploaded package sitting in the
> > > system somewhere.. does anyone on this list have admin access to
> > > mentors.debian.net that could check this for me?
> > > 
> > > Failing that, is there an alternative way to get the package reviewed?
> > > 
> > > The package is "quagga".
> > 
> > The package has been moved out of the way.
> 
> Great, thanks for your help!

Unfortunately, I spoke too soon.. the .buildinfo upload failed with 403
Forbidden again :(

Would you be able to remove that file too? :)

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: debexpo bug on upload

2016-11-12 Thread Scott Leggett
On 2016-11-12.15:02, Nicolas Dandrimont wrote:
> * Scott Leggett <sc...@sl.id.au> [2016-11-13 00:06:39 +1100]:
> 
> > Hi,
> > 
> > I've run into a bug[0] with debexpo making it impossible to upload my
> > package. I suspect there's a partially uploaded package sitting in the
> > system somewhere.. does anyone on this list have admin access to
> > mentors.debian.net that could check this for me?
> > 
> > Failing that, is there an alternative way to get the package reviewed?
> > 
> > The package is "quagga".
> 
> The package has been moved out of the way.

Great, thanks for your help!

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


debexpo bug on upload

2016-11-12 Thread Scott Leggett
Hi,

I've run into a bug[0] with debexpo making it impossible to upload my
package. I suspect there's a partially uploaded package sitting in the
system somewhere.. does anyone on this list have admin access to
mentors.debian.net that could check this for me?

Failing that, is there an alternative way to get the package reviewed?

The package is "quagga".

[0] https://alioth.debian.org/tracker/index.php?func=detail=315172

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Re: When to use "Replaces"?

2016-05-31 Thread Scott Leggett
On 2016-05-30.21:22, gregor herrmann wrote:
> On Mon, 30 May 2016 20:25:56 +0200, Adam Borowski wrote:
> 
> > >   c) Something else ...?
> > As both unclutters provide generally the same functionality, you may also
> > use the "alternatives" method.  
> 
> My impression is that the "new" version has different command line
> options. So calls to unclutter with the old options in
> .xsession/.xinit would break if /usr/bin/unclutter points to the new
> implementation via alternatives.
> 
> I think I'd go for renaming here.

Using the alternatives system is an interesting idea. Yes, the "new"
version is *not* a drop-in replacement - while the functionality is the
same, accepted arguments are not totally identical. However the
description and example given in the policy manual of different "vi"
clones seems to match the "unclutter" situation quite well.

Am I correct in understanding that using alternatives requires
co-operation between the two packages, "unclutter" and
"unclutter-fixes"? If that's the case, I'll approach the "unclutter"
maintainer, and if they disagree on using alternatives, will just rename
my package.

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


When to use "Replaces"?

2016-05-30 Thread Scott Leggett
Hi all,

I've just filed an ITP for "unclutter-xfixes"[0]. This package is a
rewrite of the "unclutter" package. It provides the same functionality
(hiding your mouse pointer after some period of inactivity), but uses a
different API which may avoid some of the issues seen in "unclutter".

Upstream's Makefile installs the binary and manpage as "unclutter" -
it's clearly intended to be used in place of the original package.

The policy manual describes using the "Replaces" (and "Breaks") control
fields only to handle a package structure change on upgrade (e.g. split
"foo" into "foo" and "foo-data")[1], but in that case the packages are
co-operatively managed by the same maintainer.

In the case of "unclutter" vs. "unclutter-fixes", it is less clear to me
whether I should use "Replaces", because "unclutter-xfixes" is not a
direct upgrade to "unclutter" (and I guess users may want to install
both at the same time??).

So should I:

  a) Patch the Makefile to rename the binary "unclutter-xfixes", with
  the downside of diverging from Upstream;

  b) Go with Upstream in naming the binary "unclutter", and make the
  "unclutter-xfixes" package Break/Replace "unclutter", with the
  downside of not being able to install both at the same time; or

  c) Something else ...?

I'm thinking a) is the best option because it avoids stomping on the
existing unclutter namespace, but just wanted a second opinion.

[0] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=825809
[1] https://www.debian.org/doc/debian-policy/ch-relationships.html#s-replaces

-- 
Regards,
Scott.


signature.asc
Description: Digital signature


Bug#729208: RFS: pasystray/0.4.0-1 [ITP]

2013-11-10 Thread Scott Leggett
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package pasystray

 Package name: pasystray
 Version : 0.4.0-1
 Upstream Author : Christoph Gysin christoph.gy...@gmail.com
 URL : https://github.com/christophgysin/pasystray
 License : LGPL-2.1+
 Section : sound

It builds those binary packages:

  pasystray  - PulseAudio controller for the system tray

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

  http://mentors.debian.net/package/pasystray

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

  dget -x
http://mentors.debian.net/debian/pool/main/p/pasystray/pasystray_0.4.0-1.dsc

More information about pasystray can be obtained from:
https://launchpad.net/~christoph-gysin/+archive/pasystray

Changes since the last upload:

pasystray (0.4.0-1) unstable; urgency=low

  * Initial release (Closes: #729205)

 -- Scott Leggett sc...@sl.id.au  Sun, 10 Nov 2013 18:00:59 +1100

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#729208: RFS: pasystray/0.4.0-1 [ITP]

2013-11-10 Thread Scott Leggett
On 10/11/13 20:10, Vincent Bernat wrote:
 
 Hi Scott!

Hi Vincent.

 
 Why do you explictely depends on libraries instead of relying on
 ${shlibs:Depends}? I suppose there is a good reason for that but I am
 lazy to test. I didn't catch any other issue.
 

No, there is no good reason for that. Just my misunderstanding of what
${shlibs:Depends} is for!

I've removed the redundant dependencies and re-uploaded the package to
mentors.debian.net if you would like to have another look :)

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#728587: RFS: compton/0.1~beta2-1

2013-11-03 Thread Scott Leggett
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package compton

 Package name: compton
 Version : 0.1~beta2-1
 Upstream Author : Richard Grenville pyx...@gmail.com
 URL : https://github.com/chjj/compton
 License : Expat
 Section : x11

It builds those binary packages:

  compton- Compositor for X11, based on xcompmgr

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

http://mentors.debian.net/package/compton

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

  dget -x
http://mentors.debian.net/debian/pool/main/c/compton/compton_0.1~beta2-1.dsc

More information about compton can be obtained from
https://wiki.archlinux.org/index.php/Compton

Changes since the last upload:

compton (0.1~beta2-1) unstable; urgency=low

  * New upstream release, 0.1~beta2.
  * Drop unnecessary dependency on hardening-wrapper.
  * Add libxinerama-dev dependency due to upstream changes.
  * Change licence tag to Expat.
  * Update patches.

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#662955: RFS: rubyripper/0.6.2-1 [ITP]

2013-08-23 Thread Scott Leggett
On 23/08/13 05:01, Paul Wise wrote:
 
 The SVG source for rubyripper.png is missing from the package.

 This is not included upstream either. Does Debian have a policy of not
 accepting rasterised icons?
 
 ftpmasters have a policy of requiring the preferred form for
 modification be available and that it be possible to build results
 from it on Debian (not necessarily that they be built at build time).
 They may or may not notice this issue though.
 
 http://www.debian.org/social_contract#guidelines
 http://ftp-master.debian.org/REJECT-FAQ.html

I'll see what upstream have, but I suspect that if the SVG was
available, it would be in the tree or on the website.

 
 No, it doesn't look safe to me either (though I hadn't noticed
 previously). See patch 6.
 
 That is a quite ugly fix that may or may not actually fix the issue. I
 would suggest that command be an array constructed in the usual way
 for ruby and passed to popen, instead of just constructing a string
 and then splitting it based on a regex.

You're right. I leant away from a proper patch initially due to not
wanting to make invasive changes. See the latest patch though - now
constructing the array of arguments properly.

 
 There is no changelog included upstream.
 
 Perhaps they would be willing to include a ChangeLog generated from
 the git log or a NEWS file.
 

I'll file another upstream bug about this.

Thanks again.

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#701693: RFS: compton/0.1~beta1-1 [ITP] -- X compositor

2013-08-21 Thread Scott Leggett
On 21/08/13 05:15, Vincent Bernat wrote:
 
 Hi Scott!

Hi Vincent, thanks for reviewing my package!

 
 The package seems fine for me. I just noticed those minor details:
 
  - debian/changelog: the Imported Upstream version  line is
useless, please remove it.

Fixed.

  - debian/control: do not start the short description with a capital.

Fixed.

  - debian/rules: debhelper 9 is doing hardening by default now, so you
don't need to depend on hardening-wrapper and set the option in
debian/rules. Maybe you did try and this doesn't work?
 

I'm not sure if debhelper was doing hardening by default when I
originally packaged compton - hence the wrapper dependency.

Removing the dependency didn't work initially due to the Makefile
ignoring CPPFLAGS. I've patched the issue, and hardening is now working
without the wrapper dependency.

The package has been re-uploaded to mentors if you would like to take a
look :)

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#720330: RFS: spout/1.4-2

2013-08-20 Thread Scott Leggett
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package spout

 Package name: spout
 Version : 1.4-2
 Upstream Author : Nick White nick.wh...@njw.me.uk
 URL : http://njw.me.uk/spout/
 License : MIT
 Section : games

It builds those binary packages:

  spout - Tiny abstract black and white 2D cave-shooter

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

http://mentors.debian.net/package/spout

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

  dget -x http://mentors.debian.net/debian/pool/main/s/spout/spout_1.4-2.dsc

Changes since the last upload:

spout (1.4-2) unstable; urgency=low

  * Added patch for LFS.
  * Fix lintian tag data.tar.xz-member-without-dpkg-pre-depends.
  * Add 'Keywords' field to .desktop file.
  * Upload to unstable.

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#662955: RFS: rubyripper/0.6.2+git-v0.6.2-6-g165107f-2013-01-02-1 [ITP] -- CD ripper

2013-08-20 Thread Scott Leggett
Dear mentors,

I am looking for a sponsor for my package rubyripper

 Package name: rubyripper
 Version : 0.6.2+git-v0.6.2-6-g165107f-2013-01-02-1
 Upstream Author : Bouke Woudstra boukewouds...@gmail.com
 URL : https://code.google.com/p/rubyripper/
 License : GPL-3.0+
 Section : sound

It builds those binary packages:

  rubyripper-cli - Error-correcting CD audio extractor (CD ripper). (CLI)
  rubyripper-gtk2 - Error-correcting CD audio extractor (CD ripper). (GUI)

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

http://mentors.debian.net/package/rubyripper

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

  dget -x
http://mentors.debian.net/debian/pool/main/r/rubyripper/rubyripper_0.6.2+git-v0.6.2-6-g165107f-2013-01-02-1.dsc

More information about rubyripper can be obtained from
http://wiki.hydrogenaudio.org/index.php?title=Rubyripper

Changes since the last upload:

rubyripper (0.6.2+git-v0.6.2-6-g165107f-2013-01-02-1) unstable; urgency=low

  * Initial release (Closes: #463584).
  * Imported Upstream version 0.6.2+git-v0.6.2-6-g165107f-2013-01-02
  * Patch: Don't manually update icon caches during build.
  * Patch: Remove obsolete translations.
  * Patch: Set UTF8 encoding for language translations.
  * Patch: Don't unnecessarily update translation during build.
  * Patch: Use the right name for the normalize-audio binary.
  * Patch: Directly invoke the encoding process.
  * Patches: Fix lint in translations.
  * Patch: Add 'Keywords' to .desktop file.

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#701693: RFS: compton/0.1~beta1-1 [ITP] -- X compositor

2013-08-20 Thread Scott Leggett
Dear mentors,

I am looking for a sponsor for my package compton

 Package name: compton
 Version : 0.1~beta1-1
 Upstream Author : Richard Grenville pyx...@gmail.com
 URL : https://github.com/chjj/compton
 License : X/MIT
 Section : x11

It builds those binary packages:

  compton- Compositor for X11, based on xcompmgr

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

http://mentors.debian.net/package/compton

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

  dget -x
http://mentors.debian.net/debian/pool/main/c/compton/compton_0.1~beta1-1.dsc

More information about compton can be obtained from
https://wiki.archlinux.org/index.php/Compton

Changes since the last upload:

compton (0.1~beta1-1) unstable; urgency=low

  * Imported Upstream version 0.1~beta1
  * Initial release (Closes: #679551)

-- 
Regards,
Scott Leggett.



signature.asc
Description: OpenPGP digital signature


Bug#662955: RFS: rubyripper/0.6.2-1 [ITP]

2013-08-18 Thread Scott Leggett
Hi Paul,

I've finally had time to sit down to try and fix all the issues with
this package. Hopefully I've had some success, but it would be great if
you could cast your eye over my fixes :)

The package is available here:

http://mentors.debian.net/package/rubyripper

On 04/03/13 00:33, Paul Wise wrote:
 Please get the patches include upstream and or change them to be
 acceptable upstream.

I have 16 patches against the upstream stable tree (though half are
translation lint), and will address each of these:

1. Debian build-system specific.
2. Submitted, not yet accepted upstream.
3. Submitted, not yet accepted upstream.
4. Fixed in upstream master (but not in stable branch).
5. Fixed in upstream master (but not in stable branch).
6. Fixed in upstream master (but not in stable branch).
7 - 15. Translation fixes submitted, not yet accepted upstream.
16. Submitted, not yet accepted upstream.


 Please use this watch file instead:

 version=3

opts=uversionmangle=s/(\d)[_\.\-\+]?((RC|rc|pre|dev|beta|alpha|b|a)\d*)$/$1~$2/
 \
  http://code.google.com/p/rubyripper/downloads/list?can=1 \
  .*/rubyripper-(\d\..+)\.(?:zip|tgz|tbz|txz|(?:tar\.(?:gz|bz2|xz)))

This didn't work for me, but I used the Google Code watch example from
the wiki[0]. Actually, the example has a bug where it looks for two
adjacent periods in the file name. I note that you were the last editor
of the page - maybe you could fix the example? (I don't have edit access)

... .*/project-(\d\S*)\.\.(?:zip|tgz|tbz|txz|(?:tar\.(?:gz|bz2|xz)))
 ^
 only one period required here


 Please call dh_auto_configure in override_dh_auto_configure instead of
 ./configure, this will allow you to drop --configure.

Fixed.


 There doesn't seem to be a need for the rubyripper.png symlink, what
 is that needed for?

Fixed. I'm not sure why I did that in the first place.


 The comments in the manual page aren't needed.

Fixed.


 Please get the manual page included upstream.

Submitted, not yet accepted upstream.


 Please run wrap-and-sort -sa so diffs of debian/* are more human-readable.

Fixed. Thanks for introducing me to this tool!


 The package descriptions are almost duplicates of each other.

Is this an issue? I modelled the descriptions on other packages which
have both a command-line and graphical interface, such as vim/vim-gtk.


 The SVG source for rubyripper.png is missing from the package.

This is not included upstream either. Does Debian have a policy of not
accepting rasterised icons?


 The upstream README contains install and MacOS information, please ask
 upstream to split that out into separate files since it isn't useful
 for Debian users.

When I get the chance, I'll split this information out and submit a patch.


 Several of the files in the source are executable, but do not need to be.

Fixed.


 I'm not familiar with ruby, but the way it uses popen doesn't look
safe to me.

No, it doesn't look safe to me either (though I hadn't noticed
previously). See patch 6.

It's been fixed differently upstream, but as part of extensive
refactoring which would be too difficult to try to pull into a patch
against the stable version.


 Automatic checks:

 https://wiki.debian.org/HowToPackageForDebian#Check_points_for_any_package

Thanks for this link, I wasn't aware of it before.


 ruby configure --update-lang:

 lots of obsolete msgid warnings

See patch 2.


 lintian:

 P: rubyripper-gtk2: no-upstream-changelog
 P: rubyripper-cli: no-upstream-changelog

There is no changelog included upstream.


 cme check dpkg-control:

 Says that the short descriptions for the packages are too long.

Fixed.


 msgfmt:

 many empty msgstr warnings

I'll file bugs upstream for these; I'm sure the results would be tragic
if I tried to fix these using google translate :P


 POFileChecker:

 lots of warnings:
 missing :
 missing .
 missing ...
 extra \n


See patches 7 - 15.

Thanks again for the review. Discovering all these techniques for
de-linting packages has been very educational!

Regards,
Scott Leggett.


[0] https://wiki.debian.org/debian/watch/
-- 
Regards,
Scott Leggett.




signature.asc
Description: OpenPGP digital signature


Bug#701693: RFS: compton/0.0.1+git-2182505-2013-02-05-1 [ITP]

2013-03-10 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 10/03/13 17:30, Vincent Cheng wrote:
 On Sat, Mar 9, 2013 at 9:00 PM, Scott Leggett sc...@sl.id.au
 wrote: snip
 * Override dh_auto_clean to quiet verbose build warnings.
 
 You have an empty override target, which is going to cause
 multiple builds in a row (e.g. debuild  debuild) to FTBFS. If
 these verbose build warnings are causing the clean target to fail,
 please fix them.
 
 Regards, Vincent
 
 

Hi Vincent,

Thanks for your review, and for spotting the issue. No, the the verbose
build warnings on the initial invocation of dh_auto_clean do not cause
the target to fail.

My use of pdebuild rather than debuild hid the problem with overriding
dh_auto_clean. This also caused the build warnings in the first place,
regarding missing libraries in the initial, empty, fakeroot environment.

Later in the build process, when build-deps have been installed and the
package is actually built, no warnings are emitted and the package
builds successfully.

Therefore I have removed the override and re-uploaded the package. It
now builds entirely without overrides or patches.


compton (0.0.1+git-b3652f6-2013-03-03-1) experimental; urgency=low

  * Imported Upstream version 0.0.1+git-b3652f6-2013-03-03
  * Initial release (Closes: #679551)

 -- Scott Leggett sc...@sl.id.au  Mon, 11 Mar 2013 00:40:43 +1100


- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJRPRztAAoJEHlzKPr+55fVSCgP/2OqAI4unicsCB5EEFRhrj//
9YLW+vA3QghMtRKFXDVeqyqQnT01s4kkHZUA5xvxEB5r8a9AelaLaPABgsANi6qT
vCebtfl1Uef9EzXE/UMqJlg9wYO8vQC/eJkm+5fCMqb6N3KFyOipXE7CLA4FAHYA
J+aaXtWTgqUrdgtp8QXp0CR046amGMz6XlG+qkd5G/TVPGaK48ki2BMHWwcCh8z7
WFyUwSyU4UgeugeZl4uY0Cw0o/aa2Mi8zwOokCwcPLaeRuH2mQA/KzRbLXp6NSuz
tN47ZoKpTdfous/j0rYI2epjk1AiHbGeCDBQzv3YrAYZPi6wYih9Uf90iCZjwLjD
KtoBr6f8AqrXjcHTVsCQWX6h1k+xCgQoq8tmZTVE+WHvE3u88BEgYi6/ChygOmLl
6EAp6BkPy9gFQzBDmESwxxgnD1BI1DJ0w0qFdLnUVJPU3X6h/bcBt7nTqAvwBiCO
YQsMAsYcWFnFQ65rYycVvyWP7XnrYtujqfBMKDPt/dJfOI1XteuJX33HT6V0PaLA
YSACBTrcB119xjmNDpuMiz8thkiEw7QYPyfUnKhI9NOwtPHGP+F+wnW7e5h//G44
+jxaRb4jjihqDiv5zmriya3VKyUGBus4kzeSFocLW0/fGxqGY5e798xpvxs9NnsB
+Gv2SO1z5nZfVVR0VGrc
=nMkX
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/513d1cfa.3020...@sl.id.au



Bug#701693: RFS: compton/0.0.1+git-2182505-2013-02-05-1 [ITP]

2013-03-09 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 04/03/13 22:25, Paul Wise wrote:
 Sounds like you should work with upstream to merge compton fixes
 back into xcompmgr, which is already in Debian.
 

Hi Paul,

Thanks for the suggestion, but compton provides many added features to
xcompmgr as well as bug fixes and IMO has diverged too far from
xcompmgr to make merging desirable or practical.

For example, compton provides a dbus interface and regex matching of
window attributes - features far outside the scope of xcompmgr as a
sample compositing manager for X servers.

I've uploaded a new package based on the latest upstream snapshot. It
can be found at:

http://mentors.debian.net/package/compton

It can also be downloaded using:

dget -x
http://mentors.debian.net/debian/pool/main/c/compton/compton_0.0.1+git-b3652f6-2013-03-03-1.dsc

Initial release changelog:

compton (0.0.1+git-b3652f6-2013-03-03-1) experimental; urgency=low

  * Imported Upstream version 0.0.1+git-b3652f6-2013-03-03
  * Override dh_auto_clean to quiet verbose build warnings.
  * Initial release (Closes: #679551)

 -- Scott Leggett sc...@sl.id.au  Sun, 10 Mar 2013 15:08:58 +1100

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJRPBNRAAoJEHlzKPr+55fVRq4P/0Pl17N7e4neHZDVFJlUMu5R
arh8eH083FBtChspFLNxte8x+i2mBny3N7x5a2hR+ZhoR7T+jI1LSv+WB47LQXiv
XfF5m1J7CUeKHXdz2X6tQU256FQ8cm3yanbIIwcIozQKEX6+ytAnASJ7QS/YPley
JgGrKHrLqHTcNa/e/qwNRqJnmJDGvZV1CNpCHSsjMqtA8sCqWK7gFZU/oZOux56N
Ric1hAFMZzg5TY55uPWBOXLufq4Db/lF/3HS/vBWNzk6d1I1xzhHZBBboby3xZms
DLd+jhM5mce55gsAZ08as3Y4bUs+6DFAVl5VUa0EjTPJiMniUoxGtby2DP0fodUP
6AOEQs21vbEC80+BmaA2Itx+WTXeuxYzNtszR1Cc4PWg7huq5lwS0bdJ5QNOj/V4
blGbc2OMorwBuP+t4Lrk6kxifI2H6KTuNhMGTPkRE3S/FEFVwQIHg+BlmekJmnab
LMXqV0ZTv+TWWmNdTkP/P7EcA+6nuZDUoZMS2LNAYNxWGVV6dLK9/ZZDW5r3fMPZ
yaXlutZAkxGfoNqBHh2PFawCZpOK9RrfbTfIBTSa/26COSXkzx4RMz8t7+IZKwPz
lJgPweYZEJd0nHfHYMkztWJdW/oVi+2PwgYYXIK3UjHXqvlmfUdpzNji2yOlS710
OCCdXbMRcombWlUo6dhX
=5IBy
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/513c1351.3030...@sl.id.au



Bug#662955: RFS: rubyripper/0.6.2-1 [ITP]

2013-03-04 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi Paul,

On 04/03/13 00:33, Paul Wise wrote:
 You might want to join #debian-au :)
 

I didn't know this existed, cool!

 Have you considered joining the multimedia team?
 
 https://wiki.debian.org/Teams/Multimedia

No I haven't, I'll take a look.

 
 I don't intend to sponsor this package, but here is a review: ...

:-/

Thank you for taking the time to do such a detailed review. It looks
like you've identified quite some issues with the current package.

I have pushed a few of those patches upstream; it's just that upstream
is (slowly) getting ready for releasing 0.7.0 and the build is
currently broken, otherwise I would package something a bit newer
(0.6.2 is quite old!).

I think the best course of action at this stage would be to work on
sending some patches upstream prior to the new release. Hopefully that
way I can avoid carrying patches for 0.7.0 when it is done, and
produce a better package.

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJRNHyvAAoJEHlzKPr+55fVymYP/2o+qNkaliu7Da6HaXerWaXH
li8jW6DP8F+3E+QpQTMqS7dcUZ8iYnhQKk5kn4H8Bg2wqPS/WsQKyL1UaOd+Qb56
xVG9TdwqbBD5NBsbm+XcnMCL/F58xKF8xZgFTlboCtQl3drp37Y5HydYiJwh1sgX
lZuGifSVdvyFtVRkpqOxT6s+5asthNOEfoaQBebCp1d3xpKOZHjXb8eDB0Bu34fO
vYk6fXIMeAh20GnMou6uwPyCo5hSbXgBblyDJYjRcDFGWFJ7M3ITi9V/0DcTAJQD
MC2fWhP6H1e40CnTNlRIW6SCJBWw3QU75nvhWMjDUn7NmfJVAR+foUofGLKRFraG
6frocetbqogCubOakkCapLL4h4c9qWS946BBq0sLlM+FKlzx3BOutaDawUomHImq
LLf0ZXLJrlQ8TAQQN/RIi+RXcG3lXQ6ZAY7p2KGHjqDzjmn1cS0fPzubRhkLpepU
AZhuH4a/GXGFRVEXqUidDuT2qO/yFrJK2ZjZOjyypYqJlHF8eHTarVQ8KH4PS5uv
W3dM/1qBWrHM7y6CzHbgPf/3UHTKd+LUxxr4en4Wf/MKWV0f3YmdF6r3ir6bcW+L
Qo3HxURjiPC+LB/RNfyRwWVG2Kpc6J9INXEakPx7xlNGSFjDBDnz8zbWTocOtnvc
lofClxTwHr1xa0ECX7tQ
=rkUq
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/51347cb9.8050...@sl.id.au



Bug#662955: RFS: rubyripper/0.6.2-1 [ITP]

2013-03-03 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Dear mentors,

I am re-uploading this package, as my previous upload was removed from
mentors after not finding a sponsor. The packaging has been updated to
the latest debhelper (v9), dependencies simplified, and packaging VCS
added.

 Package name: rubyripper
 Version : 0.6.2-1
 Upstream Author : Bouke Woudstra boukewouds...@gmail.com
 URL : https://code.google.com/p/rubyripper/
 License : GPL-3.0+
 Section : sound

It builds those binary packages:

  rubyripper-cli - Error-correcting compact disc digital audio
extractor (CD ripper). (CLI)
  rubyripper-gtk2 - Error-correcting compact disc digital audio
extractor (CD ripper). (GUI)

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

http://mentors.debian.net/package/rubyripper


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

  dget -x
http://mentors.debian.net/debian/pool/main/r/rubyripper/rubyripper_0.6.2-1.dsc

More information about rubyripper can be obtained from
http://wiki.hydrogenaudio.org/index.php?title=Rubyripper

Changes since the last upload:

rubyripper (0.6.2-1) experimental; urgency=low

  * Initial release (Closes: #463584).
  * Patches:
Ignore lack of $PWD variable in fakeroot build environment.
Don't require ruby-gtk2 to build.
Don't require deprecated gettext API.
Avoid compiling translations twice during build.
Avoid updating translations during build which are already up-to-date.
Don't call deprecated gettext API.
Use 'normalize-audio' binary, as debian has renamed 'normalize'.

 -- Scott Leggett sc...@sl.id.au  Sun, 03 Mar 2013 21:42:36 +1100

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJRM0AMAAoJEHlzKPr+55fVZBMQAJZB0geTcYIfIZgFoMwC62p1
Kr4N6UmutsbMW65lknW4WHHVLGQYnqFtaa5MwuGj5RuuVMKKM0qSnRlALyb1jHXP
flKUfC7xT9GLN3xMcZYBBWFAoEZNcHF2MGs0k+KtMrArS67roOZQ55kzItc05AYU
HgUB533Va3du/Ca+mQBDt667WJTy7EId6F/XTKqnry1NPNXtr47VUK6cayIQl4VO
XutEIcM9kObeSeqBPXT65ED+GJ+ANRLvxJ/u9YZEYH5iMX2hpAWykpABe837byNw
6G1oa8cD/DDEhf/nLbSZuTm9XfgOH1Zz28X4ugGxzdJ7DXPJ5GZYVm4RUr6irWsO
dK7CgpAhJ80gtmnT187cyTFfDQkOBAZexUkL2wJa0oXcsiY1e5tAJj8dDlzskQ1V
M3tky2W97qNAP5hh0U+zy2e+nYzrfd+NEpaDPEdKc/nywZeiPfALiY5ssgQZqpBQ
GI8LSdSNmHmtGEXyntccMNDLBAmtvozgARUTG4EJVgq4ECC/SvXY6/93HlF1SdDk
PSJVpNBEkAUuvU16dkQtzmIRv2aE0lYpKiwFpA+BgJW5nM3yHwRNcXl7Ircgu8xD
SJowjZpFNNDE46ntmoBQ+8BcSdihd8MGZSI4MDaCbTiV599wIEePqoY8H4XIHnUp
tggzUZA5EeLGN6lqAdgS
=VF3k
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/5133400c.7010...@sl.id.au



Re: Watch file warning from mentors website

2013-02-26 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 26/02/13 18:56, Bart Martens wrote:

 The redirector githubredir is obsolete, because github allows uscan
 now.  It's possible that githubredir is still documented in some
 versions of man uscan.
 
 You can choose to add a comments-only watch file or to leave out
 the watch file and ignore the lintian message.
 

Excellent. I'll change the githubredir to github and leave the other
comments. Thanks for the quick and comprehensive response.

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJRLGuRAAoJEHlzKPr+55fVOWsP/jnxmfQsuSPn9CHpCq/iiaii
r9+Sm8veQt7IymjSPaiRVtlHeKGEk0outx6BDCJoErfMWD0XjaIWdT9UwhsOwSQZ
AD/adB5lqE14CP19WS/OOXjxPU0dg99b+RF1vnV2zG99BTbuNGKYmkygctFoP0iP
lNdOxqNrticOMljcmrhBU2yBy7vrcvv6pM2mn7Z3gOQ+b4yxhsZVhe7ubgv18oPw
yDoVfcAyPFYP/PX/aiPLMoQLPwg4Zsv1mb0ExGlTBEtbXBdEn6Z8RspltLDA7Xhf
X8pHvG3+hvgDVdpC70J7uPz0ZZ15L70e0//ijwh6mJ5d7oar5m1qMfNlgSemiWIA
GyC6A7pEW8xr8sbPu5ggkbyeNdeLecnd9wyCiVUNuBC6tQ2kRJCG+Sw5EyAIUSuz
8KNjXryUhm98yVHe85i4mexE0VsoEaA1fSWOt7Kv//vHs4SgU8adEqPOr+SDSC06
9QGRP8MrydEZqF2uuMR7Ob6Bfxx+Ajkf9ZVxKw87bRcyz/vHpdK3gOn8/naxG8Hg
XLUR8U60oyDjmh0J5ntpYBfgkDB7QAgXd5t0rJxQEmYSdilnWnZKY1orlohTx2bC
Hi8AP+zdLrrtKDIBSouhwRQhqi0RIbfrpPnSxZYiIeVxnt0UKd8aL5lkm9sUVPZP
gXciaXvCjgAmjh2P7dw2
=eJPK
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/512c6b92@sl.id.au



Bug#701693: RFS: compton/0.0.1+git-2182505-2013-02-05-1 [ITP]

2013-02-26 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package compton

* Package name: compton
  Version : 0.0.1+git-2182505-2013-02-05-1
  Upstream Author : Christopher Jeffrey chjjeff...@gmail.com
* URL : https://github.com/chjj/compton
* License : X/MIT
  Section : x11

It builds those binary packages:

  compton- Compositor for X11, based on xcompmgr

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

http://mentors.debian.net/package/compton


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

  dget -x
http://mentors.debian.net/debian/pool/main/c/compton/compton_0.0.1+git-2182505-2013-02-05-1.dsc

More information about compton can be obtained from:

  https://wiki.archlinux.org/index.php/Compton

Changes since the last upload:

  compton (0.0.1+git-2182505-2013-02-05-1) experimental; urgency=low

* Imported Upstream version 0.0.1+git-2182505-2013-02-05
* Override dh_auto_clean to quiet verbose build warnings.
* Initial release (Closes: #679551)

   -- Scott Leggett sc...@sl.id.au  Tue, 26 Feb 2013 17:13:22 +110

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)

iQIcBAEBAgAGBQJRLIZnAAoJEHlzKPr+55fVG/gQAKf4Fa2gLFPjSOMnErbfTLOs
jiqXQiSyjNL8CySk1dE2ubnt/yXEbT1qHqjoHnDdPwI6fRHFTqltvSiX1NvK83Oz
KFJh46B7euVvBuLO4uJA2iIpDr89ciAMk18ZFlwkM4cmDuINdBY+bCuP5PaYT0MC
28kfphbONvTbkvqB2HPgMdVugfZv2raXzNEZuRowMCPTlMsFn+aI5lEt4mh3b+eB
vX5pWj6OJ5ESvvm117ETv4OyDVeKwroQwLa6huL/A2hjohGbPgBtntfBwAqkWAvQ
Qn1G+vttUGqVcWv27tp1DHxpunY19lddv60RSPUqCmOqd6QJPbIm/WISXq+JTT8L
PgRcNwBPgNVG2Vag8CYlWs7w4/KPtFjekkYukE1GFLyAHzlLhA36/aM94BlAOyUn
eQEYGqZ0MefwXgGl/iwHaJDt56YNFCeOJ31oa/i0y5RNwQTj2ie5Q97niyKMCMpj
67qPMOzWZfF5SihXK2c5BLo46m+V/crddtR79Tep7iIpPN9fm8viKKLWo3S61dcY
1hCBbhqfZcUlXmAwcuBVona7QrNrq8kmjFWYyRMtUPZu5lVtXYFM8Bgus72AHnI5
rTRf8yVNbXtHi3y6oZFvml5mtPG5cB+IhMJIfM/ulpw79/I0hxbl75OjbsIze6L7
rZDLv4FlmSSAZ2Wrln4S
=Y8w0
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/512c8671.9080...@sl.id.au



Bug#701543: RFS: spout/1.4-1 [ITA] -- Tiny abstract black and white 2D cave-shooter

2013-02-25 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 25/02/13 01:17, Anton Gladky wrote:
 Hi Scott,
 
 thanks for adopting the package. There are a  couple of
 notes/questions:

Hi Anton, thanks for the review. As you can see, I'm quite new at this!

 
 1. Why do you use compat level 8, not 9? 2. The current
 Standards-Version is now 3.9.4.

I used compat level 8 and Standards-Version 3.9.3 because these are
the versions which dh_make automatically inserts when debianising an
application. Thanks for the heads-up regarding the new versions - I've
fixed the packaging.

For future reference, where/when do these new versions get announced
and/or become official?

 3. Why VCS-field is commented out? It is better to use VCS for
 packaging.

Yes, you are right. I have pushed the packaging to github and updated
the VCS fields.

 4. Why libsdl1.2debian is injected to Depends manually? Is it
 linked against spout?

No. This was a mistake due to my misunderstanding of the misc:shlibs
variable. The manual depends has been removed.

 5. I would suggest to target the upload into experimental branch
 instead of unstable due to a freeze-period.

Thanks for the tip, I've changed the upload to target experimental.

 6. It seems, desktop-file is not installing.

You are right, and it also had some other issues which I've now fixed
to lintian's satisfaction.

Thanks again for the thorough review!

I've attached the updated changelog below:

- ---

spout (1.4-1) experimental; urgency=low

  * New Debian maintainer. (Closes: #691830)
  * Imported new upstream version 1.4.
  * New upstream maintainer and homepage.
  * Fix typo and other formatting issues in .desktop file. (Closes:
#556938)
  * Apply 64-bit segfault patch to upstream.
  * Merge new upstream and old Debian man pages.
  * Build-depend on hardening-wrapper.
  * Patched hardening errors.
  * Updated packaging to use debhelper, quilt 3.0.
  * Bump standards version to 3.9.4.
  * Bump compat version to 9.
  * Add new packaging VCS information.

 -- Scott Leggett sc...@sl.id.au  Mon, 25 Feb 2013 23:23:58 +1100
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJRK12FAAoJEHlzKPr+55fVw/kQAJH+rxguDh/4IX9Z1VltdCb8
Xo3Cvby4H1y+fPRFZt2ppasuSRaTaxMQ3lnu/raDMuOEb8WuwRUlQpGZzbw5OMg1
oXATlptzde9asSdj2LpwlRw83pYAICCuBk3vqneWfT91timM45bDTjBG1c/ZO+6i
FVtQpP9bY7FU2YCDa42cGIziWuVfNNtVnTy3NPX4YJy2//Ru1JqHdTozhAx08s15
Cy8X+pVElrAQJsS00xOamLr0Ug8iSW8t1mZHoMEY2RrRzad9qCsxYTwNohvHe0nq
TPOqDgiB4XbkL0kH1aqGZBCkE8/+OqZzsFtlEzOn5d3InHcIpFxuDtfpxlMkV8h9
/zcXxRQzoXM6Amr5yhnEc6Kv1PEmrxigvpWYBCg6IJkjAcEq3s9MjjMtY1sF+ykv
K78Po3CcoeCqInZ+CBjcUI2fMRL6bGAVFnj7EAXauXWkMO7rWeM0U5sr9eoEc4hD
rFBZMzyCGbjq6zTAU2oc7WVoM7fEhJkQBk5SLVYpjLpSl1lx1UMXSReJgmZ/eRnk
z6O4vP6FxY7U27I7DHQefhSzz5Kquej/lpPuu5SdoMISnh3yUs7M+4r8I70CemFy
YyUdp6195gytSKM371oBDk/IJd3hHJQU2Q2FIyFyOaIHf6y5eU1re7BVB3p4JzOr
GkyybyShDghJQ+RiaHzf
=F10K
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/512b5d85.9000...@sl.id.au



Watch file warning from mentors website

2013-02-25 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

I've got an issue with watch file usage in some software I'm trying to
package ( https://mentors.debian.net/package/compton ).

Upstream does not release tarballs, instead preferring packagers to
build straight from git. I've followed the advice of the lintian report
for debian-watch-file-is-missing, and created a watch file with a few
comments explaining the situation. It looks like this:


#version=3
#http://githubredir.debian.net/github/chjj/compton (.*).tar.gz

# Upstream currently does not release tarballs or even tag releases,
instead
# preferring packagers to work straight from git.
# When this situation improves, the watch lines above should work and this
# comment block can be removed. See
https://github.com/chjj/compton/issues/71


This passes lintian fine locally, but when uploaded the mentors website
complains that A watch file is present but doesn't work (see the link
to my package above).

Is there something I can fix in the watch file, or is the mentors
website complaining unnecessarily? It seems strange that the website would
complain while lintian does not.

Thanks for any advice.

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJRLF+NAAoJEHlzKPr+55fVQ7wP/3ITPLZkqFqIYAqem8Zn4RNp
7e7NKc2dK88BOhT0Nc0wPO/BfRzGvJD8hgOY1LfgpO7APHLKps5Wj/vhZgAAYCC8
TMeOt7zLVpcMh5/R3Q2JNP3RPbis16qJ+5bdQqySzb5e7AaZI+w38RO3YwM4n3TA
26cWCHG9mb2cyWhCNjgapqRFtZqi/ihYykrDCl0fVFT3xGtvGHJ1rzIIXmqQpAOs
/QlX89awXQfgwAq14AgoONKKeeoSskxrvNpDYg3xLpuV+MgxypEAl/BcRXDXSglq
rpBfwCm2DfxSp0imqcCCgUtX35GV0PtCVHKaxcqwYxLMF9hSViMbBmyLjH6eN/Zd
22edGbDdosfj/GQHrIHvJdR5EzoiC+bFmXsnzTj4n80iuNvkb6gXWlZOlCt4/o4k
ihvHYfJrHAuhfPMntuYJ0FY4WJBBmIaoqzPTM+sx1fslD/n3EQ/Ph80rEAQuKHQY
Cah60wJhS7gw+8slkkjbreGetr+9hvbaG4S3fxQRAFqOFLlhl7tKnvgBz1mcFr08
2t/rVxEnnHGBrqgPSicApiUBaw6e0wBb0GVfc1VjFg4Nldm/OVXq5HfevMrWZl/M
9TlsA2nk/11xdf9/fO1DqGQnpQ2KSjdKqUx/mBq/ZrnOVnabNtbKutUENyzk0yZV
cmSSqiwDEqENgiHx38r2
=Xvnb
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/512c5f8d.7060...@sl.id.au



Bug#701543: RFS: spout/1.4-1 [ITA] -- Tiny abstract black and white 2D cave-shooter

2013-02-24 Thread Scott Leggett
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package spout

   Package name: spout
   Version : 1.4-1
   Upstream Author : Nick White n...@njw.me.uk
   URL : http://njw.me.uk/spout/
   License : MIT
   Section : games

It builds those binary packages:

  spout - Tiny abstract black and white 2D cave-shooter

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

http://mentors.debian.net/package/spout

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

dget -x http://mentors.debian.net/debian/pool/main/s/spout/spout_1.4-1.dsc

More information about spout can be obtained from:
http://njw.me.uk/spout/

Changes since the last upload:

spout (1.4-1) unstable; urgency=low

  * New Debian maintainer. (Closes: #691830)
  * Imported new upstream version 1.4.
  * New upstream maintainer and homepage.
  * Fix typo in Category field of .desktop file. (Closes: #556938)
  * Apply 64-bit segfault patch to upstream.
  * Merge new upstream and old Debian man pages.
  * Build-depend on hardening-wrapper.
  * Patched hardening errors.
  * Updated packaging to use debhelper, quilt 3.0.
  * Bump standards version.

 -- Scott Leggett sc...@sl.id.au  Sun, 24 Feb 2013 15:45:03 +1100

- -- 
Regards,
Scott Leggett.
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQIcBAEBAgAGBQJRKf3UAAoJEHlzKPr+55fVOP8P/01J12Yrv6cPTP4YjN/KYNtv
GgrF8YqgAU8wdDW8me354UocQAN/5b/AlBdrcf62FgY7UgltsxarjVu5Qmuwee1j
y1NE7fn7G+2JhpEt3vsdX2ZQ+v2I77kV1R9kkCGfjBIBCT4C/APEUruGFD6VK7Qv
VbQWmbBExLPxsVyua7hLMnmTYwtDrkiv+HAlLZxg86ilghmyMYMg6w8B0lm9kBMj
9Mvl9xEtdMeYOfxcA5HOrwkQmovpj5zpKSZLsTCP6yto80VLssucdcpuMZO8BVaw
0w0U3I75J6JMdQmpSUlHFAZBv61SpqTfxA0eUclnRiR9c/DFlDXmkPaeMpsOThd0
FIJOqdctmrPyCCRbaM/hHMPg3Xg/+7BKKKZiIpNRv7XgTxCAiwjteHdmUtUB0HiA
iF+4mw9tJwOaNSINGZ4P73UdVerxJURrwyXGE3/ZPNJYoxQ8gwc2JtSsoctN5z9d
Pa2Ly/9OJmWduPANS7FNwER4N6TbmQbiBCNjxnzbyy9uk+1DmIUWdH1GrDz1t/cL
vAqGl/FO0sG7oQEB74SjANPE+HuDKZoK1blpMV8Osr8zg6IXf7YuS3t6H8ZXQhQA
VC/8lC5FSCS7g3ef4jCyU1nldDCiOFfDDNvAbgKW/DCbGXJnqX8xhNLyBfd9aZIF
qq65QdimDFP37rTpW9ae
=phkb
-END PGP SIGNATURE-


-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/5129fdd4.6050...@sl.id.au



Bug#662955: rubyripper Re: Bug#662955: Is there any progress?

2013-02-15 Thread Scott Leggett

On 12/02/13 20:28, Steffen Möller wrote:

Hello,

I would be prepred to sponsor the package, if Scott and/or Benjamin are around 
to perform the maintenance. It seems a bit like the package just found no 
sponsor, which is then a sad thing.

Steffen




Hi Steffen,

Yes, I thought it might be difficult to find a sponsor while in release 
freeze and so was waiting before I pinged the mentors list again. 
However I'd be glad to perform maintenance if you are willing to sponsor 
the package now.


This is my first attempt at maintainership, so please let me know about 
any issues with the package currently uploaded at debian mentors.


I've used/tested the current package quite a bit since I put it up, so 
if there are no packaging issues I think it is ready for uploading to 
the archive.


Regards,
Scott Leggett.


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/511e451d.10...@sl.id.au



Bug#692263: RFS: compton/0.0.1+20121102.git239796ab-1

2012-11-04 Thread Scott Leggett

Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package compton

 Package name: compton
 Version : 0.0.1+20121102.git239796ab-1
 Upstream Author : Christopher Jeffrey chjjeff...@gmail.com
 URL : https://github.com/chjj/compton
 License : X/MIT
 Section : x11

It builds those binary packages:

  compton- Compositor for X11, based on xcompmgr

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


http://mentors.debian.net/package/compton

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

  dget -x 
http://mentors.debian.net/debian/pool/main/c/compton/compton_0.0.1+20121102.git239796ab-1.dsc


Changes since the last upload:

compton (0.0.1+20121102.git239796ab-1) unstable; urgency=low

  * Initial release (Closes: #679551)

 -- Scott Leggett sc...@sl.id.au  Sat, 03 Nov 2012 02:20:30 +1100


--
Regards,
Scott.


--
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/5096496e.30...@sl.id.au



Bug#662955: RFS: rubyripper/0.6.2-2 [ITP] -- New upload

2012-03-12 Thread Scott Leggett

Dear mentors,

I am looking for a sponsor for my package rubyripper

* Package name: rubyripper
  Version : 0.6.2-2
  Upstream Author : Bouke Woudstra boukewouds...@gmail.com
* URL : https://code.google.com/p/rubyripper/
* License : GPL-3.0+
  Section : sound

It builds those binary packages:

rubyripper-cli - Error-correcting compact disc digital audio extractor (CD 
ripper)
rubyripper-gtk2 - Error-correcting compact disc digital audio extractor (CD 
ripper)

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

  http://mentors.debian.net/package/rubyripper

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

  dget -x 
http://mentors.debian.net/debian/pool/main/r/rubyripper/rubyripper_0.6.2-2.dsc

More information about rubyripper can be obtained from 
http://wiki.hydrogenaudio.org/index.php?title=Rubyripper

Changes since the last upload:

rubyripper (0.6.2-2) unstable; urgency=low

  * Removed unnecessary ${shlibs:Depends} variable from dependencies.
  * Change package priority to Optional.
  * Unset unnecessary DH_VERBOSE in debian/rules.
  * Add several optional packages to Suggests.
  * Update debian/copyright to latest format.
  * Improve package description in debian/control and manpage.
  * Split upstream patch into two.
  * Add two more patches to upstream:
Fix calls to deprecated API 'gettext/utils'.
Remove prerequisite to Makefile 'install' target to avoid compiling
translations twice.
  * Disable updating of .po files due to crash in ruby 1.9.x.
  * Add patch to use correct API for mofile generation function.

 -- Scott Leggett sc...@sl.id.au  Mon, 12 Mar 2012 16:13:16 +0800

-- 
Regards,
Scott.



-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201203121648.36385.sc...@sl.id.au



Bug#662955: review

2012-03-08 Thread Scott Leggett

On Fri, 9 Mar 2012 01:02:42 David Banks wrote:
 Hi, I'm not a DD so I can't sponsor your package, but I would be using this
 package if it was in the archive.  Thanks for packaging it. 

Hi, wow thanks for the great detailed review!

  * You don't need the substvar ${shlibs:Depends} in the control file for
 the binary packages, since they are not compiled code.  This gives a
 warning currently.

Okay, I will remove this.

  * Why is the package priority extra?  It should probably be optional.

After re-reading the Policy Manual I see you are correct.

  * Consider switching off DH_VERBOSE in debian/rules before release, or at
 least remove the comment above it.

Yes, I missed that.

  * The translations seem to be being compiled twice, once during
 dh_auto_build and once during dh_auto_install.  This is probably
 unnecessary.

Hmm, I hadn't really noticed that but you're correct. I'll have to patch the 
configure script to remove the 'all' prerequisite from the makefile's 
'install' target. It seems to be there because upstream's installation 
instructions don't require a separate 'make' and 'make install'.

 * You may want to Recommend or Suggest the 'vorbisgain' and
 'mp3gain' package, if rubyripper can use them.  The same goes for
 'normalize' but this may require a patch, since Debian uses the name
 'normalize-audio' for this command.  These were mentioned by the configure
 script.
  * Grepping the source it seems that rubyripper can also use 'cdrdao' and
 'sox' for certain things -- consider the same for them.

Yes, I will Suggest these packages. Strange that the last two aren't mentioned 
in any documentation that I've read (they're in the changelog though). I'll 
file a bug report with upstream about it.

  * Consider changing your DEP-5 format URL now that this has become
 official.
 http://www.debian.org/doc/packaging-manuals/copyright-format/1.0/

Yep, I need to run lintian with -pedantic.

 * Do
 you need to patch the prefix '/usr' into the configure script?  It seems
 you already specify the /usr prefix by an override in debian/rules anyway,
 so that seems redundant.

This was left-over from my attempts to fix a build issue. I'll remove it.

  * You might consider splitting your patch into multiple patches, which
 would allow you to give a more detailed rationale for each change.  For
 instance, your patch removes some code from the configure script probably
 to solve some build issue, but the patch header doesn't explain this.

There were actually multiple build issues, which I didn't fully understand 
until I had patched them.. so yes, I will split these patches out properly.

  * Since you use the same man page for both rrip_cli and rrip_gui, you may
 want to remove (command-line interface) from the NAME section of the
 manual page.

Totally missed that, thanks!

 
 Minor bugs that should probably be fixed by upstream:
   * Consider fixing the example copyright headers on the .po files, they
 have pasted in boilerplate.

I haven't filed a bug report with upstream over this, but it's on my TODO.

   * I get this message when running configure:
   'gettext/utils.rb' is deprecated. Use gettext/tools.rb.

Yep, that can be patched.

   * Many translations are generating warnings like:
   - Obsolete msgid exists
   - Fuzzy message was ignored
 

This is another issue I have to raise with upstream. I did notice this, but 
translations seemed to work fine anyway. Do you know what these warnings 
actually mean? 'info gettext' doesn't provide any documentation for warnings.

 Thanks for your work!  I will certainly be using this when it is uploaded.
 
 Cheers,
 David

No problem, thanks for taking the time to review the package! I should be able 
to upload another version with the changes sometime this weekend..

-- 
Regards,
Scott.



-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201203091444.43932.sc...@sl.id.au



Bug#662955: RFS: rubyripper/0.6.2-1 [ITP] -- audio extractor

2012-03-07 Thread Scott Leggett

Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package rubyripper

* Package name: rubyripper
  Version : 0.6.2-1
  Upstream Author : Bouke Woudstra boukewouds...@gmail.com
* URL : https://code.google.com/p/rubyripper/
* License : GPL-3.0+
  Section : sound

It builds those binary packages:

  rubyripper-cli - Error-correcting compact disc digital audio extractor. 
(CLI)
  rubyripper-gtk2 - Error-correcting compact disc digital audio extractor. 
(GUI)

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

  http://mentors.debian.net/package/rubyripper

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

  dget -x 
http://mentors.debian.net/debian/pool/main/r/rubyripper/rubyripper_0.6.2-1.dsc

More information about rubyripper can be obtained from 
http://wiki.hydrogenaudio.org/index.php?title=Rubyripper

Changes since the last upload:

* Initial release (Closes: #463584).

Regards,
Scott Leggett



-- 
To UNSUBSCRIBE, email to debian-mentors-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/201203072252.55078.sc...@sl.id.au