Bug#827685: RFS: elpa-undo-tree/0.6.4-1 ITP

[Sean Whitton]

Hello,

On Mon, Jun 20, 2016 at 05:06:29PM +0300, Dmitry Bogatov wrote:
> 
> > 1. The /source/ package name should actually be undo-tree not
> >elpa-undo-tree (team convention to use upstream's name).  Probably
> >not worth changing now, but you should be aware of this for packaging
> >other addons (you mentioned you wanted to package evil).
> 
> Ok. In self-protection can point on elpa-paredit source package.

Can you? :)  Might want to check again...

> > 2. Is there some reason you packaged version 0.6.4?  0.6.6 is
> > available.
> 
> $ git tag | grep 0.6.6
> [empty]

Hmm, nevermind, not sure where I thought I'd found 0.6.8.

> > 4. Please apply a PGP-signed tag to the correct commit in the team
> >repo.  Then it will appear correctly on PET.[1]
> 
> You mean, I should gpg-tag 'a9e75' as debian/0.6.4-1?

Yes, that would be useful.

-- 
Sean Whitton


signature.asc
Description: PGP signature

Bug#827037: RFS: xpad/4.8.0-1 [ITA]

[Sean Whitton]

control: tag -1 +confirmed

Thanks for your response.  I checked the new package and it looks good.

Nitpick only; fix at your discretion:

On Mon, Jun 20, 2016 at 03:41:52PM +0200, JCF Ploemen wrote:
> > >   * Patches:
> > > + remove missing.diff, no longer needed.  
> > 
> > It would be good to say in the changelog why it's no longer needed
> > (obsoleted by XX?  applied upstream?).
> 
> Unfortunately, that patch came completely undocumented and didn't make
> much sense, so I went with a delete-and-see approach here. Could well
> be that using autoreconf was all it took to get rid of it. The lack of
> info about why it was there in the first place makes it hard to provide
> more details beyond the simple notion of being 'no longer needed'.

Might want to say "no longer needed with new d/rules", in that case.

-- 
Sean Whitton


signature.asc
Description: PGP signature

Bug#827487: bytes-circle_2.2-1_amd64.changes REJECTED

[Roberto S. Galende]

It'd be available now: I didn't toggle the 'needs a sponsor?' option.

Bug#826769: RFS: arc-theme/20160605-1 ITP

[Adam Borowski]

Control: owner -1 !
Control: tags -1 +moreinfo

On Wed, Jun 08, 2016 at 09:05:54PM +0100, foss.freedom wrote:
>   I am looking for a sponsor for my package "arc-theme"
> 
>  * Package name: arc-theme
>Version : 20160605-1

Two minor issues:

W: arc-theme: description-synopsis-starts-with-article

Vcs-Browser points to upstream rather than debianisation (like in your two
other packages).


Otherwise, everything seems fine.


Meow!
-- 
An imaginary friend squared is a real enemy.

Bug#827748: marked as done (RFS: growl-for-linux/0.8.5-1)

[Debian Bug Tracking System]

Your message dated Tue, 21 Jun 2016 04:28:26 +
with message-id 
and subject line closing RFS: growl-for-linux/0.8.5-1
has caused the Debian Bug report #827748,
regarding RFS: growl-for-linux/0.8.5-1
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
827748: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=827748
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "growl-for-linux"

 * Package name: growl-for-linux
   Version : 0.8.5-1
   Upstream Author : Yasuhiro Matsumoto 
 * URL : https://mattn.github.io/growl-for-linux/
 * License : BSD-2-clause
   Section : gnome

  It builds those binary packages:

growl-for-linux - Pluggable notification system which supports GNTP

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

  https://mentors.debian.net/package/growl-for-linux


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

dget -x 
https://mentors.debian.net/debian/pool/main/g/growl-for-linux/growl-for-linux_0.8.5-1.dsc

  More information about growl-for-linux can be obtained from 
https://github.com/mattn/growl-for-linux

  Changes since the last upload:

  * New upstream release
  * debian/patches/series
- Removed drop-removed-growl4linuxjpg.patch because it was already
  applied in upstream.
  * debian/patches/disable-pie-for-libgolplug.patch
- Added a patch to disable PIE explicitly for libgolplug not to be
  conflicted with shared libraries under display/*.
  * debian/watch,debian/upstream/signing-key.asc
- Supported cryptographic signature verification.

Regards,


pgpj_85RpgZvB.pgp
Description: PGP signature
--- End Message ---
--- Begin Message ---
Package growl-for-linux version 0.8.5-1 is in unstable now.
https://packages.qa.debian.org/growl-for-linux--- End Message ---

Bug#826974: RFS: moka-icon-theme/5.3.2-1 ITP

[Adam Borowski]

Control: owner -1 !
Control: tags -1 +moreinfo

On Fri, Jun 10, 2016 at 08:18:12PM +0100, foss.freedom wrote:
>   I am looking for a sponsor for my package "moka-icon-theme"
> 
>  * Package name: moka-icon-theme
>Version : 5.3.2-1
> 
> moka-icon-theme - Moka Icon Theme

First, please run lintian on the package.  It currently returns:
E: moka-icon-theme: copyright-should-refer-to-common-license-file-for-gpl
E: moka-icon-theme: description-is-pkg-name Moka Icon Theme

The Vcs-Browser needs to point to a VCS of the packaging rather than the
upstream code.  The VCS might contain just the debian/ dir or everything
-- a ready-to-build source package.  If you haven't used git for packaging,
please delete the field.

debian/rules contains:
override_dh_auto_install:
--sourcedir=Moka
which has no effect.  Not sure what you intended to do here; this paragraph
can be safely deleted.


>From minute differences I noticed in a few places, I see logos included in
this package were drawn by hand by the author rather than copied exactly. 
However, they are still derived from the originals.  Copying by wetware is
still copying.  This package includes a massive number of icons for various
software and companies, both free and non-free.  I'm not sure what the exact
rules are: faenza was forced to remove most logos, yet I see those in a
number of other packages.  I guess it's up to ftpmasters to decide.

Meow!
-- 
An imaginary friend squared is a real enemy.

Bug#826986: marked as done (RFS: faba-icon-theme/4.1.2-1 ITP)

[Adam Borowski]

On Tue, Jun 21, 2016 at 03:00:05AM +, Debian Bug Tracking System wrote:
> Looks good, uploaded.

One minor issue I noticed too late:
the Vcs-Browser field should point to a repository containing the Debian
packaging, rather than just the upstream code.  The latter may be put into
debian/upstream/metadata which you already included.

Thus, if you have your packaging in git, please point both Vcs-Git and
Vcs-Browser to it.  Otherwise, delete the field.

-- 
An imaginary friend squared is a real enemy.

Bug#826986: marked as done (RFS: faba-icon-theme/4.1.2-1 ITP)

[Debian Bug Tracking System]

Your message dated Tue, 21 Jun 2016 04:56:12 +0200
with message-id <20160621025612.ga26...@angband.pl>
and subject line Re: Bug#826986: RFS: faba-icon-theme/4.1.2-1 ITP
has caused the Debian Bug report #826986,
regarding RFS: faba-icon-theme/4.1.2-1 ITP
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
826986: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=826986
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
  Severity: wishlist

  Dear mentors,

  I am looking for a sponsor for my package "faba-icon-theme"

 * Package name: faba-icon-theme
   Version : 4.1.2-1
   Upstream Author : Sam Hewitt 
 * URL : github.com/moka-project/faba-icon-theme
 * License : LGPL-3.0 or CC-BY-SA-4.0
   Section : misc

  It builds those binary packages:

faba-icon-theme - Tango influenced icon theme called Faba

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

  https://mentors.debian.net/package/faba-icon-theme


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

dget -x 
https://mentors.debian.net/debian/pool/main/f/faba-icon-theme/faba-icon-theme_4.1.2-1.dsc

Notes:

I am the maintainer of a Ubuntu based distro called budgie-remix which

uses moka-icon-theme & faba-icon-theme as its key icon themes - hence
why I'm excited to maintain this package for the wider

Debian community. budgie-remix uses the new desktop environment called
budgie-desktop which I have also packaged here

 - https://lists.debian.org/debian-mentors/2016/05/msg00551.html

moka-icon-theme is a highly popular icon theme used by several
distros.  faba-icon-theme is the base theme for moka.

It is installed currently by Ubuntu users through the upstream
maintainer PPA repository or via manual compilation.

I'm providing this package to allow all Debian users and Debian
derivatives a native

package rather than depending upon a third-party Ubuntu only repo.


I've asked the question here as to whether upstream will sign future
tag releases:

 - https://github.com/moka-project/moka-icon-theme/issues/232

The upstream maintainer has indicated that they do not wish to GPG
sign releases on his project.


 Regards,

 David Mohammed
--- End Message ---
--- Begin Message ---
On Fri, Jun 10, 2016 at 09:51:20PM +0100, foss.freedom wrote:
>   I am looking for a sponsor for my package "faba-icon-theme"
> 
>  * Package name: faba-icon-theme
>Version : 4.1.2-1

Looks good, uploaded.

-- 
An imaginary friend squared is a real enemy.--- End Message ---

Bug#827082: RFS: libredjackipset/1.1.1+20150311-1 [ITP] -- C library to store sets/maps of IP address

[Roger Shimizu]

On Tue, Jun 21, 2016 at 2:38 AM, Roger Shimizu  wrote:
>
> On Mon, Jun 20, 2016 at 10:03 PM, Gianfranco Costamagna
>  wrote:
>
>> 6) did you forward the rename to upstream?
>
> https://github.com/redjack/ipset/issues/41

Wait a minute, now I think your concern regarding renaming is right.
Consider the upstream contributor all left that company, using company
name in library name is not good idea.

Since this library is actually a sub-library of libcork, I'd rather
call it libcorkipset now.
I'll make the change and retitle the ITP/RFS later.

Thanks for your understanding!

Cheers,
-- 
Roger Shimizu, GMT +9 Tokyo
PGP/GPG: 17B3ACB1

Bug#827487: bytes-circle_2.2-1_amd64.changes REJECTED

[Adam Borowski]

On Mon, Jun 20, 2016 at 07:34:54PM +0200, Roberto S. Galende wrote:
> I've updated license in source code and
> also at github.com to match it, with "GPL v3 or higher".

Where may I get the updated package?  I see nothing on mentors.debian.net.

> I'd update the man page with very few mathematical details (hope this
> doesn't disturb the process).

Feel free to make any updates you want, man page changes are among least
intrusive ones.  And in general, the more documentation, the better.

Feel free to make any updates other than man page, too -- some extra review
churn is a good price to pay for having better quality packages.


Meow!
-- 
An imaginary friend squared is a real enemy.

Re: Help with python packaging

[Peter Spiess-Knafl]

Thank you both. I will try it out.

Greetings
Peter

On 06/20/2016 12:05 PM, Gianfranco Costamagna wrote:
> Hi
> 
> 
> 
>> Get it fixed upstream.
> 
> 
> other than that, the manpage issue can be "probably" fixed by
> "PYTHONPATH=."
> before invoking help2man
> 
> and of course using ./bin/codespell.py as argument
> 
> oh, and I think manpage generation should be done 
> 
> in override_dh_installman, not dh_auto_build target.
> 
> so, something like
> 
> removing the dh_auto_build override
> 
> +override_dh_installman:
> +   PYTHONPATH=. help2man debian/codespell/usr/bin/codespell.py \
> +   --include debian/codespell.1.include \
> +   --no-info \
> +   --output codespell.1
> +   sed -i '/\.SS \"Usage/,+2d' codespell.1
> +   dh_installman
> 
> 
> should work.
> 
> (not sending you a good patch by purpose, sorry!)
> 
> also, you need to tell debhelper to install the manpage ;)
> 
> man ./codespell.1 seems to return something good to me
> 
> 
> G.
> 

Bug#827425: RFS: lacme/0.1-1 [ITP] -- ACME client written with process isolation and minimal privileges in mind

[Guilhem Moulin]

Hi Harlan,

On Wed, 15 Jun 2016 at 22:30:18 -0400, Harlan Lieberman-Berg wrote:
> Guilhem Moulin  writes:
>> I am looking for a sponsor for my package "lacme"
>
> This looks like a well-Debianized package to me.
> […]
> I also want to make you aware of the Let's Encrypt Debain Packaging
> Team, in case you were interested in having the package be co-maintained
> with us (with the team in Uploaders), or maintained under that heading
> entirely (with the team in Maintainer).

Although you didn't take Bug Ownership, did I understand correctly that
you or your fellow team members might be interested in sponsoring this
upload?  No hurries, I'm just wondering if I should poke the list again
in two weeks or so ;-)

Cheers,
-- 
Guilhem.


signature.asc
Description: PGP signature

Bug#827333: RFS: newlisp/10.7.0-1 [ITP]

[Andrey Rahmatullin]

On Mon, Jun 20, 2016 at 03:37:55PM -0400, Sergio Durigan Junior wrote:
> > I've uploaded the package, thanks for your contribution to Debian :)
> Thank you!  I guess I will ping you again when the package has been
> accepted, so that you can allow me to make further uploads as a DM,
> right?
OK.


-- 
WBR, wRAR


signature.asc
Description: PGP signature

Bug#827333: RFS: newlisp/10.7.0-1 [ITP]

[Sergio Durigan Junior]

On Monday, June 20 2016, Andrey Rahmatullin wrote:

> On Sun, Jun 19, 2016 at 04:58:32PM -0400, Sergio Durigan Junior wrote:
>> > The package generates quite a lot of lintian output:
>> >
>> > P: newlisp source: source-contains-prebuilt-java-object 
>> > guiserver/guiserver.jar
>> 
>> This file is generated by guiserver/Makefile.  It is a jar file created
>> From the .java files under guiserver/java.
> Looks like it isn't used in the package. If it is/will be used, it needs
> to be rebuilt.

OK, thanks for the heads up.

>> > I: newlisp: spelling-error-in-manpage usr/share/man/man1/newlisp.1.gz 
>> > Excute Execute
>> > I: newlisp: spelling-error-in-manpage usr/share/man/man1/newlisp.1.gz 
>> > Excute Execute
>> > I: newlisp: spelling-error-in-manpage usr/share/man/man1/newlispdoc.1.gz 
>> > documetation documentation
>> 
>> Somehow I thought that these were not worth fixing with local patches,
> Yes, but they are worth fixing upstream and for that you would want to
> provide a patch anyway.

Oh, certainly, I thought that they were not worth fixing with local
patches to the package, but I am definitely going to propose the fixes
upstream :-).

> Don't forget to send all useful patches upstream, btw.

Will do!

> I've uploaded the package, thanks for your contribution to Debian :)

Thank you!  I guess I will ping you again when the package has been
accepted, so that you can allow me to make further uploads as a DM,
right?

Cheers,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


signature.asc
Description: PGP signature

Bug#827333: marked as done (RFS: newlisp/10.7.0-1 [ITP])

[Debian Bug Tracking System]

Your message dated Mon, 20 Jun 2016 23:39:55 +0500
with message-id <20160620183955.gr15...@belkar.wrar.name>
and subject line Re: Bug#827333: RFS: newlisp/10.7.0-1 [ITP]
has caused the Debian Bug report #827333,
regarding RFS: newlisp/10.7.0-1 [ITP]
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
827333: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=827333
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

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

* Package name: newlisp
  Version : 10.7.0-1
  Upstream Author : Lutz Mueller 
* URL : http://www.newlisp.org
* License : GPL-3+, BSD-3-clause, GFDL-NIV-1.2
  Section : lisp

It builds those binary packages:

newlisp - LISP like, general purpose scripting language
newlisp-dbg - LISP like, general purpose scripting language (debug
symbols)

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

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

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

  dget -x 
https://mentors.debian.net/debian/pool/main/n/newlisp/newlisp_10.7.0-1.dsc

The ITP bug for the package can be found at:

  https://bugs.debian.org/425456

Cheers,

-- 
Sergio
GPG key ID: 237A 54B1 0287 28BF 00EF  31F4 D0EB 7628 65FC 5E36
Please send encrypted e-mail if possible
http://sergiodj.net/


signature.asc
Description: PGP signature
--- End Message ---
--- Begin Message ---
On Sun, Jun 19, 2016 at 04:58:32PM -0400, Sergio Durigan Junior wrote:
> > The package generates quite a lot of lintian output:
> >
> > P: newlisp source: source-contains-prebuilt-java-object 
> > guiserver/guiserver.jar
> 
> This file is generated by guiserver/Makefile.  It is a jar file created
> From the .java files under guiserver/java.
Looks like it isn't used in the package. If it is/will be used, it needs
to be rebuilt.

> > I: newlisp: spelling-error-in-manpage usr/share/man/man1/newlisp.1.gz 
> > Excute Execute
> > I: newlisp: spelling-error-in-manpage usr/share/man/man1/newlisp.1.gz 
> > Excute Execute
> > I: newlisp: spelling-error-in-manpage usr/share/man/man1/newlispdoc.1.gz 
> > documetation documentation
> 
> Somehow I thought that these were not worth fixing with local patches,
Yes, but they are worth fixing upstream and for that you would want to
provide a patch anyway.
Don't forget to send all useful patches upstream, btw.

I've uploaded the package, thanks for your contribution to Debian :)

-- 
WBR, wRAR


signature.asc
Description: PGP signature
--- End Message ---

Bug#827082: RFS: libredjackipset/1.1.1+20150311-1 [ITP] -- C library to store sets/maps of IP address

[Roger Shimizu]

Dear G,

Thanks for your review!

On Mon, Jun 20, 2016 at 10:03 PM, Gianfranco Costamagna
 wrote:
> control: owner -1 !
> control: tags -1 moreinfo
>
> Hi, lets see
>
> 1) control: why the -dev package is not multiarch?
> sounds like it is multiarch ready to me
> (I didn't check but include/* seems to have the same md5sum on different archs

Yes, it's multi-arch, just forgot to mark it in d/control.
Fixed.

> 2) the dev package should suggest the doc one

Fixed.

> 3) d/p/0002-fix-build-with-static-libcheck-library.patch
>
> I can't complain here, I seem to know the author of that patch :p

Thanks for your great help!
This convinced me that this library can be packaged, too.

> 3) d/p/0004-remove-test-of-python-cram.patch
>
> why? python-cram is available on Debian

Yes, I used python3-cram in libcork, but this time it just doesn't work.
So I removed it.

> 4) d/p/0007-*
> why? please be more verbose on patches, otherwise I find difficult to 
> understand why you are using sphinx instead
> of pandoc to build documentation

upstream's docs/CMakeLists.txt just output nothing.
You can see "set(MAN_PAGES)", the target is empty!

So I used the same way in libcork to generate docs.
Actually the upstream was during the transition from sphinx to pandoc.
They removed sphinx setting, added pandoc setting, but didn't remove
sphinx docs.
So I patched back sphinx setting, just works.

(docs from upstream just poor, even I wrote a patch to add index page)

> 5) rules:
> I don't like hacks
> and your sed makes the package non-rebuildable.
>
> maybe overriding the
>
> dh_install target, and acting under debian/libfoo-dev/usr/include
>
> is a "better" way (also, a mv instead of ln -s since this old directory 
> should go away)

gbp-buildpackage and git-pbuilder just behave differently.
the former does: build => install => build test + test
the latter does: build => build test + test => install

Because the library is renamed from libipset to libredjackipset,
current hack is the simplest way to make both gbp-buildpackage and
git-pbuilder happy.

> --buildsystem=cmake --builddirectory=build should be useless

I removed "--buildsystem=cmake"
Thanks for your suggestion!

Without "--builddirectory=build", the docs will be built under
/docs/
This make install file hard to write.

> also
> DPKG_EXPORT_BUILDFLAGS = 1
> include /usr/share/dpkg/buildflags.mk

Removed.
Thanks for your suggestion!

> 6) did you forward the rename to upstream?

https://github.com/redjack/ipset/issues/41

> 7) libredjackipset-docs.docs
> is it useful? there is no package with that name, so probably the file can be 
> removed

Fixed.

> 8) I'm not a fan of dh-exec, and your patch about multiarch seems incomplete
>
> +include(GNUInstallDirs)
>
> this has to be done *before*
>
> "set(CMAKE_INSTALL_LIBDIR lib${LIB_SUFFIX} CACHE STRING"
> note the LIB_SUFFIX
>
> and then the install files can change from
> usr/lib/lib*.so usr/lib/${DEB_HOST_MULTIARCH}
> to
> usr/lib/*/*.so
> and similar for others

Yes, moving "include(GNUInstallDirs)" makes dh-exec useless.
My mistake forced me to introduced this complex. Sorry about that.

> 9)
>
> did you try compat level 10?
> it should speed up the build by running dh --parallel :)
> (you can also experiment with --parallel on the default dh call in rules file)

I'd love to try new stuff after first release :-D

Thanks again!

Cheers,
-- 
Roger Shimizu, GMT +9 Tokyo
PGP/GPG: 17B3ACB1

Bug#827487: bytes-circle_2.2-1_amd64.changes REJECTED

[Roberto S. Galende]

Hi,
I've updated license in source code and
also at github.com to match it, with "GPL v3 or higher".

I'd update the man page with very few mathematical details (hope this
doesn't disturb the process).

Thank you.

Bug#827765: marked as done (RFS: gdbm/1.12-2 ITP)

[Debian Bug Tracking System]

Your message dated Mon, 20 Jun 2016 17:20:20 + (UTC)
with message-id 
<1026164991.10784021.1466443220878.javamail.ya...@mail.yahoo.com>
and subject line Re: Bug#827765: RFS: gdbm/1.12-2 ITP
has caused the Debian Bug report #827765,
regarding RFS: gdbm/1.12-2 ITP
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
827765: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=827765
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---

Package: sponsorship-requests
Severity: wishlist

