Bug#695366: RFS: gringotts/1.2.10-1

2013-01-07 Thread Helmut Grohne
Control: tag -1 + moreinfo

On Fri, Dec 07, 2012 at 04:53:26PM +0100, Jose G. López wrote:
> gringotts (1.2.10-1) unstable; urgency=low
> 
>   * New upstream release.

Please hold back new upstream releases from unstable until after the
freeze and target experimental instead. This would be a small step at
speeding up the wheezy release.

If you change an older changelog entry (1.2.10~pre3-2) please explain
why in the current changelog entry.

>   * Split libgringotts from gringotts source package again. It's an
> independent library and could be useful in other software.

This sounds even more inappropriate during the freeze.

>   * debian/copyright: rewrite to machine-readable format.

It is nice to see machine readability get more adoption. However in this
case human readability seems to have suffered. Your license paragraphs
are very short. Specifically they mention "GPL-2+" which suggests the
"or any later version" which is not included in your license paragraph.
Please either drop the "+" or include the "or any later version" for
consistency.

Indeed without "or any later version" the (binary) package is not
redistributable, because GPL-2 and GPL-3 are incompatible[1]. This is
why I tagged the bug moreinfo. Please remove the tag after fixing this
issue.

>   * debian/rules: rewrite to get rid of unnecessary dependencies and add
> some hardening flags.

This sounds confusing given that you actually disabled hardening:

gringotts-1.2.10/debian/rules:
| # Errors compiling with hardening
| #CFLAGS:=$(shell dpkg-buildflags --get CFLAGS) $(CPPFLAGS)

What is the problem with hardening? Maybe it can be resolved?

Please clean up the following patch. It contains more TODO-ish stuff.

gringotts-1.2.10/debian/patches/fix-mkinstalldirs-old-gettext.patch:
| Description: 
|  TODO: Put a short summary on the line above and replace this paragraph
|  with a longer explanation of this change. Complete the meta-information
|  with other relevant fields (see below for details). To make it easier, the
|  information below has been extracted from the changelog. Adjust it or drop
|  it.
|  .

The following boilerplate can likely be removed:

gringotts-1.2.10/debian/rules:
| # Sample debian/rules that uses debhelper.
| # This file was originally written by Joey Hess and Craig Small.
| # As a special exception, when this file is copied by dh-make into a
| # dh-make output file, you may use that output file without restriction.
| # This special exception was added by Craig Small in version 0.37 of dh-make.

Helmut

[1] http://www.gnu.org/licenses/gpl-faq.html#AllCompatibility


-- 
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/20130107085524.ga22...@alf.mars



Processed: Re: Bug#695366: RFS: gringotts/1.2.10-1

2013-01-07 Thread Debian Bug Tracking System
Processing control commands:

> tag -1 + moreinfo
Bug #695366 [sponsorship-requests] RFS: gringotts/1.2.10-1
Added tag(s) moreinfo.

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


--
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/handler.s.b695366.135754894811097.transcr...@bugs.debian.org



Bug#694776: RFS: sglib/1.0.4-1 [ITP] - dependency-less, header-only data structures in C

2013-01-07 Thread Jakub Wilk

