[needs-packaging] gst-thumbnailers

Bug #2137704 reported by Charles
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu
New
Wishlist
Unassigned

Bug Description

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.

Revision history for this message
Brian Murray (brian-murray) wrote :

*** 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.

Changed in ubuntu:
importance: Undecided → Wishlist
Revision history for this message
Sebastien Bacher (seb128) wrote :

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

Revision history for this message
Charles (charles05) wrote :

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.

Revision history for this message
Sebastien Bacher (seb128) wrote :

> 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 :-)

Revision history for this message
Charles (charles05) wrote :

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.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.