Dear mentors,

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

* Package name: gdbm
  Version : 1.12-2
  Upstream Author : bug-g...@gnu.org
* Url : http://directory.fsf.org/project/gdbm/
* Licenses: GFDL-1.3+, GPL-3+
  Section : libs

It builds those binary packages:

libgdbm4 -- GNU dbm database routines (runtime version)
libgdbm-dev -- GNU dbm database routines (development files)
gdbmtool -- GNU dbm database routines (command line tools)
libgdbm-compat4 -- GNU dbm database routines (legacy support runtime 
version)
libgdbm-compat-dev -- GNU dbm database routines (legacy support development 
files)

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

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

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

http://mentors.debian.net/debian/pool/main/g/gdbm/gdbm_1.12-2.dsc

Alternatively, you can access package debian/ directory via git from URL:

https://anonscm.debian.org/cgit/users/kaction-guest/gdbm.git

More information about gdbm can be obtained from 
http://directory.fsf.org/project/gdbm/

Changes since last upload:

  * Install localization into libgdbm4 binary package
  * Install info manual into libgdbm-dev binary package
  * Separate dbm and ndbm compat layer into separate package

Regards,
  Dmitry Bogatov
--- End Message ---
--- Begin Message ---
Hi


>sponsoring in a few seconds, with a change
>s/UNRELEASED/experimental.


in deferred/1
G.--- End Message ---

Bug#827765: RFS: gdbm/1.12-2 ITP

[Gianfranco Costamagna]

Hi



>I am looking for a sponsor for my package "gdbm"


BTW not ITP :) (wrt message header)

sponsoring in a few seconds, with a change
s/UNRELEASED/experimental.

G.

Bug#827765: RFS: gdbm/1.12-2 ITP

[Dmitry Bogatov]

Package: sponsorship-requests
Severity: wishlist

Dear mentors,

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

* Package name: gdbm
  Version : 1.12-2
  Upstream Author : bug-g...@gnu.org
* Url : http://directory.fsf.org/project/gdbm/
* Licenses: GFDL-1.3+, GPL-3+
  Section : libs

It builds those binary packages:

libgdbm4 -- GNU dbm database routines (runtime version)
libgdbm-dev -- GNU dbm database routines (development files)
gdbmtool -- GNU dbm database routines (command line tools)
libgdbm-compat4 -- GNU dbm database routines (legacy support runtime 
version)
libgdbm-compat-dev -- GNU dbm database routines (legacy support development 
files)

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

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

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

http://mentors.debian.net/debian/pool/main/g/gdbm/gdbm_1.12-2.dsc

Alternatively, you can access package debian/ directory via git from URL:

https://anonscm.debian.org/cgit/users/kaction-guest/gdbm.git

More information about gdbm can be obtained from 
http://directory.fsf.org/project/gdbm/

Changes since last upload:

  * Install localization into libgdbm4 binary package
  * Install info manual into libgdbm-dev binary package
  * Separate dbm and ndbm compat layer into separate package

Regards,
  Dmitry Bogatov

Bug#827685: RFS: elpa-undo-tree/0.6.4-1 ITP

[Dmitry Bogatov]