(I don't intend to sponsor this package, sorry.)

* Dima Kogan , 2012-11-30, 00:03:

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


Link to .dsc for the lazy:
http://mentors.debian.net/debian/pool/main/s/sglib/sglib_1.0.4-1.dsc

Typos in doc/index.html.sss:
inplementation -> implementation
occurence -> occurrence
withount -> without
elemens -> elements
usualy -> usually
finaly -> finally
memebr -> member
lenght -> length

Typos in samples/hash.c:
occurences -> occurrences

Typos in samples/rbtree.c:
occurences -> occurrences
paramaters- > parameters

doc/sssi creates temporary files in insecure way. It also contains 
bashisms:


$ checkbashisms doc/sssi
possible bashism in doc/sssi line 31 (should be '.', not 'source'):
source ${tmp}
possible bashism in doc/sssi line 35 (should be '.', not 'source'):
source ${tmp} >> ${output}

Once the above problems are fixed, please rebuild documentation from 
source at build time.


Upstream provides a test suite (samples/selftest.c). Please run it at 
build time.


Wouldn't it make more sense to set Homepage field to
http://sglib.sourceforge.net/ ?

License text in machine-readable copyright file should be indented by 
exactly one space. (More that one is not syntactically wrong, but it's 
likely not what you meant; see Policy §5.6.13 for details.)


"License: GPL-1+ or any OSI-approved license" is not syntactically valid 
in DEP-5 copyright files, as license names cannot contains spaces.


The upstream's "license conditions" leaves a lot to be desired... They 
wrote:


License Conditions: You can use a verbatim copy (including this 
copyright notice) of sglib freely in any project, commercial or not. 
You can also use derivative forms freely under terms of Open Source 
Software license or under terms of GNU Public License.


What does "use" mean? For example, Does it include (re-)distributing the 
software?

Is "GNU Public License" supposed to mean "GNU General Public License"?
What does "Open Source Software license" mean?
Texts of none of the mentioned licenses are included in the tarball, so 
how do I know which rights have been granted and on what conditions?


If you need to use a derivative form in a commercial project, or you 
need sglib under any other license conditions, contact the author.


Does it mean that I can't "use" the software under the licenses 
mentioned above in "commercial" projects without contacting the author?


(I'm purposefully ignoring what upstream wrote on the homepage; it's not 
particularly unambiguous either.)


--
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/20130107181858.ga9...@jwilk.net



Re: Bug#694778: RFS: tcpflow/1.3.0-1 [ITA] TCP flow recorder

2013-01-07 Thread Jakub Wilk

* Paul Gevers , 2013-01-06, 15:21:
you bump the Standards-Version but do not mention which (if any) 
changes were needed for that.

In 99% cases, there are no changes needed,


93% of all statistics are made up.

so it's pointless to spam the changelog, and if some changes did 
indeed occur, they are not really related to the policy itself (merely 
required by it).


Noting policy bumps had some miniscule reason before lintian started 
shouting, but nowadays, it is NOT bumping that could be notable.


Yes, normally no changes are needed. But people that we sponsor need to 
know that they have to CHECK if they can update the standard. They can 
express this knowledge by writing in the changelog that they did not 
need to make any changes. So I am in favor of having this in the 
changelog.


ACK. Whenever I see:

|  * Bump Standards-Version to 3.4.5.

I read it as:

|  * Bump Standards-Version to 3.4.5, because Lintian yelled at me;
|but actually I have no idea what I'm doing.

--
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/20130107184957.ga...@jwilk.net



Bug#697627: RFS: gogglesmm/0.12.7-1

2013-01-07 Thread Hendrik Rittich
Package: sponsorship-requests
Severity: normal

Dear mentors,

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

 * Package name: gogglesmm
   Version : 0.12.7-1
   Upstream Author : Sander Jansen 
 * URL : http://code.google.com/p/gogglesmm/
 * License : GPLv3
   Section : sound

It builds those binary packages:

  gogglesmm  - Goggles Music Manager

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

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

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

  dget -x 
http://mentors.debian.net/debian/pool/main/g/gogglesmm/gogglesmm_0.12.7-1.dsc

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

Changes since the last upload:

  * New upstream release, closes: #697587
  * Updated build dependencies
  * Added the use of buildflags.mk

Regards,
  Hendrik Rittich



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


Help in packaging gedit-classbrowser3g

2013-01-07 Thread B. Clausius
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi,

Package : gedit-classbrowser3g
Version : 1.0.2
Upstream Author : B. Clausius 
URL : https://launchpad.net/gedit-classbrowser3g
License : GPL-3+
Programming Lang: Python

It builds the binary package:
gedit-classbrowser3g-plugin - Class Browser for gedit

Packaging in PAPT svn repo:
svn://svn.debian.org/svn/python-apps/packages/gedit-classbrowser3g

Am 11.05.2012 15:20, schrieb Piotr Ożarowski:
> [B. Clausius, 2012-05-11]
>> I intend to package a gedit plugin (written in Python). Is the
>> Python Applications Packaging Team ok for that?
> 
> sure it is. Please consider asking Debian GNOME Maintainers¹ if
> they're interested in co-maintaining it, though (they know more
> about gedit than we do so)
> 
> [¹] i.e. gedit and gedit-plugins' maintainers;
> pkg-gnome-maintain...@lists.alioth.debian.org
> 

Maybe someone familiar with gedit/gedit-plugins could give feedback
whether the packaging satisfies the requirements of gedit plugins in
Debian.

The similar plugin gedit-source-code-browser-plugin is already in
Debian, but gedit-classbrowser3g has additional features. See the ITP
bug http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=672484 for a
comparison.

BC
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQIcBAEBAgAGBQJQ60cxAAoJEEDg0A3D0GN4YW0QALWlC2pXD5Bl6mWQsbSNJffN
f+2FcYqcl8UBTdmL+kd/yJT+0jNYjeJVgcqk6gBb7n75xTiVTzZOJEvRfYDmgCTi
KT+qjP6l7O2ydgtnaKFByE3pwizwFzxvmKqSW6F1s1uhMBw6MvBfCg4C6vn8+TFJ
1o430yedjwZAM00p9ZMyXqNqIWayrBUieotc8HREomyNmCyFCkykfD85QCohW4Pn
VkeH82w9Jjw491PapG2xjs/mu8w2GKUFX00RVltNt8MSkCNAx5vVEp4OwuLpnt1l
/n4n03paqrJ4kj6CJBLG73WrljhWqnU8QXIFtmo6ucDuA0JvN8Gl4Hp0XG/NuSm4
qmPwR/qDKI6fL+CofBGrlz+UpP+8/NA4X7vdc/kdKuw8eN1muxCy7qyf2RyVgEdz
dtlKTUw6dEyWonDEYsBwHlg0sXWIafJrzvgUI8hCMsk8xfkiJhI4W5rZ6a104wOo
1rCinxfp9Xq+sle6ZDpuDUJ/DPwTZ3f6XFNkINXvL8TcWcsw/QxZmy5/hCH0eiBi
bu4G+yuW/ZBAWkr/ZXzFuTfHbnWB0tEzIXnXHMzCqVu3wTgYiZ/pxNjDXVXRspo3
UraDMYf/n424JPZyIoojHjHFqljF/ZYqRBqvvb5GAXPSNSa79+Zxdbaj4QmWOYzS
QgnOof4JppqxwCNuOVFk
=HYbU
-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/50eb4731.6050...@gmx.de



Help in packaging gedit-projects

2013-01-07 Thread B. Clausius
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Package name : gedit-projects
Version : 1.1.1
Upstream Author : B. Clausius 
URL : https://launchpad.net/gedit-projects
License : GPL-3+
Programming Lang: Python

It builds the binary package:
gedit-projects-plugin - Manage projects in gedit

Packaging in PAPT svn repo:
svn://svn.debian.org/svn/python-apps/packages/gedit-projects

Am 11.05.2012 15:20, schrieb Piotr Ożarowski:
> [B. Clausius, 2012-05-11]
>> I intend to package a gedit plugin (written in Python). Is the
>> Python Applications Packaging Team ok for that?
> 
> sure it is. Please consider asking Debian GNOME Maintainers¹ if
> they're interested in co-maintaining it, though (they know more
> about gedit than we do so)
> 
> [¹] i.e. gedit and gedit-plugins' maintainers;
> pkg-gnome-maintain...@lists.alioth.debian.org
> 

Maybe someone familiar with gedit/gedit-plugins could give feedback
whether the packaging satisfies the requirements of gedit plugins in
Debian.

BC
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with undefined - http://www.enigmail.net/

