On Thu, 2007-08-16 at 00:13 +0200, Krzysztof Sobolewski wrote:
> I heard that Debian packagers (re-)appeared so I hope they won't mind that
> I'm overstepping their responsibilities :)

We are all learning together.

Try building using "build-deb.sh" script. Ultimately I think the stuff
in that should be merged into the ./debian/rules file?

It does things like makes sure you are building from a clean tree and
also "fixes" the version.py file (so when you install it the git info
isn't lost).

> I'm a Debian user so I thought that before I start playing "for real" with
> the sources, I'd get them to build as .debs first. Unfortunately, after some
> days of trying still no luck. They're aaaaaaaaaalmost OK, after some small
> tweaks, but not quite (hey, I'm still learning!). I'm attaching patches to
> packages I tried to fix - some are easy, some are not.

I'm only going to comment on bits which I maintain...

> For Python packages I'm not sure if I should use dh_pysupport or
> dh_pycentral or both and what is really necessary... I'm a Java guy, you know 
> ;)

http://wiki.debian.org/DebianPython/NewPolicy
        The main principles of this policy are shared by many people,
        however there's no consensus on the best tool to implement it.
        That's the reason why you will have to choose between
        python-central and python-support. We have done our best to make
        it easy to switch from one to the other and their usage is very
        similar from the maintainer's point of view. Either tool should
        support any package; maintainers should choose the tool with the
        interface that they prefer.

So I don't think we need the addition of dh_pysupport in the patches you
sent.

> * libtpproto-py - seems OK too, but there's a problem with
> tp.netlib.discover.LocalBrowser (tpclient-pywx won't see it - does it need
> Avahi, or something?)

It will try "python-avahi", then "python-bonjour" then use it's internal
(customised pyZeroconf) implementation.

libtpproto-py-0001-Some-tweaks-to-Debian-package-plus-one-ommission-in.patch
        Partially committed (only the setup.py fix). Found another
        ommission along the same lines which should fix the above
        problem.
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=libtpproto-py.git;a=commit;h=27b6f6ca628fdf3bc34f011f8d3833ce85cc54e3


> * libtpclient-py - probably OK

libtpclient-py-0001-Fix-the-changelog-so-dpkg-source-won-t-barf.patch
        Didn't use this patch, instead removed the offending items from
        the changelog as they shouldn't have been committed anyway.
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=libtpclient-py.git;a=commit;h=a81b1baea764acc5a8fcbbd62ceab766139d2544

> * tpclient-pywx - this one needed some fixes to the sources too; it builds
> and installs but fails on startup (due to aforementioned problem with
> LocalBrowser)

tpclient-pywx-0001-Make-the-package-buildable-and-somewhat-acceptable-f.patch
        Merged (no dependecy on py_support)
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pywx.git;a=commit;h=86cc1b7d7b9bb51839008241249e37e79d7b25f2

tpclient-pywx-0002-Make-it-pdebuildable-Build-Depends-rules-and-Pyth.patch 
        Merged without the py_support stuff
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pywx.git;a=commit;h=fa39d724d7127df8947da3049461c39a47575d86
        
tpclient-pywx-0003-Fix-indents-spaces-in-extra-wxFloatCanvas-Arro.patch 
        Merged with slightly less dramatic comment (not this file isn't
        actually used at the moment).
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pywx.git;a=commit;h=6516e1038b6d09ca30eaa942e2c8473eb3fdb988

tpclient-pywx-0004-I-think-it-was-supposed-to-be-a-comment.patch
        Merged. (Again this file isn't used at the moment.)
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pywx.git;a=commit;h=42ed878e004d054292ff969ef22b56893e1c21e7

tpclient-pywx-0005-Fix-executable-symlink-not-to-point-to-build-tempora.patch 
        I'm not quite sure this patch is correct. From what I
        understand, symlinks in debs should always be absolute.

tpclient-pywx-0006-Make-it-not-barf-on-disallowed-chmod-while-starting.patch 
        Committed a fix which catches the exception and ignores it.
http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pywx.git;a=commit;h=4cf08fd8dd92f87742c4d3562d3063dc871243ac

> Of course there are lots of problems, lintian errors and such... Anywa, hope
> that helps :)
> -KS

How do you get lintian errors and such? I have never used those tools...

Thanks for the patches, they rock!

Mithro

_______________________________________________
tp-devel mailing list
[email protected]
http://www.thousandparsec.net/tp/mailman.php/listinfo/tp-devel

Reply via email to