> 1. The /source/ package name should actually be undo-tree not
>elpa-undo-tree (team convention to use upstream's name).  Probably
>not worth changing now, but you should be aware of this for packaging
>other addons (you mentioned you wanted to package evil).

Ok. In self-protection can point on elpa-paredit source package.

> 2. Is there some reason you packaged version 0.6.4?  0.6.6 is available.

$ git tag | grep 0.6.6
[empty]

> 3. The homepage in d/control is wrong:
>http://www.dr-qubit.org/git/undo-tree.git is just a git repository,
>not a webpage.  It should be
> (my script
>got this mostly right!).
>I've gone ahead and fixed this in the team repo.

Thanks.

> 4. Please apply a PGP-signed tag to the correct commit in the team
>repo.  Then it will appear correctly on PET.[1]

You mean, I should gpg-tag 'a9e75' as debian/0.6.4-1?

-- 
Accept: text/plain, text/x-diff
Accept-Language: eo,en,ru
X-Keep-In-CC: yes
X-Web-Site: sinsekvu.github.io

Bug#827037: RFS: xpad/4.8.0-1 [ITA]

[JCF Ploemen]

control: tag -1 -moreinfo

> Here's a more complete review.  I hope it is helpful to you.

Sure is, thanks!

 
> I ran debdiff and found that the new version installs
> /usr/share/xpad/help/xpad-user-help.txt.  Shouldn't this be in
> /usr/share/doc/xpad?  If xpad needs to load this file, you could make
> a symlink from /usr/share/xpad/help to /usr/share/doc or add a patch.

Good find. While the file in itself doesn't do any harm as installed by
upstream, it could indeed be useful as documentation. Decided to link
the other way around though, i.e. from /usr/share/doc/xpad to its
current location (in accordance with policy section 12.3).

> You seem to have added an upstream signing key and modified d/watch,
> but I couldn't find mention of this in your changelog entry.

My bad, had to revert some of the initial changes as the pubkey was
nowhere to be found (despite signed releases) and then forgot to
restore the changelog entry after asking upstream about that in
https://answers.launchpad.net/xpad/+question/295164

> It's great you are closing so many bugs :)

All for that warm feeling on rainy days such as this ;)

> >   * Patches:
> > + remove missing.diff, no longer needed.  
> 
> It would be good to say in the changelog why it's no longer needed
> (obsoleted by XX?  applied upstream?).

Unfortunately, that patch came completely undocumented and didn't make
much sense, so I went with a delete-and-see approach here. Could well
be that using autoreconf was all it took to get rid of it. The lack of
info about why it was there in the first place makes it hard to provide
more details beyond the simple notion of being 'no longer needed'.

> >   * Remove Debian menu support as per #741573.  
> 
> That is a very long bug report.  It would be better to refer to the
> section of the Debian Policy that states the deprecation.

This is a standard and well known change implemented in numerous
packages nowadays. Afaik, no section in policy explicitly mentions the
deprecation; the text was just changed to that effect. I'll mention the
ctte decision in d/changelog in the hope of preventing others from
wasting time reading that lengthy bug report but that's about all I can
do here.

> I think you mean "debhelper build dependency".  Bumping the version of
> debhelper means making a release of debhelper...

Obviously. Reworded.
 
> It seems that you ran wrap-and-sort to improve the formatting of the
> control file.  This is usually mentioned in the Debian changelog with
> the options that you passed to wrap-and-sort, e.g.
> 
>* Run wrap-and-sort -abst

Sorted/reordered them alright, never even imagined someone would have
created a tool for that though. I'll add a note to the changelog.

> >   * Docs: don't install AUTHORS, THANKS, TODO (not relevant for end
> > users) or NEWS (superseded by ChangeLog).  
> 
> (It's been discussed above that I think this is unnecessary and has
> the potential to damage relations with upstream; just mentioning it
> here so that this e-mail forms a complete review.)

Just to be clear: I'm not going to let something this trivial stand in
the way of sponsorship. But until then: let's agree to disagree.

> >   * Rules:
> > + simplify to just dh sequencer with autoreconf.
> > + enable all hardening.  
> 
> Are you sure you need the override_dh_installchangelogs target?  I
> thought that dh_installchangelogs automatically tried common names
> like "ChangeLog".

Seems not, removed.


Package on mentors has been updated.


pgpzOOAFEgoRE.pgp
Description: OpenPGP digital signature

Bug#814859: RFS: runescape/0.1-1 [ITP] -- Set in a fantasy world of war, landscapes and sinister powers

[Gianfranco Costamagna]

Hi,




>Corrected the dependencies in d/control


ack
>Made to create a makefile. :/


but why?

[Sergio Durigan Junion]
> metadata stuff

Sergio is correct, thanks!


anyway, it seems working, so please answer the above and I'll do the final 
checks
and hopefully upload.

G.

Bug#825532: RFS: shadowsocks-libev/2.4.6+20160526+ds-1 [ITP]

[Gianfranco Costamagna]

Hi Roger

>Packaging another library (libipset) just takes some time.


I can understand that, and my review will probably take some more time :)
>Technically yes, but since it's a quite old version, I think nobody would do 
>it.
>And currently "shadowsocks" is a python program in Debian archive, we don't 
>recommend to install the old C-written shadowsocks.


wonderful

>So I packaged it and filed ITP#827080 / RFS#827082.>Hope you can also sponsor 
>it.


sure, and I reviewed it

>Thanks for your detailed tips!>I just remove the text after "-delete", others 
>are just as what you said.
>Works fantastic!


:)

>>It's runtime config file, which will be installed to: etc/shadowsocks-libev
>
> not sure about hardcoding passwords that way... maybe generate them at install
> time with apg or similar?
> (postinst or similar)
>
> I don't want all the userbase to have the same tool with the same default 
> password.

>Done.
>Add postinst script to handle this.


>I like that solution!


but shouldn't this be executed only during install?
the place where you put suggests the code will be executed and re-executed 
during all
the calls (configure deconfigure abort upgrade and so on)

I would do it in configure, and don't care about the upgrade, because even 
re-running
it will be a no-op (unless apg generates the same "barfoo!" password :) )
LOL, in that case it will be changed during an apt-get {dist-}upgrade :)

>Hope you cannot find a nitpick this time :-P

there is always something to nitpick :)

e.g. 
https://mentors.debian.net/package/shadowsocks-libev
"W debian-watch-file-should-mangle-version
line 7"

what does it mean?
I admit I didn't investigate it

Gianfranco

Bug#827748: RFS: growl-for-linux/0.8.5-1

[Gianfranco Costamagna]

Hi,
>I am looking for a sponsor for my package "growl-for-linux"


LGTM

Uploaded.

G.

Bug#827082: RFS: libredjackipset/1.1.1+20150311-1 [ITP] -- C library to store sets/maps of IP address

[Gianfranco Costamagna]

control: owner -1 !
control: tags -1 moreinfo

Hi, lets see

