[med-svn] r2703 - in trunk/packages/theseus/trunk/debian: . patches

[Andreas Tille]

On Mon, 17 Nov 2008, mok0-guest at alioth.debian.org wrote:

> --- trunk/packages/theseus/trunk/debian/rules	2008-11-17 13:53:58 UTC (rev 2702)
> +++ trunk/packages/theseus/trunk/debian/rules	2008-11-17 23:07:09 UTC (rev 2703)
> @@ -6,10 +6,11 @@
> # extract the upstream version from debian/changelog
> uvers := $(shell sed 's/.*(\(.*\)-.*).*/\1/;q' < debian/changelog)
>
> +# Get the auxilliary script distributed separately
> get-orig-source:
> -	wget -q http://www.theseus3d.org/src/theseus.tar.gz \
> -	     -O ../theseus_$(uvers).orig.tar.gz
> -
> +	dh_testdir
> +	wget -q http://www.theseus3d.org/src/theseus_align \
> +	     -O - | sed 's!/usr/local/bin!/usr/bin!' > debian/theseus_align
> build: patch-stamp build-stamp

Hmmm, I perfectly see the reason why you try to wget the theseus_align script
and move it to the debian directory - but I somehow have a bad feeling to
put this into the get-orig-source target.  There is probably no harm done and
I will not say that you should respect my _feeling_ in your packaging but
this is at least an unusual solution.

As I said I would love to see some README.Debian explaining the issues
around the theseus_align script in any case.  Perhaps you might also give
a hint about your script getexamples inside this README.Debian.  This
file is the first entry point for users and it might be interesting to
learn about this script here.

I see no other issues than this README.Debian and would upload once you
ask me to do so.

Kind regards and thanks for working on this

       Andreas.

-- 
http://fam-tille.de