iQIcBAEBAgAGBQJQ60fIAAoJEEDg0A3D0GN4WcIQAIhRcCiJh9VUryzP/miEnvQY
ifUHAK1Ztkw9SFtWrdfvhW0V2aPTWGfnR8W7R7p5c6YTTDgcEfhoGBc9ONsBrwD3
APXpDyZ4rH5Jb430H3uSuFiWD3fLJFy1LR7G71Ma3d1s7xeozK3RNGCMDio4WSfc
iN0aufXId6a6Ao3RchS1l1UyR4cXllrJ9rS99hQvL0Vp/L1ZbGcvH3kaylJVlQcZ
hLlWZu12KCkUa8y060K6PD+36SG/K4ZYzhZ8UnR73KrI7VqVJMMQqDKcZqWK5aYp
HRQiXRcmZs8TQM2FxiIE2fztf0NyWWJ+8crq8pnvWozOONHN0v35tjbu7LtV41Rk
0l8AbTs+9WKaspRF2kG/VPEGnsVrj1WfgcK7BfCyBwzDywv/14BtnPifNzqXElJ2
DfOr7R/LIJzCb0tmeGWv+ruHwWHWo5GOlgcJWSS0SvSVyTC75/H+w7VPTq1mMuLV
NEH1E2xAOxRNSfgKZyPHIGmm/c//+jQ0tF17nbWb0ErFHSIUtWdjk0UR3AUw15n+
mISnVZ9aILGaKzvBe5Cea4zfY7nZ/oMS4GtAUPFAeaD6p4SSiDCNcXQvcwNHzvYc
sKOxn13LmNOwg2wGo/U1O266ae4GXH4KfCwtQYta0eVDvkg/xik0jTnO9/QZYcq/
mErPbh7ozLsb66QrGNLR
=Y0LA
-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/50eb47c8.5010...@gmx.de



Bug#695366: RFS: gringotts/1.2.10-1

2013-01-07 Thread Jose G . López
El lun, 07-01-2013 a las 09:55 +0100, Helmut Grohne escribió:
> Please hold back new upstream releases from unstable until after the
> freeze and target experimental instead. This would be a small step at
> speeding up the wheezy release.

Seems reasonable. I was worry by duplicities. Changed to target
experimental.

> If you change an older changelog entry (1.2.10~pre3-2) please explain
> why in the current changelog entry.

Sorry, unwanted changes. Reverted.

> >   * Split libgringotts from gringotts source package again. It's an
> > independent library and could be useful in other software.
> 
> This sounds even more inappropriate during the freeze.
> 
> >   * debian/copyright: rewrite to machine-readable format.
> 
> It is nice to see machine readability get more adoption. However in this
> case human readability seems to have suffered. Your license paragraphs
> are very short. Specifically they mention "GPL-2+" which suggests the
> "or any later version" which is not included in your license paragraph.
> Please either drop the "+" or include the "or any later version" for
> consistency.
> 
> Indeed without "or any later version" the (binary) package is not
> redistributable, because GPL-2 and GPL-3 are incompatible[1]. This is
> why I tagged the bug moreinfo. Please remove the tag after fixing this
> issue.

