RFS: python-mplayer

2011-12-18 Thread Olivier Schwander
Dear mentors and Python modules team,

I am looking for sponsorship and reviews for my package
"python-mplayer".

 * Package name: python-mplayer
   Version : 0.7.0-1
   Upstream Author : Darwin M. Bautista
 * URL : http://code.google.com/p/python-mplayer/
 * License : LGPL-3.0
   Section : python

It builds those binary packages:

python-mplayer - Lightweight and dynamic MPlayer wrapper with a Pythonic
API

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

  http://mentors.debian.net/package/python-mplayer

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

  dget -x
  
http://mentors.debian.net/debian/pool/main/p/python-mplayer/python-mplayer_0.7.0-1.dsc

The package is also available from the DPMT repository:

  svn://svn.debian.org/python-modules/packages/python-mplayer/trunk/

or

  http://anonscm.debian.org/viewvc/python-modules/packages/python-mplayer/trunk/

Best regards,

Olivier Schwander


-- 
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20111218133410.ga10...@bugaga.lan



Re: Bug#651962: Fwd: Bug#651962: python-argparse redundant?

2011-12-18 Thread Julian Taylor
On 12/17/2011 05:28 PM, Zbigniew Jędrzejewski-Szmek wrote:
> 
> I think it would be easier from the long-term maintenance point of view
> to use the python-standard-library-argparse.

is argparse really such a heavy maintenance package that its unfeasible
to just keep it?
I'd prefer it to be in the archive for 2.6 _and_ 2.7  until all
dependencies have disappeared (which won't happen before 2.6 removal)

You can't let python2.7 provide it as packages having 2.7 and 2.6
executables will not run anymore for 2.6 as argparse will never get
installed.
Using python (>= 2.7) | python-argparse Depends has the same problem.

Given that we have to maintain argparse until 2.6 removal in any case I
think the least work solution is to just keep argparse for 2.7 too.



signature.asc
Description: OpenPGP digital signature


Re: Bug#651962: Fwd: Bug#651962: python-argparse redundant?

2011-12-18 Thread Zbigniew Jędrzejewski-Szmek

On 12/18/2011 04:31 PM, Julian Taylor wrote:

On 12/17/2011 05:28 PM, Zbigniew Jędrzejewski-Szmek wrote:


I think it would be easier from the long-term maintenance point of view
to use the python-standard-library-argparse.


is argparse really such a heavy maintenance package that its unfeasible
to just keep it?
I'd prefer it to be in the archive for 2.6 _and_ 2.7  until all
dependencies have disappeared (which won't happen before 2.6 removal)

You can't let python2.7 provide it as packages having 2.7 and 2.6
executables will not run anymore for 2.6 as argparse will never get
installed.
Using python (>= 2.7) | python-argparse Depends has the same problem.

Right.

I see that python-argparse has Provides:python2.7-argparse. Python2.7 
could have this too, except that it currently will not change anything.



Given that we have to maintain argparse until 2.6 removal in any case I
think the least work solution is to just keep argparse for 2.7 too.

Yes.

Note that the separate argparse module will not be used at all with 
python2.7: '/usr/lib/python2.7' is before 
'/usr/lib/python2.7/dist-packages' on sys.path.

I'm not sure how much of a problem this will be: we know that
zope will complain (the original bug report), but nothing worse than the 
occasional warning should happen.


Maybe python-argparse package should be modified to skip installation of 
the symlinks for python2.7.


Best,
Zbyszek


--
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4eee104d.8060...@in.waw.pl



Re: RFS: python-mplayer

2011-12-18 Thread Jakub Wilk

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

* Olivier Schwander , 2011-12-18, 14:34:

 http://mentors.debian.net/package/python-mplayer

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

 dget -x
 
http://mentors.debian.net/debian/pool/main/p/python-mplayer/python-mplayer_0.7.0-1.dsc


I'd recommend using the upstream name (mplayer.py) for the source 
package name.


Unless there's a compelling reason not do so, I'd merge both changelog 
entries into one.


What is the "debian/python-module-stampdir" directory for?

Why "Priority: extra"?

It should be "mplayer (>= 1.0~rc3)" rather than "mplayer (>= 1.0rc3)".

Architecture should be "all", not "any".

"${shlibs:Depends}" can be removed from Depends, it's unused.

The module doesn't handle filenames with special characters correctly:
| >>> mplayer.Player('eggs"ham.avi')
| Traceback (most recent call last):
|   File "", line 1, in 
|   File "/usr/lib/python2.7/dist-packages/mplayer/core.py", line 102, in 
__init__
| self.args = args
|   File "/usr/lib/python2.7/dist-packages/mplayer/core.py", line 142, in args
| args = shlex.split(args)
|   File "/usr/lib/python2.7/shlex.py", line 279, in split
| return list(lex)
|   File "/usr/lib/python2.7/shlex.py", line 269, in next
| token = self.get_token()
|   File "/usr/lib/python2.7/shlex.py", line 96, in get_token
| raw = self.read_token()
|   File "/usr/lib/python2.7/shlex.py", line 172, in read_token
| raise ValueError, "No closing quotation"
| ValueError: No closing quotation

It looks like upstream supports also Python 3.X. Please consider 
supporting it in Debian, too (in a separate binary package, 
python3-mplayer).


The comment at the top of debian/rules doesn't make sense.

--
Jakub Wilk


--
To UNSUBSCRIBE, email to debian-python-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/20111218221258.ga6...@jwilk.net