Bug#525348: ffmpeg-debian: get-orig-source rule doesn't include libswcale in generated tarball

[Andres Mejia]

On Saturday 02 May 2009 04:51:50 Reinhard Tartler wrote:
> Andres Mejia <mcitadel at gmail.com> writes:
> > I'm using Version: 3:0.svn20090303-1.1 of the ffmpeg-debian packages.
> > I've attached the log.
>
> that particular version has never been uploaded to debian. Have you
> mistyped the version number or did you get the package from somewhere
> else?

Oh right. Actually it is Version: 3:0.svn20090303-1. The '.1' at the end is 
there because I rebuilt ffmpeg-debian with the unstripped tarball and I didn't 
want a new upload from Debian overriding the rebuilt packages.

> Anyway, I now understand what happened. I've changed the script to fetch
> from the 0.5 release branch. At time of writing, it contained an
> svn:external to libswscale. However, the branch was changed to include
> libswscale rather than reference it. The problem in the script is that
> it fetches the svn:external information from the HEAD revision of the
> branch rather than the revision that is being checked out. the following
> patch should fix the issue:
>
> diff --git a/debian/get-orig-source.sh b/debian/get-orig-source.sh
> index e1bc738..f798ea8 100644
> --- a/debian/get-orig-source.sh
> +++ b/debian/get-orig-source.sh
> @@ -87,7 +87,7 @@ svn info -r{${SVNDATE}} \
>
>  	> ${TMPDIR}/${PACKAGENAME}/.svnrevision
>
>  # get svn externals
> -svn pg svn:externals $baseurl | \
> +svn pg svn:externals -r${${SVNDATE}} $baseurl | \
>  while read external url; do
>      [ -z $url ] && continue
>      dest="${TMPDIR}/${PACKAGENAME}/${external}"

Ok, great. :)

-- 
Regards,
Andres
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.alioth.debian.org/pipermail/pkg-multimedia-maintainers/attachments/20090502/d17fa86e/attachment.pgp>