[PR PATCH] eva: fix checksum

[PR PATCH] eva: fix checksum

[jcgruenhage]

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

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

https://github.com/jcgruenhage/void-packages eva-0.3.0_1
https://github.com/void-linux/void-packages/pull/40280

eva: fix checksum
<!-- Uncomment relevant sections and delete options which are not applicable -->

Checksum changed since this was packaged. Does a revbump for stuff like this make sense? Probably not, right?

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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/40280.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eva-0.3.0_1-40280.patch --]
[-- Type: text/x-diff, Size: 866 bytes --]

From 44df17536a9da9fc57264ef72fbc328c60a652c7 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Christian=20Gr=C3=BCnhage?=
 <jan.christian@gruenhage.xyz>
Date: Tue, 1 Nov 2022 20:43:14 +0100
Subject: [PATCH] eva: fix checksum

---
 srcpkgs/eva/template | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/srcpkgs/eva/template b/srcpkgs/eva/template
index d364c98f9516..7fe24e9cb688 100644
--- a/srcpkgs/eva/template
+++ b/srcpkgs/eva/template
@@ -8,7 +8,7 @@ maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
 license="MIT"
 homepage="https://github.com/NerdyPepper/eva"
 distfiles="https://github.com/NerdyPepper/eva/archive/v${version}.tar.gz"
-checksum=05e2cdcfd91e6abef91cb01ad3074583b8289f6e74054e070bfbf6a4e684865e
+checksum=5ee80a64de05ff952e9fd69fff6050d8e5c602428ad8ad3b357db6b813e1372f
 
 post_install() {
 	vlicense LICENSE

Re: eva: fix checksum

[classabbyamp]

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299035593

Comment:
did the contents of the tarball change?

Re: [PR REVIEW] eva: fix checksum

[tranzystorek-io]

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

New review comment by tranzystorek-io on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#discussion_r1010848687

Comment:
```suggestion
distfiles="https://github.com/NerdyPepper/eva/archive/refs/tags/v${version}.tar.gz"
```

Re: eva: fix checksum

[jcgruenhage]

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

New comment by jcgruenhage on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299202748

Comment:
I don't know whether the contents changed, I don't have an old checkout lying around...

Re: [PR PATCH] [Updated] eva: fix checksum

[jcgruenhage]

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

There is an updated pull request by jcgruenhage against master on the void-packages repository

https://github.com/jcgruenhage/void-packages eva-0.3.0_1
https://github.com/void-linux/void-packages/pull/40280

eva: fix checksum
<!-- Uncomment relevant sections and delete options which are not applicable -->

Checksum changed since this was packaged. Does a revbump for stuff like this make sense? Probably not, right?

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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/40280.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eva-0.3.0_1-40280.patch --]
[-- Type: text/x-diff, Size: 1010 bytes --]

From 7a9efcb4b8b316bda6e37cd0fb86b9d29ac6616b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Christian=20Gr=C3=BCnhage?=
 <jan.christian@gruenhage.xyz>
Date: Tue, 1 Nov 2022 20:43:14 +0100
Subject: [PATCH] eva: fix checksum

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

diff --git a/srcpkgs/eva/template b/srcpkgs/eva/template
index d364c98f9516..ecc88d5c53a4 100644
--- a/srcpkgs/eva/template
+++ b/srcpkgs/eva/template
@@ -7,8 +7,8 @@ short_desc="Simple calculator REPL, similar to bc(1)"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
 license="MIT"
 homepage="https://github.com/NerdyPepper/eva"
-distfiles="https://github.com/NerdyPepper/eva/archive/v${version}.tar.gz"
-checksum=05e2cdcfd91e6abef91cb01ad3074583b8289f6e74054e070bfbf6a4e684865e
+distfiles="https://github.com/NerdyPepper/eva/archive/refs/tags/v${version}.tar.gz"
+checksum=5ee80a64de05ff952e9fd69fff6050d8e5c602428ad8ad3b357db6b813e1372f
 
 post_install() {
 	vlicense LICENSE

Re: eva: fix checksum

[classabbyamp]

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

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299313053

Comment:
> I don't know whether the contents changed, I don't have an old checkout lying around...

https://sources.voidlinux.org/eva-0.3.0/v0.3.0.tar.gz

Re: eva: fix checksum

[jcgruenhage]

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

New comment by jcgruenhage on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299599025

Comment:
Okay, yeah, the contents did change, and quite a bit. Reformatting, dependency updates, lots of new tests, functionality changes. This would mean we do also do a revbump here?

Re: eva: fix checksum

[tranzystorek-io]

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

New comment by tranzystorek-io on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299660926

Comment:
Maybe we could persuade upstream to make a proper new release number and just skip this?

Re: eva: fix checksum

[jcgruenhage]

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

New comment by jcgruenhage on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299876565

Comment:
That would just cause work for the maintainers of nixpkgs and Arch, who've already "dealt" with this by bumping a revision number though.

Re: eva: fix checksum

[tranzystorek-io]

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

New comment by tranzystorek-io on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299883241

Comment:
Ah, nevermind then, I was thinking maybe upstream have something on master branch worth a tiny patch bump, sth like that

Re: eva: fix checksum

[pickfire]

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

New comment by pickfire on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299939433

Comment:
Sorry for the retag, but I can do a version bump if needed.

Re: eva: fix checksum

[tranzystorek-io]

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

New comment by tranzystorek-io on void-packages repository

https://github.com/void-linux/void-packages/pull/40280#issuecomment-1299943032

Comment:
@pickfire I think as @jcgruenhage mentioned, a version bump would work if there are any meaningful changes left to be released, otherwise we just create churn for other distros, and I believe we can do a revision bump on our end for now

Re: [PR PATCH] [Updated] eva: fix checksum

[jcgruenhage]

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

There is an updated pull request by jcgruenhage against master on the void-packages repository

https://github.com/jcgruenhage/void-packages eva-0.3.0_1
https://github.com/void-linux/void-packages/pull/40280

eva: fix checksum
<!-- Uncomment relevant sections and delete options which are not applicable -->

Checksum changed since this was packaged. Does a revbump for stuff like this make sense? Probably not, right?

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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/40280.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-eva-0.3.0_1-40280.patch --]
[-- Type: text/x-diff, Size: 1211 bytes --]

From 9015829893abc368cdc809f9d54ecad7229b84b1 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jan=20Christian=20Gr=C3=BCnhage?=
 <jan.christian@gruenhage.xyz>
Date: Tue, 1 Nov 2022 20:43:14 +0100
Subject: [PATCH] eva: bump revision and update checksum

Upstream re-released v0.3.0, so the builds fail with bad checksum right
now.
---
 srcpkgs/eva/template | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/eva/template b/srcpkgs/eva/template
index d364c98f9516..f9909970861b 100644
--- a/srcpkgs/eva/template
+++ b/srcpkgs/eva/template
@@ -1,14 +1,14 @@
 # Template file for 'eva'
 pkgname=eva
 version=0.3.0
-revision=1
+revision=2
 build_style=cargo
 short_desc="Simple calculator REPL, similar to bc(1)"
 maintainer="Jan Christian Grünhage <jan.christian@gruenhage.xyz>"
 license="MIT"
 homepage="https://github.com/NerdyPepper/eva"
-distfiles="https://github.com/NerdyPepper/eva/archive/v${version}.tar.gz"
-checksum=05e2cdcfd91e6abef91cb01ad3074583b8289f6e74054e070bfbf6a4e684865e
+distfiles="https://github.com/NerdyPepper/eva/archive/refs/tags/v${version}.tar.gz"
+checksum=5ee80a64de05ff952e9fd69fff6050d8e5c602428ad8ad3b357db6b813e1372f
 
 post_install() {
 	vlicense LICENSE

Re: [PR PATCH] [Merged]: eva: bump revision and update checksum

[classabbyamp]

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

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

eva: bump revision and update checksum
https://github.com/void-linux/void-packages/pull/40280

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

<del>Checksum changed since this was packaged. Does a revbump for stuff like this make sense? Probably not, right?</del>

Upstream re-released v0.3.0, so the builds fail with bad checksum right now.
    
#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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
-->