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



Re: RFS: libengsas packages

2013-02-25 Thread Игорь Пашев
2013/2/25 Joachim Langenbach joachim.langenb...@engsas.de:
 Without the dependencies to binary packages defined in this package (like
 dependency of libengsas0-common from libengsas0), right?


I think libengsas0-common should be kept.
Also, probably 0 should be removed from its name:
libengsas0-common - libengsas-common


-- 
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/call-q8xrijmkpbyw4px7u9zkofhzs0tnj98e72xkdpgujhj...@mail.gmail.com



Re: RFS: libengsas packages

2013-02-25 Thread Andrew Shadura
Hello,

 6) Copyright-file must be in DEP-5 format.

Please don't say 'must' when you're speaking about what's actually a
recommendation, not a rule. Thanks.

-- 
WBR, Andrew


-- 
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/camb-mayx1a41oqr2yueutjwossklrjqm2k+txw2cnqfaz6a...@mail.gmail.com



Re: RFS: libengsas packages

2013-02-25 Thread Anton Gladky
Hi,


On 02/25/2013 07:52 AM, Joachim Langenbach wrote:
 1) changelog says, that you are targeting the software to stable
 release. It is impossible. Use unstable instead. Also you should put
 your name there.
 
 So this stable or unstable line refers to the distribution status the package 
 is build for? Like build for squeeze is stable and build for sid is unstable?

Please, check that.

http://www.debian.org/doc/manuals/maint-guide/dreq.en.html#changelog

 5) Depends-section contains a lot of libraries, they should be
 automatically picked up due to dh_shlibdeps, if it is possible.
 
 Without the dependencies to binary packages defined in this package (like 
 dependency of libengsas0-common from libengsas0), right?

Please, check that:

http://www.debian.org/doc/manuals/maint-guide/advanced.en.html#library

Игорь is right,  libengsas0-common should be kept.

For example. there is a list of of dependencies auto-generated by
dh_shlibdeps  for this library:

libc6 (= 2.2.5), libengsas0, libgcc1 (= 1:4.1.1), libqt4-network (=
4:4.5.3), libqt4-sql (= 4:4.5.3), libqt4-xml (= 4:4.5.3),
libqt4-xmlpatterns (= 4:4.5.3), libqtcore4 (= 4:4.7.0~beta1),
libqtgui4 (= 4:4.5.3), libstdc++6 (= 4.1.1)

 7) Remove postinst and postrm scripts. Review other ones.
 
 I had added them, because lintian complains, that they are not there 
 otherwise, is this wrong? Or do you mean a specific one?

Some of your scripts are empty. They should be removed.
Other ones should be reviewed. If you do not really need them - remove them,

Cheers,

Anton



signature.asc
Description: OpenPGP digital signature


Re: RFS: libengsas packages

2013-02-25 Thread Anton Gladky
On 02/25/2013 05:37 PM, Andrew Shadura wrote:
 Hello,
 
 6) Copyright-file must be in DEP-5 format.
 
 Please don't say 'must' when you're speaking about what's actually a
 recommendation, not a rule. Thanks.
 

Hello,

ok, agree, my typo. But I think it is desirable to ask for DEP-5 for
new uploads.

Thanks,

Anton



signature.asc
Description: OpenPGP digital signature


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

2013-02-25 Thread Anton Gladky
Hi,

On 02/25/2013 01:48 PM, Scott Leggett wrote:
 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?

New policy versions are announced on debian-devel-announce [1].
Not sure, where new compat-level is announced. Now you can find the
current version in maint-guide [2].

[1] http://lists.debian.org/debian-devel-announce/2012/09/msg6.html
[2] http://www.debian.org/doc/manuals/maint-guide/dother.en.html#compat

 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.

If you use git, please, consider using pristine-tar technique next time.

Good job! Uploaded.

Anton



signature.asc
Description: OpenPGP digital signature


Bug#701668: RFS: medit/1.1.1-1 [ITA] -- Useful programming and around-programming text editor

2013-02-25 Thread Jose G . López
Package: sponsorship-requests
Severity: wishlist

Dear mentors,

I am looking for a sponsor for my package medit

* Package name: medit
  Version : 1.1.1-1
  Upstream Author : Yevgen Muntyan emunt...@users.sourceforge.net
Daniel Poelzleithner mooe...@poelzi.org
* URL : http://mooedit.sourceforge.net/
* License : GNU Lesser General Public License
  Section : editors

It builds those binary packages:

medit - Useful programming and around-programming text editor

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

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

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

dget -x http://mentors.debian.net/debian/pool/main/m/medit/medit_1.1.1-1.dsc

Changes since the last upload:

medit (1.1.1-1) experimental; urgency=low

  * New maintainer (Closes: #575203).
  * New upstream release.
  * debian/compat: update to 9.
  * debian/control:
- Update to debhelper 9.
- Bump to Standards-Version 3.9.4. No changes required.
- Add dh-autoreconf to Build-Depends to update the build system.
  * debian/rules:
- Add '--parallel' compilation.
- Rewrite to get rid of unnecessary dependencies and add hardening flags.
  * Add fix_package_name.patch to change program name from medit-1 to medit.
  * Add doc-base control files to register documentation with doc-base.
  * Add a menu file.

Regards,
Jose G. López


signature.asc
Description: This is a digitally signed message part


Bug#700360: RFS: openfst/1.3.3-1 -- weighted finite-state transducers library

2013-02-25 Thread Jakub Wilk

* Giulio Paci giuliop...@gmail.com, 2013-02-23, 19:37:
I just sent another email further explaining the issue and pointing out 
it is a general issue.


Thanks.


I think this change
  * Move plugins from /usr/lib/fst/ to /usr/lib/fst/1/.
warrants updating README.Debian.

--
Jakub Wilk


--
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/20130225225008.ga5...@jwilk.net



Bug#701679: RFS: phonetisaurus/0.7.8-2 -- Grapheme to Phoneme conversion tool

2013-02-25 Thread Giulio Paci
Package: sponsorship-requests
  Severity: wishlist

  Dear mentors,

  I am looking for a sponsor for my package phonetisaurus

 * Package name: phonetisaurus
   Version : 0.7.8-2
   Upstream Author : Josef Novak josef.robert.no...@gmail.com
 * URL : http://code.google.com/p/phonetisaurus/
 * License : BSD-2-clause
   Section : science


  It builds those binary packages:

phonetisaurus - Grapheme to Phoneme conversion tool

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

 git://anonscm.debian.org/collab-maint/phonetisaurus.git

  Regards,
   Giulio Paci


-- 
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/512c08fb.9020...@gmail.com



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

2013-02-25 Thread Debian Bug Tracking System
Your message dated Tue, 26 Feb 2013 04:20:26 +
with message-id e1uac1i-0006mo...@quantz.debian.org
and subject line closing RFS: spout/1.4-1 [ITA] -- Tiny abstract black and 
white 2D cave-shooter
has caused the Debian Bug report #701543,
regarding RFS: spout/1.4-1 [ITA] -- Tiny abstract black and white 2D 
cave-shooter
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.)


-- 
701543: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=701543
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
---BeginMessage---
-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-
---End Message---
---BeginMessage---
Package spout version 1.4-1 is in experimental now.
http://packages.qa.debian.org/spout---End Message---


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



Re: Watch file warning from mentors website

2013-02-25 Thread Bart Martens
On Tue, Feb 26, 2013 at 06:09:01PM +1100, Scott Leggett wrote:
 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

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.

 
 
 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).

You can ignore that.  I guess mentors doensn't check for comments-only watch
files.

 
 Is there something I can fix in the watch file,

At first site upstream doesn't support uscan, so no.

 or is the mentors website complaining unnecessarily?

In my opinion, yes.

 It seems strange that the website would complain while lintian does not.

I guess mentors doesn't use lintian to check the watch file, hence the
different behavior.

Regards,

Bart Martens


-- 
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/20130226075614.ga29...@master.debian.org