RFS: python-unicodecsv (Closes: #669678)

2012-04-22 Thread Thomas Bechtold
Hi,

I am looking for a sponsor for my package "python-unicodecsv"

 * Package name: python-unicodecsv
   Version : 0.9.0-1
   Upstream Author : Jeremy Dunck
 * URL : https://github.com/jdunck/python-unicodecsv/
 * License : BSD
   Section : python

The package is a drop-in replacement for python2 csv module. There's a
RFP bug #669678 [2]. I would be happy if the package could be managed by
the Python Modules Packaging Team. I'm not sure if i handled the entries
in debian/control correct.


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

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


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

dget -x
http://mentors.debian.net/debian/pool/main/p/python-unicodecsv/python-unicodecsv_0.9.0-1.dsc


Cheers,

Tom



-- 
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/1335097011.2567.24.ca...@salbei.fritz.box



Re: RFS: python-unicodecsv (Closes: #669678)

2012-04-23 Thread Thomas Bechtold
On Mon, 2012-04-23 at 12:32 +0200, Piotr Ożarowski wrote:
> [Thomas Bechtold, 2012-04-22]
> > The package is a drop-in replacement for python2 csv module. There's a
> > RFP bug #669678 [2].
> 
> If you plan to maintain it, please rename the bug to ITP and set yourself
> as bug owner

done.

> > I would be happy if the package could be managed by
> > the Python Modules Packaging Team. I'm not sure if i handled the entries
> > in debian/control correct.
> 
> http://wiki.debian.org/Teams/PythonModulesTeam/HowToJoin

thanks for the info. sent a "request to join" with Alioth.




-- 
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/1335191767.31036.7.ca...@salbei.fritz.box



Re: RFS: python-unicodecsv (Closes: #669678)

2012-04-23 Thread Thomas Bechtold
On Mon, 2012-04-23 at 16:49 +0200, Jakub Wilk wrote:

i fixed most of the comments and uploaded a new version to
mentors.debian.net . see comments below

> "debhelper (>= 7.0.50)" - I think this could be simply "(>= 7)", I don't 
> see anything in your package that would require 7.0.50.

fixed in debian/control

> s/unicode/Unicode/ in the package description.

fixed in debian/control

> Please run tests at build time.

fixed in debian/rules

> The package fails to build:
> | dh clean --with python2
> |dh_testdir
> |dh_auto_clean
> | Traceback (most recent call last):
> |   File "setup.py", line 3, in 
> | from setuptools import setup, find_packages
> | ImportError: No module named setuptools
> | dh_auto_clean: python setup.py clean -a returned exit code 1

fixed in debian/control build-dependencies

> Lintian reports:
> I: python-unicodecsv source: debian-watch-file-is-missing

fixed in debian/watch

> I: python-unicodecsv: extended-description-is-probably-too-short

i don't know what i should write about the package. sorry.


Cheers,

Tom


-- 
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/1335195277.31036.12.ca...@salbei.fritz.box



Re: RFS: python-unicodecsv (Closes: #669678)

2012-04-23 Thread Thomas Bechtold
On Mon, 2012-04-23 at 17:52 +0200, Jakub Wilk wrote:
> * Thomas Bechtold , 2012-04-23, 17:34:
> >>Please run tests at build time.
> >fixed in debian/rules
> 
> It should be override_dh_auto_test, not overwrite_dh_auto_test.
> Please honour DEB_BUILD_OPTIONS=nocheck.

fixed both on mentors.debian.net .
seems that the tests never run. fixed this as well and disabled 2 tests
(see debian/patches) because the tests failed.

> Please test against all supported Python versions, not only the default 
> one.

at build time? how to do this? what are the build dependencies in
debian/control and what should i do in debian/rules ? do you have an
example package maybe?

Cheers,

Tom


-- 
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/1335198730.31036.17.ca...@salbei.fritz.box



Re: RFS: python-unicodecsv (Closes: #669678)

2012-04-25 Thread Thomas Bechtold
i fixed all the stuff (see comments below) and uploaded a new package to
mentors.debian.net . if there are no other issues, i'll add the
debian-dir to the python-modules svn (i'm a team member since some days,
my name is toabctl-guest) and search for a sponsor.

Many thanks for the help, Jakub!

On Mon, 2012-04-23 at 20:03 +0200, Jakub Wilk wrote:
> Lintian now emits:
> W: python-unicodecsv source: debhelper-overrides-need-versioned-build-depends 
> (>= 7.0.50~)
> I: python-unicodecsv source: quilt-patch-missing-description 
> 01disable_float_double_tests.diff

fixed on mentors.debian.net

> * Thomas Bechtold , 2012-04-23, 18:32:
> >>>>Please run tests at build time.
> >>>fixed in debian/rules
> >>It should be override_dh_auto_test, not overwrite_dh_auto_test. Please 
> >>honour DEB_BUILD_OPTIONS=nocheck.
> >
> >fixed both on mentors.debian.net .
> >seems that the tests never run. fixed this as well and disabled 2 tests 
> >(see debian/patches) because the tests failed.
> 
> How about fixing the tests rather than disabling them? :) See the 
> attached patch.

thanks. i forwarded the patch upstream and included the patch in
debian/patches

> >>Please test against all supported Python versions, not only the 
> >>default one.
> >
> >at build time? how to do this? what are the build dependencies in 
> >debian/control
> 
> python-all

done.

> 
> >and what should i do in debian/rules ? do you have an example package maybe?
> 
> Use "pyversions -r" to get list of supported Python versions, then 
> iterate over them. TIMTOWTDI:
> 1) abuse %-rules, see e.g. flufl.enum;
> 2) use a for loop, see e.g. sphinxcontrib-speeling;
> 3) (my favourite:) use xargs, see e.g. python-byteplay.

done with your favourite.


-- 
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/1335415687.2744.5.ca...@salbei.fritz.box



Re: RFS: python-unicodecsv (Closes: #669678)

2012-05-04 Thread Thomas Bechtold
On Fri, 2012-04-27 at 00:39 +0200, Jakub Wilk wrote:
> * Thomas Bechtold , 2012-04-26, 06:48:
> >i fixed all the stuff (see comments below) and uploaded a new package 
> >to mentors.debian.net . if there are no other issues, i'll add the 
> >debian-dir to the python-modules svn
> 
> Please feel free to inject the package even though there are some (minor) 
> issues. :) We turn a blind eye to non-perfect packages in our svn. ;)

done. 

> >thanks. i forwarded the patch upstream and included the patch in 
> >debian/patches
> 
> I think Forwared should be simply:
> 
> Forwarded: https://github.com/jdunck/python-unicodecsv/pull/11
> 
> (The DEP-3 specifications reads: “Any value other than "no" or "not-needed"
> means that the patch has been forwarded upstream. Ideally the value is an URL
> proving that it has been forwarded and where one can find more information
> about its inclusion status.”)
> 
> >>>>Please test against all supported Python versions, not only the 
> >>>>default one.
> >>>at build time? how to do this? what are the build dependencies in 
> >>>debian/control
> >>python-all
> >done.
> 
> ...But you shouldn't have removed "python (>= 2.6.6-3~)", as this 
> version is needed for dh_python2. Alternatively, you could make 
> python-all versioned. (This is not very important, as the required 
> version is available in stable anyway.)

fixed this already in svn.

> What are "build/*" in debian/clean and "rmdir unicodecsv.egg-info | true" 
> in debian/rules for? They look suspicious to me.

when i build the package, the directories "build" and
"unicodecsv.egg-info" are created. i thought i have to remove both
during the clean.


-- 
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/1336164517.11133.2.ca...@salbei.fritz.box



request to join the team

2025-02-11 Thread Thomas Bechtold
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA256


hi,

I would like to help maintaining Debian python packages. I'm currently
interested in python-s3transfer (given that I already help with the AWS
related packages in the cloud-team on salsa) and already submitted MRs
[0][1]. I'm happy to help with other python packages, too.

My salsa login name is: toabctl

I've read the python policy[2] and accept it.

Best,

Tom

[0]
https://salsa.debian.org/python-team/packages/python-s3transfer/-/merge_requests/1
[1]
https://salsa.debian.org/python-team/packages/python-s3transfer/-/merge_requests/2
[2] 
https://salsa.debian.org/python-team/tools/python-modules/blob/master/policy.rst

- -- 
Thomas Bechtold
-BEGIN PGP SIGNATURE-

iQJPBAEBCAA5FiEEHBcWbcVXgEZhiIhRQu5y2sJzGa0FAmerLcEbHHRob21hc2Jl
Y2h0b2xkQGpwYmVybGluLmRlAAoJEELuctrCcxmtpDYP/jaVJnPAQHh5211dX2Et
xbbTTXx5eI3jMMauJTuxP1EymHGr7+d3Ds01xnK4E/QHnt7Ouf2YqRCFzuPkdJ8x
jvaluZ1lu6+NVhUie40IJWmoIhci9z9/xfQijJUzhXGCRyP3g9o5pExXOMnQR4Ke
14yts7tBChsAhHalmkUAiDi4SJ8qMpYAVt3oWs0/EuhIW1K/aMMgkRJ72BerTPf6
ozHv2hbobHvJU/CqkPgNuZyJY/dZI8k998BDSEbIl8OnPBM2hn1cRq0XBA4RCopH
6Y5RkLQzFrtpGcpeB/n773mIeYjYcbTrmUaLxTZhSnvOOySxc+zxWNixvlHA/qNb
jU52ICLDXSMNmst8Ck2kP4XIYMWWtqDVgm6SWxVNt062kf3Pqnruh2/FRRW2YwJM
3oOgktHBGE4XNgcNe0vFA4mzrKiD1Whe/SMajLzLQnjZMxCGASD+EzUD3gV686Gw
RKHLrWeUitNeSwlTkFG29FS/SJPX+9lhCs+MyzKv8kIqiloip+LqwhzaElUiJwEP
8c7I87ZdxoSBYfCKzoIcK9ZzYl8565V8WK2/K2iK1GhW1LgWZLIEY9AzBLzX6sUl
c+xCBe9XpsNStv/xzh4vtmvefLKjdYQor+P/FERmq1fx3sc5EXWlYN/dS3n1uAtU
QMMLKiLbXfzL/Minmrv+N25b
=vEPf
-END PGP SIGNATURE-