[Pkg-phototools-devel] RFS: openjpeg

[Robin Cornelius]

On Jan 18, 2008 2:01 PM, Cyril Brulebois
<cyril.brulebois at enst-bretagne.fr> wrote:
> On 15/01/2008, Robin Cornelius wrote:
> > Thanks for the patches, I will have a play with git and experiment a
> > little before i commit anything. I have only used git a little before
> > so i may take a couple of days to just check things out a little :-)
>
> OK, no problem. Anyway, you could eventually break the public repository
> somehow, since it's just been started, so that won't do much harm
> anyway. That would be more problematic after several uploads, some tags,
> and so on. It's quite easy to log onto alioth and eventually issue some
> "git-branch -D" or "git-reset" there (at the beginning).

I think everything is ok now. I seem to have the hang of the basics of
git. I have committed your patches you emailed to me, and a couple
more based on your previous suggestions.

>
> Basically, I'd just use "unstable" (branched from trunk), delete
> (git-rm) everything but debian/, then commit that. Let me know if I can
> be of any help.

Yep, done. I only have the unstable branch now and it only contains debian/

I've implemented a get-orig-source rule for the makefile, but i don't
like it. It seems a hack to me so if you can comment on a better way
to do it that would be very welcome.

Out of interest which build command do you then run, i assume
dpkg-buildpackage as git-buildpackage requires the upstream source to
be present in the tree (unless you run with --ignore-new) ?

>
> Another patch for the watch file:
> | opts=uversionmangle=s/_/./g \
> | http://www.openjpeg.org/index.php?menu=download openjpeg_v([0-9_.]+)\.tar\.gz

Applied, for some reason uscan when called from debian/rules (in my
get-orig-source rule) will not respect the continue line \ terminator,
from the shell it is fine so i have made your changes but left them on
a single line.

>
> Upstream' site seems a bit buggy, since about 50% of the time, one gets:
> | Parse error: syntax error, unexpected T_STRING, expecting ')' in
> | /export/OpenJpeg/bbclone/var/last.php on line 571
>
> Which results in:
> | $ uscan --report-status
> | uscan warning: In debian/watch,
> |   no matching hrefs for watch line
> |   http://www.openjpeg.org/index.php?menu=download openjpeg_v([0-9_.]+)\.tar\.gz

Not seen this myself before, but yes i have noticed this now. Looks
like an upstream bug report is due.

Thanks for your help

Robin