[PR PATCH] gifsicle: switch upstream, build gifview

[PR PATCH] gifsicle: switch upstream, build gifview

[cinerea0]

[-- Attachment #1: Type: text/plain, Size: 1600 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/33882

gifsicle: switch upstream, build gifview
<!-- 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?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] 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/33882.patch is attached

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

From 526dad147906da6b709e66377e40db908611cee5 Mon Sep 17 00:00:00 2001
From: cinerea0 <cinerea0@disroot.org>
Date: Wed, 3 Nov 2021 19:38:11 -0400
Subject: [PATCH] gifsicle: switch upstream, build gifview

---
 srcpkgs/gifsicle/template | 13 +++++--------
 srcpkgs/gifsicle/update   |  2 ++
 2 files changed, 7 insertions(+), 8 deletions(-)
 create mode 100644 srcpkgs/gifsicle/update

diff --git a/srcpkgs/gifsicle/template b/srcpkgs/gifsicle/template
index d466886ae54e..3cddf6dc77ef 100644
--- a/srcpkgs/gifsicle/template
+++ b/srcpkgs/gifsicle/template
@@ -1,19 +1,16 @@
 # Template file for 'gifsicle'
 pkgname=gifsicle
 version=1.93
-revision=2
+revision=3
 build_style=gnu-configure
-hostmakedepends="automake libtool pkg-config"
+makedepends="libX11-devel"
 short_desc="Command-line tool for information about GIF images"
 maintainer="cinerea0 <cinerea0@protonmail.com>"
 license="GPL-2.0-or-later"
 homepage="http://www.lcdf.org/gifsicle/"
-distfiles="https://github.com/kohler/gifsicle/archive/refs/tags/v${version}.tar.gz"
-checksum=941c0b1c3ebc748949a0d14363f0fb3c9d2c06ee122b3548315b9233eb956d88
-
-pre_configure() {
-	autoreconf -fi
-}
+changelog="https://www.lcdf.org/gifsicle/changes.html"
+distfiles="${homepage}/gifsicle-${version}.tar.gz"
+checksum=92f67079732bf4c1da087e6ae0905205846e5ac777ba5caa66d12a73aa943447
 
 Gifsicle_package() {
 	depends="gifsicle"
diff --git a/srcpkgs/gifsicle/update b/srcpkgs/gifsicle/update
new file mode 100644
index 000000000000..4ecd4d7c083e
--- /dev/null
+++ b/srcpkgs/gifsicle/update
@@ -0,0 +1,2 @@
+site="https://www.lcdf.org/gifsicle/changes.html"
+pattern='<b>\K[0-9\.]+(?=</b>)'

Re: [PR PATCH] [Updated] gifsicle: switch upstream, build gifview

[cinerea0]

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

There is an updated 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/33882

gifsicle: switch upstream, build gifview
<!-- 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?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] 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
-->
---
Closes #30677

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

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

From 6b981dcd544d4cf596e9e83932b00ab59dc44cf9 Mon Sep 17 00:00:00 2001
From: cinerea0 <cinerea0@disroot.org>
Date: Wed, 3 Nov 2021 22:46:19 -0400
Subject: [PATCH] gifsicle: switch upstream, build gifview

---
 srcpkgs/gifsicle/template | 14 ++++++++------
 srcpkgs/gifsicle/update   |  2 ++
 2 files changed, 10 insertions(+), 6 deletions(-)
 create mode 100644 srcpkgs/gifsicle/update

diff --git a/srcpkgs/gifsicle/template b/srcpkgs/gifsicle/template
index d466886ae54e..bf1461300a1d 100644
--- a/srcpkgs/gifsicle/template
+++ b/srcpkgs/gifsicle/template
@@ -1,18 +1,20 @@
 # Template file for 'gifsicle'
 pkgname=gifsicle
 version=1.93
-revision=2
+revision=3
 build_style=gnu-configure
-hostmakedepends="automake libtool pkg-config"
+makedepends="libX11-devel"
+checkdepends="perl"
 short_desc="Command-line tool for information about GIF images"
 maintainer="cinerea0 <cinerea0@protonmail.com>"
 license="GPL-2.0-or-later"
 homepage="http://www.lcdf.org/gifsicle/"
