[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: RFS: python-twitter



On Feb 10, 2008 9:37 AM, Piotr Ożarowski <piotr@debian.org> wrote:
> * it should be "Architecture: all" package, not "any"
>  + build your package in binary-indep
>  + remove dh_makeshlibs, dh_shlibdeps (BTW: you've missed dh_strip) from rules
>  + you can remove all build rule content (leave it empty)
>  + move as many packages as you can from Build-Depends to Build-Depends-Indep
> * FTBFS ("dh_install: python-twitter missing files (debian/tmp/*), aborting")
> * bump python-support required version to >= 0.6.4 or make sure (in
>   debian/rules) that .egg directory has the right name
> * remove hashbang from twitter.py (via patch or using sed in debian/rules)
> * "s/python/Python" in long desc.
> * remove "(s)" from "Upstream Author(s)" in debian/copyright (there's
>   only one author listed there)
> * add "dh_compress -X.py" in binary-indep rule
> * consider installing scripts from examples directory in /usr/bin/
>   (without ".py" in file name)
> * use `lintian -i python-twitter_*.changes` before sending RFS mail
>

Hello Piotr,
i did all the changes you've recommended me to do.
Btw, i added a few lines to install the examples in /usr/bin (one of
them is a mini client)
the FTBFS bug was because of i forgot to remove an (old) install file.
I used pbuilder to test that it actually builds OK and it and worked
fine, also i did used lintian to test the changes file and is "clean"

well, here is the url:
http://mentors.debian.net/debian/pool/main/p/python-twitter/python-twitter_0.5-1.dsc

if you see anything else that i should work on, just tell me.

Thanks,
Mauro

-- 
----BEGIN GEEK CODE BLOCK----
Version: 3.12
GCM/O d->dpu$ s-:- a-->a+++$ C+++
LU P+ L++ E W+++ N !o K w O !M !V
PS+ PE Y+ PGP t 5– X R tv++ b- DI D++ G+ e
h!>h-- r>r+++ y+
----END GEEK CODE BLOCK----

Reply to: