[needs-packaging] gst-thumbnailers

[Charles]

gst-thumbnailers contains thumbnailing executables for video and audio files. They are written with Rust using the gstreamer-rs bindings.

PPA: https://launchpad.net/~charles05/+archive/ubuntu/gst-thumbnailers/+packages

URL: https://gitlab.gnome.org/GNOME/gst-thumbnailers
License: GPL-3+
Notes: Debian also started packaging this (ITP: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1121723, Salsa: https://salsa.debian.org/gnome-team/gst-thumbnailers) but it has been stuck in the NEW queue for a month. Since this package will need to go into Ubuntu main eventunally, the Rust vendoring process has been followed https://salsa.debian.org/ubuntu-dev-team/gst-thumbnailers/-/tree/ubuntu/latest - the packages proposed here come from that repository.

[Brian Murray (brian-murray)]

*** This is an automated message ***

This bug is tagged needs-packaging which identifies it as a request for a new package in Ubuntu. As a part of the managing needs-packaging bug reports specification, https://wiki.ubuntu.com/QATeam/Specs/NeedsPackagingBugs, all needs-packaging bug reports have Wishlist importance. Subsequently, I'm setting this bug's status to Wishlist.

[Sebastien Bacher (seb128)]

Thanks Charles, I've review the current 1.0~alpha.1-1ubuntu1~ppa4 version in the ppa and it's looking good, I've only a few minor comments

- debian/control

shouldn't gst-thumbnailer-test-data be 'arch: all'? It seems to contain some media files and no architecture dependent content?

also do we really need that binary? If that's only for the autopkgtests and the files are included in the source can't we just make TEST_DIR point to the SRCDIR/tests?

- debian/copyright

> Source: https://gitlab.gnome.org/sophie-h/gst-thumbnailers

it was moved to https://gitlab.gnome.org/GNOME/gst-thumbnailers so that reference can be updated

- debian/README.source

> [TODO: get access to ubuntu-dev-team https://salsa.debian.org/ubuntu-dev-team/gst-thumbnailers]

that is resolved now and such the description can be updated

[Charles (charles05)]

Thanks Seb!

> shouldn't gst-thumbnailer-test-data be 'arch: all'? It seems
> to contain some media files and no architecture dependent
> content?

I misinterpreted `Architecture: any` as meaning architecture independent, but it only means it can be built for any architecture -- the resulting binary package is architecture dependent.

Since this is intended to be bit-identical across architectures, I'll put `Architecture: all` on it. Thanks for catching that!

> also do we really need that binary? If that's only for the
> autopkgtests and the files are included in the source
> can't we just make TEST_DIR point to the SRCDIR/tests?

Thanks for pointing that out, there is no need for the test-data package!

> it was moved to https://gitlab.gnome.org/GNOME/gst-thumbnailers so that reference can be updated

This was updated in the ubuntu/latest branch, but you may have had an older checkout.

> that is resolved now and such the description can be updated

Ditto

All updates applied to ubuntu/latest now.

[Sebastien Bacher (seb128)]

> This was updated in the ubuntu/latest branch, but you may have had an older checkout.

Ah, sorry, I reviewed the package from the ppa and not the Vcs branch :-)

[Charles (charles05)]

Thanks for reviewing @seb128

There was a new upstream release. The latest changes are in https://salsa.debian.org/ubuntu-dev-team/gst-thumbnailers again. They started publishing tarballs on download.gnome.org, so I updated the watch file. Apart from that, minimal changes to the last package.