-distfiles="https://github.com/kohler/gifsicle/archive/refs/tags/v${version}.tar.gz"
-checksum=941c0b1c3ebc748949a0d14363f0fb3c9d2c06ee122b3548315b9233eb956d88
+changelog="https://www.lcdf.org/gifsicle/changes.html"
+distfiles="${homepage}/gifsicle-${version}.tar.gz"
+checksum=92f67079732bf4c1da087e6ae0905205846e5ac777ba5caa66d12a73aa943447
 
-pre_configure() {
-	autoreconf -fi
+do_check() {
+	./test/testie -p ./src ./test
 }
 
 Gifsicle_package() {
diff --git a/srcpkgs/gifsicle/update b/srcpkgs/gifsicle/update
new file mode 100644
index 000000000000..4ecd4d7c083e
--- /dev/null
+++ b/srcpkgs/gifsicle/update
@@ -0,0 +1,2 @@
+site="https://www.lcdf.org/gifsicle/changes.html"
+pattern='<b>\K[0-9\.]+(?=</b>)'

Re: gifsicle: switch upstream, build gifview

[ericonr]

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33882#issuecomment-960432939

Comment:
Please fix commit message: this isn't switching upstream, it is using pre-generated tarballs.

Re: [PR PATCH] [Updated] gifsicle: switch upstream, build gifview

[cinerea0]

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

There is an updated 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/33882

gifsicle: switch upstream, build gifview
<!-- 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?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] 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
-->
---
Closes #30677

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

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

From d3570e0e8f91ddec96b0047f4acea08c8e734126 Mon Sep 17 00:00:00 2001
From: cinerea0 <cinerea0@disroot.org>
Date: Wed, 3 Nov 2021 23:57:09 -0400
Subject: [PATCH] gifsicle: use pre-generated tarball, build gifview

---
 srcpkgs/gifsicle/template | 14 ++++++++------
 srcpkgs/gifsicle/update   |  2 ++
 2 files changed, 10 insertions(+), 6 deletions(-)
 create mode 100644 srcpkgs/gifsicle/update

diff --git a/srcpkgs/gifsicle/template b/srcpkgs/gifsicle/template
index d466886ae54e..bf1461300a1d 100644
--- a/srcpkgs/gifsicle/template
+++ b/srcpkgs/gifsicle/template
@@ -1,18 +1,20 @@
 # Template file for 'gifsicle'
 pkgname=gifsicle
 version=1.93
-revision=2
+revision=3
 build_style=gnu-configure
-hostmakedepends="automake libtool pkg-config"
+makedepends="libX11-devel"
+checkdepends="perl"
 short_desc="Command-line tool for information about GIF images"
 maintainer="cinerea0 <cinerea0@protonmail.com>"
 license="GPL-2.0-or-later"
 homepage="http://www.lcdf.org/gifsicle/"
-distfiles="https://github.com/kohler/gifsicle/archive/refs/tags/v${version}.tar.gz"
-checksum=941c0b1c3ebc748949a0d14363f0fb3c9d2c06ee122b3548315b9233eb956d88
+changelog="https://www.lcdf.org/gifsicle/changes.html"
+distfiles="${homepage}/gifsicle-${version}.tar.gz"
+checksum=92f67079732bf4c1da087e6ae0905205846e5ac777ba5caa66d12a73aa943447
 
-pre_configure() {
-	autoreconf -fi
+do_check() {
+	./test/testie -p ./src ./test
 }
 
 Gifsicle_package() {
diff --git a/srcpkgs/gifsicle/update b/srcpkgs/gifsicle/update
new file mode 100644
index 000000000000..4ecd4d7c083e
--- /dev/null
+++ b/srcpkgs/gifsicle/update
@@ -0,0 +1,2 @@
+site="https://www.lcdf.org/gifsicle/changes.html"
+pattern='<b>\K[0-9\.]+(?=</b>)'

Re: [PR PATCH] [Merged]: gifsicle: use pre-generated tarball, build gifview

[Johnnynator]

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

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

gifsicle: use pre-generated tarball, build gifview
https://github.com/void-linux/void-packages/pull/33882

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?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] 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
-->
---
Closes #30677