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

Re: RFS: uniconvertor

On Sat, 2008-03-15 at 14:15 +0100, Piotr Ożarowski wrote:
> > http://mentors.debian.net/debian/pool/main/u/uniconvertor/uniconvertor_1.1.1-1.dsc

> * rename uniconvertor-modules package to python-uniconvertor
> * IMHO you can merge uniconvertor and python-uniconvertor packages
>   (new name: python-uniconvertor). Yeah, that means lots of architecture
>   independent files will be in arch. dependent package, but if you'll
>   additionally use bzip2 compression (dh_builddeb -- -Z bzip2), that
>   will not be a big problem (467 KiB), I guess
Merged back into one package and using bzip2 compression.
> * change python-uniconvertor's Section to python
> * add "Conflicts: yudit" to uniconvertor package (/usr/bin/uniconv)
Thanks; conflicts added. We can remove that after the symlinking is
removed later.
> * build depend on python-central (>= 0.6)
> * add "Provides: ${python:Provides}" in python-uniconvertor package
> * add dh_shlibdeps in binary-arch rule
> * replace "libc6 (>= 2.6), python (>= 2.4)" with
>   "${python:Depends}, ${shlibs:Depends}" in Depends
> * build python-uniconvertor-dbg package (see DPMT repo for some examples),
>   join #debian-python if you will need help
Figured it out with a little help =)
> * remove last line (" .") in long description
> * remove dh_strip from binary-indep rule
No binary indep rule anymore, so nothing to remove
> * hint: you can use dirs and links files (and dh_installdirs, dh_link in
>   debian rules)
Thanks for the hint. Changed to use these functions of course.
> * why uniconvertor wrapper is in /usr/sbin and not in /usr/bin ?
Mindslip... changed back of course.
> * add "exec" before "python -c" in ^^
> * consider joining DPMT [1]
Not right now ;-) Still learning while finding out, where I belong.

There is still two lintian warnings, which I'm ignoring. The one is after joining the packages again; and the second one is for sym-linking the doc-dir between the two pacakges.

Thanks a lot for looking at the package.

New version avaible at mentors.

Andreas Wenning <awen@awen.dk>

Reply to: