Hi,

On Sun, Aug 24, 2014 at 11:31:46AM -0700, intrigeri wrote:
> David Paleino wrote (19 Aug 2014 12:58:38 GMT) :
> > I haven't been able to do *ANY* Debian-related work because I'm in my
> > summer house, with very limited connectivity. Thus, it would be nice
> > if you could handle the package upload for me. :)
> 
> OK, cool. Michael Banck (cc'd) will probably handle the sponsoring.

I looked at the package on mentors. 

The following must be changed:

1. The debian revision version number must reflect the NMU, i.e.
should be -0.1, not -1

2. The distribution must be "unstable", not "unreleased"

3. The package fails to build for me, it apparently required
python3-setuptools (which would be consistent with the other python
package Build-Depends):

|I: pybuild base:170: python3.4 setup.py clean
|Traceback (most recent call last):
|  File "setup.py", line 6, in <module>
|    from setuptools import setup, find_packages
|ImportError: No module named 'setuptools'
|E: pybuild pybuild:256: clean: plugin distutils failed with: exit
|code=1: python3.4 setup.py clean
|dh_auto_clean: pybuild --clean -i python{version} -p 3.4 --dir .
|returned exit code 13
|make: *** [clean] Error 13
|debian/rules:17: recipe for target 'clean' failed
|dpkg-buildpackage: error: fakeroot debian/rules clean gave error exit
|status 2

4. The package in unstable got repackaged (+ds1 upstream version number)
to remove a duplicated copy of isocodes, according to README.source. The
new package does not have that, either the upstream version number shold
be amended to reflect that, or README.source should get
removed/adjusted

The following should really be changed:

5. 02-support_language_common-name.patch got removed, that should be
mentioned in the changelog

6. There's a typo in the changelog: "Add .zip urls to wach",
s/wach/watch


Cheers,

Michael


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to