[PR PATCH] gifski: update to 1.5.1

[PR PATCH] gifski: update to 1.5.1

[cinerea0]

[-- Attachment #1: Type: text/plain, Size: 1583 bytes --]

There is a new pull request by cinerea0 against master on the void-packages repository

https://github.com/cinerea0/void-packages gif
https://github.com/void-linux/void-packages/pull/33822

gifski: update to 1.5.1
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


A patch file from https://github.com/void-linux/void-packages/pull/33822.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-gif-33822.patch --]
[-- Type: text/x-diff, Size: 987 bytes --]

From fbafc800f0b52e0ee57d872348e656865a82095a Mon Sep 17 00:00:00 2001
From: cinerea0 <cinerea0@disroot.org>
Date: Sat, 30 Oct 2021 00:50:23 -0400
Subject: [PATCH] gifski: update to 1.5.1

---
 srcpkgs/gifski/template | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/srcpkgs/gifski/template b/srcpkgs/gifski/template
index 9a1057cb13af..9c4837e11ca2 100644
--- a/srcpkgs/gifski/template
+++ b/srcpkgs/gifski/template
@@ -1,6 +1,6 @@
 # Template file for 'gifski'
 pkgname=gifski
-version=1.4.3
+version=1.5.1
 revision=1
 build_style=cargo
 configure_args="--features=openmp"
@@ -10,7 +10,7 @@ maintainer="Benjamín Albiñana <benalb@gmail.com>"
 license="AGPL-3.0-only"
 homepage="https://gif.ski"
 distfiles="https://github.com/ImageOptim/gifski/archive/${version}.tar.gz"
-checksum=74b98ede664e3da420ead70e3da1dfeb1a6e33545a618124e6b49da8f399902f
+checksum=88beeb896b6a1138046f665c3495f85670a74a527e34743080d8976d3f1b73b7
 
 post_install() {
 	vlicense LICENSE

Re: gifski: update to 1.5.1

[zdykstra]

[-- Attachment #1: Type: text/plain, Size: 1362 bytes --]

New comment by zdykstra on void-packages repository

https://github.com/void-linux/void-packages/pull/33822#issuecomment-955240394

Comment:
I tested this on `x86_64/glibc`, and like 1.4.3 it's missing the ability to directly convert a video into a gif.

```
$ gifski --fps 10 -o ~/tmp/doot.gif ~/Downloads/tz56y1O.mp4
error: Video support is permanently disabled in this executable.

To enable video decoding you need to recompile gifski from source with:
cargo build --release --features=video
or
cargo install gifski --features=video

Alternatively, use ffmpeg command to export PNG frames, and then specify
the PNG files as input for this executable. Instructions on https://gif.ski
```

Running my video through ffmpeg first and then converting the output PNGs to a GIF seems to work, though.

```
$ gifski --version
gifski 1.5.1

$ ffmpeg -i tz56y1O.mp4 frame%04d.png
....
Input #0, mov,mp4,m4a,3gp,3g2,mj2, from 'tz56y1O.mp4':
...
Output #0, image2, to 'frame%04d.png':
frame=   30 fps=0.0 q=-0.0 Lsize=N/A time=00:00:03.00 bitrate=N/A speed=58.4x
video:4134kB audio:0kB subtitle:0kB other streams:0kB global headers:0kB muxing overhead: unknown

$ gifski -o doot.gif frame*.png
gifski created /home/zdykstra/Downloads/test/doot.gif

$ webify -p doot.gif
https://pastebin.stratumzero.date/u/p/2142e388af/doot.gif
```

Re: gifski: update to 1.5.1

[cinerea0]

[-- Attachment #1: Type: text/plain, Size: 282 bytes --]

New comment by cinerea0 on void-packages repository

https://github.com/void-linux/void-packages/pull/33822#issuecomment-955563466

Comment:
@zdykstra Yeah, that second way is how I tested it. Would you like me to try compiling it with the `--features=video` flag and testing that?

Re: gifski: update to 1.5.1

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 230 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33822#issuecomment-955568633

Comment:
https://github.com/void-linux/void-packages/pull/28506 for discussion on the ffmpeg stuff

Re: [PR PATCH] [Merged]: gifski: update to 1.5.1

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 1434 bytes --]

There's a merged pull request on the void-packages repository

gifski: update to 1.5.1
https://github.com/void-linux/void-packages/pull/33822

Description:
<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [x] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->