I have doubts here. I changed to GPL2+ to be compatible but previous
package (Wesley's) was released as GPL3 and I'm not sure if I can
relicense debian files as GPL2+. Completed the license paragraph, too.

> >   * debian/rules: rewrite to get rid of unnecessary dependencies and add
> > some hardening flags.
> 
> This sounds confusing given that you actually disabled hardening:
> 
> gringotts-1.2.10/debian/rules:
> | # Errors compiling with hardening
> | #CFLAGS:=$(shell dpkg-buildflags --get CFLAGS) $(CPPFLAGS)
> 
> What is the problem with hardening? Maybe it can be resolved?

True, a forgotten entry ... At that time I couldn't fix it but today I
looked at it more deeply and I have created a patch to allow compile
with hardening build flags.

> Please clean up the following patch. It contains more TODO-ish stuff.
> 
> gringotts-1.2.10/debian/patches/fix-mkinstalldirs-old-gettext.patch:
> | Description: 
> |  TODO: Put a short summary on the line above and replace this paragraph
> |  with a longer explanation of this change. Complete the meta-information
> |  with other relevant fields (see below for details). To make it easier, the
> |  information below has been extracted from the changelog. Adjust it or drop
> |  it.
> |  .

Done.

> The following boilerplate can likely be removed:
> 
> gringotts-1.2.10/debian/rules:
> | # Sample debian/rules that uses debhelper.
> | # This file was originally written by Joey Hess and Craig Small.
> | # As a special exception, when this file is copied by dh-make into a
> | # dh-make output file, you may use that output file without restriction.
> | # This special exception was added by Craig Small in version 0.37 of 
> dh-make.

Cleaned, too.

> Helmut

Thank you very much for your comments, they are very welcome.
Re-uploaded to mentors.

Regards




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


Bug#667904: RFS: mitlm/0.4-1 [ITP] -- MIT Language Modeling toolkit

2013-01-07 Thread Jakub Wilk
Does debian/patches/compilation_fixes really fix only "compilation on 
Debian systems"?


I see several problems with the __fls function:
- You can't use "__" in your own identifiers; such names are reserved.
- The implementation uses x86 assembly.
- The unsigned long -> int typecast looks redundant to me.

Typo in src/optimize/lbfgs.f: THE THE -> THE.
Typo in src/optimize/lbfgsb.f: easible -> feasible.

--
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/20130108010649.ga2...@jwilk.net



Bug#689692: marked as done (RFS: byzanz/0.3.0+git20120930-1 [ITA])

2013-01-07 Thread Debian Bug Tracking System
Your message dated Tue, 08 Jan 2013 04:20:25 +
with message-id 
and subject line closing RFS: byzanz/0.3.0+git20120930-1 [ITA]
has caused the Debian Bug report #689692,
regarding RFS: byzanz/0.3.0+git20120930-1 [ITA]
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.)


-- 
689692: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=689692
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 "byzanz" which i would like to
adopt. I am also interested in maintaining it in a git repository at
git.debian.org.

Package name: byzanz
Version : 0.3.0+git20120930-1
Upstream Author : Benjamin Otte 
URL : http://git.gnome.org/browse/byzanz
License : GPL-3+, LGPL-3+, Public Domain
Section : graphics

It builds those binary packages:

byzanz - small screencast creator

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

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


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

dget -x
http://mentors.debian.net/debian/pool/main/b/byzanz/byzanz_0.3.0+git20120930-1.dsc


Changes since the last upload:

  * New Maintainer. (Closes: #654614)
  * New Upstream git snapshot which adds various translations.
  * Switch to packaging format 3.0 (quilt).
  * Create patches instead of modifying the sources.
- Remove config.sub and config.guess and add them automatically with
  autotools-dev.
- Incorporate the last NMUs and create proper patches for them.
- Add patches from Jari Aalto again which were overwritten by the
  last NMUs.
  * debian/rules:
- Simplify debian/rules by using dh.
- Build with hardening+=all.
- Build with -Wl, --as-needed to avoid unnecessary dependencies.
  * debian/control:
- Add autotools-dev to Build-Depends.
- Homepage field: Point to the git repository because the old
  homepage does not exist anymore.
- Remove the VCS-field because it points to the upstream repository.
  (Closes: #685358)
- Description: Make clear that Byzanz is an applet and command line
  tool.
- New Standards-Version 3.9.4, no changes needed.
  * Bump compat level to 9.
  * Add a short introductory manpage for Byzanz which points to
byzanz-record and byzanz-playback. (Closes: #606673)
  * Improve README.Debian and make clear where you can find the panel
applet. Create examples in case you can only use the command line
tools.(Closes: #606676)
  * Fix the wrong command in byzanz-playback's synopsis.
(Closes: #641663)
  * Update debian/copyright to copyright format 1.0.
  * Update the license to GPL-3+, LGPL-3+ and Public Domain where
necessary.


Regards,
Markus Koschany



signature.asc
Description: OpenPGP digital signature
--- End Message ---
--- Begin Message ---
Package byzanz has been removed from mentors.--- End Message ---