Re: RFS: bino

2011-10-02 Thread Dominique Dumont 
Le Wednesday 28 September 2011 19:04:51, Daniel Schaal a écrit :
 I also bumped the debhelper compat to 9 to get the values from
 dpkg-buildflags
 
 New package uploaded to [2]

Reviewed, rebuilt, rebuilt in a chroot, tested, tested with piuparts and 
uploaded.

Now we wait for FTP master to do their work.

All the best

Dominique
--
http://config-model.wiki.sourceforge.net/ -o- http://search.cpan.org/~ddumont/
http://www.ohloh.net/accounts/ddumont -o- http://ddumont.wordpress.com/


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

Re: RFS: bino

2011-10-02 Thread Daniel Schaal 
Am Sonntag, 2. Oktober 2011, 18:48:21 schrieb Dominique Dumont:
 Reviewed, rebuilt, rebuilt in a chroot, tested, tested with piuparts and
 uploaded.
 
 Now we wait for FTP master to do their work.

Thanks!

Daniel Schaal


-- 
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/13584259.8s9zUK2PV2@daniels-laptop

Re: RFS: bino

2011-09-28 Thread Dominique Dumont 
Le Wednesday 14 September 2011 06:44:11, Daniel Schaal a écrit :
 I uploaded an updated package to http://mentors.debian.net/package/bino

Here are some comments:

- debian/copyright: In fact the simple All-Permissive License you mention 
  is very close to BSD 2-clause license [1]. To respect Dep-5 syntax, you 
  should use a short name in license field. I.e not:

   Files: pkg/macosx/*
   Copyright: 2011 Frédéric Devernay frederic.dever...@inrialpes.fr
   License: Simple All-Permissive License

 But, IMHO, something like this (also easier to review):

   Files: pkg/macosx/*
   Copyright: 2011 Frédéric Devernay frederic.dever...@inrialpes.fr
   License: BSD-2-clause-like

- debian/control: 'config-edit -appli dpkg -ui none' reports some issues 
  on some versioned dependencies:

  Warning in 'control source Build-Depends:3' value 'libavcodec-dev (= 0.7)': 
unnecessary versioned dependency: 
= 0.7. Debian has etch-security - 0.cvs20060823-8+etch1; etch - 
0.cvs20060823-8+etch1; lenny - 
0.svn20080206-18+lenny1; lenny-p-u - 0.svn20080206-18+lenny3; lenny-security 
- 0.svn20080206-18+lenny3; squeeze 
- 4:0.5.2-6; squeeze-p-u - 4:0.5.4-1; squeeze-security - 4:0.5.4-1; wheezy 
- 4:0.7.1-5; sid - 4:0.7.1-7;
  Warning in 'control source Build-Depends:4' value 'libavformat-dev (= 0.7)': 
unnecessary versioned dependency: 
= 0.7. Debian has etch-security - 0.cvs20060823-8+etch1; etch - 
0.cvs20060823-8+etch1; lenny - 
0.svn20080206-18+lenny1; lenny-p-u - 0.svn20080206-18+lenny3; lenny-security 
- 0.svn20080206-18+lenny3; squeeze 
- 4:0.5.2-6; squeeze-p-u - 4:0.5.4-1; squeeze-security - 4:0.5.4-1; wheezy 
- 4:0.7.1-5; sid - 4:0.7.1-7;
 Warning in 'control source Build-Depends:5' value 'libavdevice-dev (= 0.7)': 
unnecessary versioned dependency: 
= 0.7. Debian has lenny - 0.svn20080206-18+lenny1; lenny-p-u - 
0.svn20080206-18+lenny3; lenny-security - 
0.svn20080206-18+lenny3; squeeze - 4:0.5.2-6; squeeze-p-u - 4:0.5.4-1; 
squeeze-security - 4:0.5.4-1; wheezy - 
4:0.7.1-5; sid - 4:0.7.1-7;

  In these cases, the versioned dependency is useless if it does not 
  include the epoch (here the '4:' version prefix)

  [ config-edit also complains about 
'https://gitorious.org/farbing-packages/bino' 
   not matching regexp ^http://.. Ahem... Ooops ... That's a config-edit bug 
that I'm 
   going to fix ASAP :-p ]

- lintian -i -I ../bino_1.2.0-1_amd64.deb reports:
  I: bino: hyphen-used-as-minus-sign usr/share/man/man1/bino.1.gz:201
  I: bino: hyphen-used-as-minus-sign usr/share/man/man1/bino.1.gz:203
  I: bino: possible-documentation-but-no-doc-base-registration

  Is it possible to fix these issues ?

Otherwise, the package looks good. Good work :-)

Dominique

[1] http://www.opensource.org/licenses/bsd-license.php
--
http://config-model.wiki.sourceforge.net/ -o- http://search.cpan.org/~ddumont/
http://www.ohloh.net/accounts/ddumont -o- http://ddumont.wordpress.com/


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

Re: RFS: bino

2011-09-28 Thread Daniel Schaal 
Am Mittwoch, 28. September 2011, 17:22:35 schrieb Dominique Dumont:
 Le Wednesday 14 September 2011 06:44:11, Daniel Schaal a écrit :
  I uploaded an updated package to http://mentors.debian.net/package/bino
 
 Here are some comments:

Thanks for the review.

 - debian/copyright: In fact the simple All-Permissive License you mention
   is very close to BSD 2-clause license [1]. To respect Dep-5 syntax, you
   should use a short name in license field. I.e not:

It seems the license text comes from [1], which calls it All-permissive 
Copying License, but I changed it to BSD-2-clause-like as it is probably 
clearer.

 
 - debian/control: 'config-edit -appli dpkg -ui none' reports some issues
   on some versioned dependencies:
 
   Warning in 'control source Build-Depends:3' value 'libavcodec-dev (= 
 0.7)': unnecessary versioned dependency:
 = 0.7. Debian has etch-security - 0.cvs20060823-8+etch1; etch -
 0.cvs20060823-8+etch1; lenny -

Fixed Build-Depends to also include the epoch.

 - lintian -i -I ../bino_1.2.0-1_amd64.deb reports:
   I: bino: hyphen-used-as-minus-sign usr/share/man/man1/bino.1.gz:201
   I: bino: hyphen-used-as-minus-sign usr/share/man/man1/bino.1.gz:203
   I: bino: possible-documentation-but-no-doc-base-registration
 
   Is it possible to fix these issues ?

Added patch to escape hyphens in man page (will see if I can get this 
upstream), and added a doc-base control file.

I also bumped the debhelper compat to 9 to get the values from dpkg-buildflags

New package uploaded to [2]

Thanks,
Daniel Schaal

[1] http://www.gnu.org/s/hello/manual/texinfo/All_002dpermissive-Copying-
License.html
[2] http://mentors.debian.net/package/bino


--
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/19959022.aNVSps6yA3@daniels-laptop

Re: RFS: bino

2011-09-13 Thread Johan Van de Wauw 
Daniel,

I am not a debian developer, so I can not upload your package. However
I have checked it contents and it looks very good: I hope this message
encourages sponsors to check your package.

One comment: you require libswscale =0.14 to build, which is present
in unstable but not (yet) in testing.
It may be useful to specify the version needed in debian/control:
libswscale-dev (=4.0.7) [or whatever is more correct]

a little nitpicking:
There is a double @@ symbol in line 56 of your debian/copyright file

Johan

On Sun, Sep 11, 2011 at 12:47 PM, Daniel Schaal farb...@web.de wrote:
 Dear mentors,

 I am looking for a sponsor for my package bino.

  * Package name    : bino
   Version         : 1.2.0-1
   Upstream Author : Martin Lambers mar...@marlam.de
  * URL             : http://bino3d.org
  * License         : GPL-3.0+
   Section         : video

 It builds those binary packages:

 bino  - 3D video player
  bino-dbg   - debugging symbols for bino 3D video player

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

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

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

  dget -x http://mentors.debian.net/debian/pool/main/b/bino/bino_1.2.0-1.dsc

 I would be glad if someone uploaded this package for me.

 Kind regards,

 Daniel Schaal


 --
 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/1429545.snOU8C3YbQ@daniels-laptop




--
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/CAJOp35nNGhRHi4vXMfEZX4RHXcG0r8N5-a1=ytxnkpb6_0b...@mail.gmail.com

Re: RFS: bino

2011-09-13 Thread Daniel Schaal 
Am Dienstag, 13. September 2011, 21:58:42 schrieb Johan Van de Wauw:
 Daniel,
 
 I am not a debian developer, so I can not upload your package. However
 I have checked it contents and it looks very good: I hope this message
 encourages sponsors to check your package.

Thanks for the review.

 One comment: you require libswscale =0.14 to build, which is present
 in unstable but not (yet) in testing.
 It may be useful to specify the version needed in debian/control:
 libswscale-dev (=4.0.7) [or whatever is more correct]

I changed it to build-depend on libav{codec|format..} = 0.7
 
 a little nitpicking:
 There is a double @@ symbol in line 56 of your debian/copyright file

Fixed.

I uploaded an updated package to http://mentors.debian.net/package/bino

Thanks,
Daniel Schaal


-- 
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/2076643.TDiVt2J0JZ@daniels-laptop