1) control: why the -dev package is not multiarch?
sounds like it is multiarch ready to me
(I didn't check but include/* seems to have the same md5sum on different archs


2) the dev package should suggest the doc one


3) d/p/0002-fix-build-with-static-libcheck-library.patch

I can't complain here, I seem to know the author of that patch :p


3) d/p/0004-remove-test-of-python-cram.patch

why? python-cram is available on Debian

4) d/p/0007-*
why? please be more verbose on patches, otherwise I find difficult to 
understand why you are using sphinx instead
of pandoc to build documentation

5) rules:
I don't like hacks
and your sed makes the package non-rebuildable.

maybe overriding the 

dh_install target, and acting under debian/libfoo-dev/usr/include 

is a "better" way (also, a mv instead of ln -s since this old directory should 
go away)

--buildsystem=cmake --builddirectory=build should be useless


also
DPKG_EXPORT_BUILDFLAGS = 1
include /usr/share/dpkg/buildflags.mk


6) did you forward the rename to upstream?

7) libredjackipset-docs.docs
is it useful? there is no package with that name, so probably the file can be 
removed

8) I'm not a fan of dh-exec, and your patch about multiarch seems incomplete

+include(GNUInstallDirs)

this has to be done *before*

"set(CMAKE_INSTALL_LIBDIR lib${LIB_SUFFIX} CACHE STRING"
note the LIB_SUFFIX

and then the install files can change from
usr/lib/lib*.so usr/lib/${DEB_HOST_MULTIARCH}
to
usr/lib/*/*.so
and similar for others

9)

did you try compat level 10?
it should speed up the build by running dh --parallel :)
(you can also experiment with --parallel on the default dh call in rules file)

I guess that's all for now :)

Gianfranco

Bug#827748: RFS: growl-for-linux/0.8.5-1

[Kentaro Hayashi]

Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "growl-for-linux"

 * Package name: growl-for-linux
   Version : 0.8.5-1
   Upstream Author : Yasuhiro Matsumoto 
 * URL : https://mattn.github.io/growl-for-linux/
 * License : BSD-2-clause
   Section : gnome

  It builds those binary packages:

growl-for-linux - Pluggable notification system which supports GNTP

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

  https://mentors.debian.net/package/growl-for-linux


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

dget -x 
https://mentors.debian.net/debian/pool/main/g/growl-for-linux/growl-for-linux_0.8.5-1.dsc

  More information about growl-for-linux can be obtained from 
https://github.com/mattn/growl-for-linux

  Changes since the last upload:

  * New upstream release
  * debian/patches/series
- Removed drop-removed-growl4linuxjpg.patch because it was already
  applied in upstream.
  * debian/patches/disable-pie-for-libgolplug.patch
- Added a patch to disable PIE explicitly for libgolplug not to be
  conflicted with shared libraries under display/*.
  * debian/watch,debian/upstream/signing-key.asc
- Supported cryptographic signature verification.

Regards,


pgpCElXRwWBcc.pgp
Description: PGP signature

Bug#827736: marked as done (RFS: mpd-sima/0.14.1-2 [RC])

[Debian Bug Tracking System]

Your message dated Mon, 20 Jun 2016 12:20:07 + (UTC)
with message-id 
<2132665896.10382063.1466425207854.javamail.ya...@mail.yahoo.com>
and subject line Re: Bug#827736: RFS: mpd-sima/0.14.1-2 [RC]
has caused the Debian Bug report #827736,
regarding RFS: mpd-sima/0.14.1-2 [RC]
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
827736: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=827736
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---

Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "mpd-sima"

  * Package name: mpd-sima
Version : 0.14.1-2
Upstream Author : Jack Kaliko 
  * URL : http://kaliko.me/code/mpd-sima/
  * License : GPL3+
Section : sound

It builds those binary packages:

   mpd-sima   - Automagically add titles to MPD playlist

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


   https://mentors.debian.net/package/mpd-sima

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

   dget -ux 
https://mentors.debian.net/debian/pool/main/m/mpd-sima/mpd-sima_0.14.1-2.dsc


Changes since the last upload (mainly a RC bug fix):

   * Fixed wrong path value for stat overrides. (Closes: #825561)
   * Bumped Standards-Version to 3.9.8, no changes needed
   * Add missing systemd service file documentation key
   * Fixed insecure vcs-field uri



Regards,
Geoff
--- End Message ---
--- Begin Message ---
Hi

>* Fixed insecure vcs-field uri



that commit was bad, missing a /git/ in path.

corrected, changed target from UNRELEASED to unstable, signed, uploaded
and committed on git.

G.--- End Message ---

Bug#827736: RFS: mpd-sima/0.14.1-2 [RC]

[Geoff]

Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "mpd-sima"

  * Package name: mpd-sima
Version : 0.14.1-2
Upstream Author : Jack Kaliko 
  * URL : http://kaliko.me/code/mpd-sima/
  * License : GPL3+
Section : sound

It builds those binary packages:

   mpd-sima   - Automagically add titles to MPD playlist

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


   https://mentors.debian.net/package/mpd-sima

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

   dget -ux 
https://mentors.debian.net/debian/pool/main/m/mpd-sima/mpd-sima_0.14.1-2.dsc


Changes since the last upload (mainly a RC bug fix):

   * Fixed wrong path value for stat overrides. (Closes: #825561)
   * Bumped Standards-Version to 3.9.8, no changes needed
   * Add missing systemd service file documentation key
   * Fixed insecure vcs-field uri



Regards,
Geoff

Bug#824466: RFS: setop/0.1-1 [ITP]

[Gianfranco Costamagna]

Hi,

 Il Sabato 18 Giugno 2016 21:45, Frank Stähr  ha scritto:

>I think we are nearly ready, don’t give up.


I *never* give up :)
>Nevertheless, I don’t see why e. g. boost/algorithm/string/trim.hpp is 
>guaranteed to be installed. It might be a coincidence that it is 
>included by regex or program-options. In my case e. g. libboost1.58-dev 
>was automatically installed together with regex/program-options.
>(But you do not need to explain. I just wanted to exclude an error.)


you gave me a good answer here, so, please add it again then (libboost-dev is 
fine
in this case!)

grep boost . -R |grep include
./src/main.cpp:#include 
./src/main.cpp:#include 
./src/main.cpp:#include 
./src/main.cpp:#include 
./src/main.cpp:#include 



seems also available in boost1.60
>There is a good reason, but I see that it is unnecessarily tortuous to 
>do so. That’s why everything under GPL-2+ now.


if you provide an explanation I can accept it, this is not about you being wrong
and me being right, it is about discussion and accept a common point of view :)


as I did above, I accepted your explanation and asked to restore your solution

>Really?
> says, 
>common licenses may just be refered.


we are talking about copyright in upstream tarball.
I can understand a symlink in debian/copyright, but shouldn't the upstream 
tarball
have a LICENSE file explaining the license text?
https://ftp-master.debian.org/REJECT-FAQ.html
>Thx for all the work and your patience,

thanks for the followup :)


I guess we are mostly ready now!

G.

Bug#820705: marked as done (RFS: subuser/0.5.7-1 [ITP])

[Debian Bug Tracking System]

Your message dated Mon, 20 Jun 2016 10:48:18 + (UTC)
with message-id 
<1604567679.10307054.1466419698699.javamail.ya...@mail.yahoo.com>
and subject line Re: Bug#820704: Bug#820705: RFS: subuser/0.5.6-3 [ITP]
has caused the Debian Bug report #820704,
regarding RFS: subuser/0.5.7-1 [ITP]
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
820704: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=820704
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

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

* Package name: subuser
  Version : 0.5.6-3
  Upstream Author : Timothy Hobbs 
* URL : http://subuser.org
* License : LGPL-3
  Section : admin

It builds those binary packages:

subuser- Run programs on Linux with selectively restricted permissions

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

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


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

dget -x http://mentors.debian.net/debian/pool/main/s/subuser/subuser_0.5.6-3.dsc

More information about hello can be obtained from http://subuser.org.

Changes since the last upload:

subuser (0.5.6-3) unstable; urgency=medium

 * New debian revision
 - Fix bug (closes: #820689)

-- Stanislas Leduc   Mon, 11 Apr 2016 15:11:10 +0200

subuser (0.5.6-1) unstable; urgency=medium

 * New upstream release
 - Add --update option to dev command
 - Fix /dev/dri permissions issues
 - Fix encoding bug when communicating with Docker daemon
 - Improve support for packaging subuser

-- Stanislas Leduc   Mon, 11 Apr 2016 12:37:34 +0200


Best Regards,
Stanislas Leduc
--- End Message ---
--- Begin Message ---
Hi,

>I remove export PYTHON from debian/rules, also python3-setuprools from 
>package, infact not use.
>I use debian/clean for remove subuser.egg-info when i build package from my 
>host build, now i use pbuilder for build package, debian/clean doesn't 
>necessary.
>copyright file update to add missing licences and respective copyright.
>
>codespell and other commands return several warning that wrong words, i see to 
>log that and send them to developer for fix them.
>
>I check for upstream Metadata
>
>New version reflect change of them upload to mentors


uploaded the package on deferred/5, with a little change on rules (attached)

-override_dh_auto_install:
-   python3 setup.py install --root=$(CURDIR)/debian/subuser 
--install-layout=deb
-
override_dh_auto_clean:
-   rm -rf build
-   python3 setup.py clean -a
+   rm -rf subuser.egg-info


in 5 days it will go in new queue, if you don't ask me to cancel the upload.
g.


debdiff
Description: Binary data
--- End Message ---

Bug#820704: marked as done (RFS: subuser/0.5.7-1 [ITP])

[Debian Bug Tracking System]

Your message dated Mon, 20 Jun 2016 10:48:18 + (UTC)
with message-id 
<1604567679.10307054.1466419698699.javamail.ya...@mail.yahoo.com>
and subject line Re: Bug#820704: Bug#820705: RFS: subuser/0.5.6-3 [ITP]
has caused the Debian Bug report #820704,
regarding RFS: subuser/0.5.7-1 [ITP]
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
820704: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=820704
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

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

* Package name: subuser
  Version : 0.5.6-3
  Upstream Author : [fill in name and email of upstream]
* URL : [fill in URL of upstreams web site]
* License : [fill in]
  Section : admin

It builds those binary packages:

subuser- Run programs on Linux with selectively restricted permissions

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

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


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

dget -x http://mentors.debian.net/debian/pool/main/s/subuser/subuser_0.5.6-3.dsc

More information about hello can be obtained from http://www.example.com.

Changes since the last upload:

subuser (0.5.6-3) unstable; urgency=medium

 * New debian revision
 - Fix bug (closes: #820689)

-- Stanislas Leduc   Mon, 11 Apr 2016 15:11:10 +0200

subuser (0.5.6-1) unstable; urgency=medium

 * New upstream release
 - Add --update option to dev command
 - Fix /dev/dri permissions issues
 - Fix encoding bug when communicating with Docker daemon
 - Improve support for packaging subuser

-- Stanislas Leduc   Mon, 11 Apr 2016 12:37:34 +0200


Best Regards,
Stanislas Leduc
--- End Message ---
--- Begin Message ---
Hi,

>I remove export PYTHON from debian/rules, also python3-setuprools from 
>package, infact not use.
>I use debian/clean for remove subuser.egg-info when i build package from my 
>host build, now i use pbuilder for build package, debian/clean doesn't 
>necessary.
>copyright file update to add missing licences and respective copyright.
>
>codespell and other commands return several warning that wrong words, i see to 
>log that and send them to developer for fix them.
>
>I check for upstream Metadata
>
>New version reflect change of them upload to mentors


uploaded the package on deferred/5, with a little change on rules (attached)

-override_dh_auto_install:
-   python3 setup.py install --root=$(CURDIR)/debian/subuser 
--install-layout=deb
-
override_dh_auto_clean:
-   rm -rf build
-   python3 setup.py clean -a
+   rm -rf subuser.egg-info


in 5 days it will go in new queue, if you don't ask me to cancel the upload.
g.


debdiff
Description: Binary data
--- End Message ---

Re: Help with python packaging

[Gianfranco Costamagna]

Hi



>Get it fixed upstream.


other than that, the manpage issue can be "probably" fixed by
"PYTHONPATH=."
before invoking help2man

and of course using ./bin/codespell.py as argument

oh, and I think manpage generation should be done 

in override_dh_installman, not dh_auto_build target.

so, something like

removing the dh_auto_build override

+override_dh_installman:
+   PYTHONPATH=. help2man debian/codespell/usr/bin/codespell.py \
+   --include debian/codespell.1.include \
+   --no-info \
+   --output codespell.1
+   sed -i '/\.SS \"Usage/,+2d' codespell.1
+   dh_installman


should work.

(not sending you a good patch by purpose, sorry!)

also, you need to tell debhelper to install the manpage ;)

man ./codespell.1 seems to return something good to me


G.

Re: Help with python packaging

[Jakub Wilk]

* Peter Spiess-Knafl , 2016-06-20, 11:23:

2.) The script gets installed with the .py suffix, how can I drop this?


Get it fixed upstream.

--
Jakub Wilk

Help with python packaging

[Peter Spiess-Knafl]

Hi Mentors!

I currently maintain codespell [1]. Upstream has dropped the makefile
support which has currently been used for packaging codespell.

Now I am trying to switch to pybuild but I am facing two problems:

1.) How can I invoke help2man in a pybuild based package?
2.) The script gets installed with the .py suffix, how can I drop this?

My current approach is pushed to the dev branch at:

http://anonscm.debian.org/cgit/collab-maint/codespell.git/log/?h=dev


Thank you and greetings,
Peter


[1]: http://anonscm.debian.org/cgit/collab-maint/codespell.git/

Bug#827685: RFS: elpa-undo-tree/0.6.4-1 ITP

[Sean Whitton]

Hello,

On Mon, Jun 20, 2016 at 07:55:34AM +0900, Sean Whitton wrote:
> As a member of the Emacs packaging team, I've had a look at this, and it
> seems fine.  Thanks Dmitry.

In an attempt to improve the output of dh-make-elpa I'm looking at
Dmitry's package again, and I came up with the following questions and
suggestions.  My apologies for not coming up with these before the
package was uploaded.

1. The /source/ package name should actually be undo-tree not
   elpa-undo-tree (team convention to use upstream's name).  Probably
   not worth changing now, but you should be aware of this for packaging
   other addons (you mentioned you wanted to package evil).

2. Is there some reason you packaged version 0.6.4?  0.6.6 is available.

3. The homepage in d/control is wrong:
   http://www.dr-qubit.org/git/undo-tree.git is just a git repository,
   not a webpage.  It should be
    (my script
   got this mostly right!).

   I've gone ahead and fixed this in the team repo.

4. Please apply a PGP-signed tag to the correct commit in the team
   repo.  Then it will appear correctly on PET.[1]

[1] https://pet.debian.net/pkg-emacsen/pet.cgi

-- 
Sean Whitton


signature.asc
Description: PGP signature

Bug#827715: RFS: muttprint/0.73-5.2 [NMU, RC] -- Pretty printing of mails

[Sean Whitton]

On Mon, Jun 20, 2016 at 07:34:06AM +0200, Rene Engelhard wrote:
> On Mon, Jun 20, 2016 at 02:01:24PM +0900, Sean Whitton wrote:
> > I am looking for a sponsor for an NMU of muttprint, fixing a stretch RC
> > bug (older than 7 days and no maintainer activity).  I use muttprint
> 
> That is not correct. It is only RC since *2* days.
> 
> "Severity set to 'serious' from 'important' Request was from Niels Thykier 
>  to cont...@bugs.debian.org. (Sat, 18 Jun 2016 17:04:31 
> GMT) Full text and rfc822 format available."

Thanks for correcting my misunderstanding, and for fixing the package.

-- 
Sean Whitton


signature.asc
Description: PGP signature

Bug#827037: RFS: xpad/4.8.0-1 [ITA]

[Sean Whitton]

control: tag -1 +moreinfo

Hello again JCF,

Here's a more complete review.  I hope it is helpful to you.

I ran debdiff and found that the new version installs
/usr/share/xpad/help/xpad-user-help.txt.  Shouldn't this be in
/usr/share/doc/xpad?  If xpad needs to load this file, you could make a
symlink from /usr/share/xpad/help to /usr/share/doc or add a patch.

You seem to have added an upstream signing key and modified d/watch, but
I couldn't find mention of this in your changelog entry.

On Sat, Jun 11, 2016 at 03:03:38PM +0200, JCF Ploemen wrote:
>   * New upstream release: (Closes: #805723) (LP: #1525656)
> + fixes markup issue in russian translation. (Closes: #797630)
> + fixes moving xpad window rendering the application unresponsive.
>   (Closes: #614337)
> + includes a configurable wait for the systray. (Closes: #555934)

It's great you are closing so many bugs :)

>   * Patches:
> + remove missing.diff, no longer needed.

It would be good to say in the changelog why it's no longer needed
(obsoleted by XX?  applied upstream?).

>   * Remove Debian menu support as per #741573.

That is a very long bug report.  It would be better to refer to the
section of the Debian Policy that states the deprecation.

>   * Control:
> + add build-depends on libgtk-3-dev, libgtksourceview-3.0-dev,
>   dh-autoreconf.
> + remove build-depends on libgtk2.0-dev, libmagickcore-extra,
>   autotools-dev, imagemagick.
> + add VCS links.
> + change upstream homepage to their launchpad site.
>   * Bump compat and debhelper version to 9.

I think you mean "debhelper build dependency".  Bumping the version of
debhelper means making a release of debhelper...

It seems that you ran wrap-and-sort to improve the formatting of the
control file.  This is usually mentioned in the Debian changelog with
the options that you passed to wrap-and-sort, e.g.

   * Run wrap-and-sort -abst

>   * Docs: don't install AUTHORS, THANKS, TODO (not relevant for end
> users) or NEWS (superseded by ChangeLog).

(It's been discussed above that I think this is unnecessary and has the
potential to damage relations with upstream; just mentioning it here so
that this e-mail forms a complete review.)

>   * Rules:
> + simplify to just dh sequencer with autoreconf.
> + enable all hardening.

Are you sure you need the override_dh_installchangelogs target?  I
thought that dh_installchangelogs automatically tried common names like
"ChangeLog".

>   * Copyright:
> + convert to machine-readable format.
> + update all upstream info.
> + add myself as a copyright holder for the packaging.
>   * Bump Standards-Version to 3.9.8 (from 3.9.3; no further changes).
> 
> 
> Regards.



-- 
Sean Whitton


signature.asc
Description: PGP signature

Bug#827487: bytes-circle_2.2-1_amd64.changes REJECTED

[Adam Borowski]

On Mon, Jun 20, 2016 at 08:00:20AM +, Thorsten Alteholz wrote:
> Hi Roberto,
> 
> shouldn't the license be only GPL-3 instead of GPL-3+? At least the 
> file headers and the website say so ..

As you're the sole author, I guess it'd be better to change the headers
instead, to say "version 3 or higher".

But either way will do.

-- 
An imaginary friend squared is a real enemy.