Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] update xiate to new home and version
@ 2021-11-05 14:49 loreb
  2021-11-05 14:58 ` [PR PATCH] [Updated] " loreb
                   ` (14 more replies)
  0 siblings, 15 replies; 16+ messages in thread
From: loreb @ 2021-11-05 14:49 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1470 bytes --]

From 1cf16c1dfde7fe9060160b3d0bfb3033d36fbca8 Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] update xiate to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.
---
 srcpkgs/xiate/template | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..9e79a3211b63 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,6 +1,6 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=v21.05 # extracts to builddir/xiate-v21.05, not xiate-21-05
 revision=1
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
@@ -9,12 +9,10 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
 	sed -i 's;/usr/local;/usr;g' Makefile
 }
 

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
@ 2021-11-05 14:58 ` loreb
  2021-11-05 15:01 ` loreb
                   ` (13 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 14:58 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1470 bytes --]

From 3a96e0f1ae7f6117d974dfb68de350ab7a3e7647 Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] update xiate to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.
---
 srcpkgs/xiate/template | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..9e79a3211b63 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,6 +1,6 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=v21.05 # extracts to builddir/xiate-v21.05, not xiate-21-05
 revision=1
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
@@ -9,12 +9,10 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
 	sed -i 's;/usr/local;/usr;g' Makefile
 }
 

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
  2021-11-05 14:58 ` [PR PATCH] [Updated] " loreb
@ 2021-11-05 15:01 ` loreb
  2021-11-05 15:08 ` loreb
                   ` (12 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 15:01 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1417 bytes --]

From ffff9ecc81fc54530960c2016cc6535f5c46aaf1 Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] update xiate to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.
---
 srcpkgs/xiate/template | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..2958d7e5a96f 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,6 +1,6 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=v21.05
 revision=1
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
@@ -9,12 +9,10 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
 	sed -i 's;/usr/local;/usr;g' Makefile
 }
 

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
  2021-11-05 14:58 ` [PR PATCH] [Updated] " loreb
  2021-11-05 15:01 ` loreb
@ 2021-11-05 15:08 ` loreb
  2021-11-05 16:46 ` [PR REVIEW] " ericonr
                   ` (11 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 15:08 UTC (permalink / raw)
  To: ml

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

New comment by loreb on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#issuecomment-961975253

Comment:
CI complains that "srcpkgs/xiate/template:3: missing reverts= for downgrade 18.05 to v21.05",
is there a clean way out or should I add a fake "reverts=" to work around it?
The manual says that "The versions defined in reverts must be bigger than the one defined in version",
and I don't want to know how "18.05" compares to "v21.05".

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR REVIEW] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (4 preceding siblings ...)
  2021-11-05 16:46 ` ericonr
@ 2021-11-05 16:46 ` ericonr
  2021-11-05 16:46 ` ericonr
                   ` (8 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-05 16:46 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#discussion_r743822228

Comment:
```suggestion
distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
```

Add it here and in `wrksrc=`

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR REVIEW] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (2 preceding siblings ...)
  2021-11-05 15:08 ` loreb
@ 2021-11-05 16:46 ` ericonr
  2021-11-05 16:46 ` ericonr
                   ` (10 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-05 16:46 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#discussion_r743822409

Comment:
Please use vsed or create a patch.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR REVIEW] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (3 preceding siblings ...)
  2021-11-05 16:46 ` [PR REVIEW] " ericonr
@ 2021-11-05 16:46 ` ericonr
  2021-11-05 16:46 ` ericonr
                   ` (9 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-05 16:46 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#discussion_r743822017

Comment:
Don't add the `v` here.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR REVIEW] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (5 preceding siblings ...)
  2021-11-05 16:46 ` ericonr
@ 2021-11-05 16:46 ` ericonr
  2021-11-05 18:59 ` [PR PATCH] [Updated] " loreb
                   ` (7 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-05 16:46 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#discussion_r743822228

Comment:
```suggestion
distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
```

Add it here and in `wrksrc=xiate-v${version}`

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (6 preceding siblings ...)
  2021-11-05 16:46 ` ericonr
@ 2021-11-05 18:59 ` loreb
  2021-11-05 19:02 ` loreb
                   ` (6 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 18:59 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1543 bytes --]

From da30cb3f2a5b78404f18fbd3969bca99e38ce203 Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] update xiate to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

Also use vsed instead of plain sed.
---
 srcpkgs/xiate/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..68de86ac3b08 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,6 +1,6 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=21.05
 revision=1
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
@@ -9,13 +9,12 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
+wrksrc=xiate-v${version}
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
-	sed -i 's;/usr/local;/usr;g' Makefile
+	vsed -i 's;/usr/local;/usr;g' Makefile
 }
 
 post_install() {

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (7 preceding siblings ...)
  2021-11-05 18:59 ` [PR PATCH] [Updated] " loreb
@ 2021-11-05 19:02 ` loreb
  2021-11-05 19:07 ` [PR PATCH] [Updated] " loreb
                   ` (5 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 19:02 UTC (permalink / raw)
  To: ml

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

New comment by loreb on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#issuecomment-962142801

Comment:
ok thank you; and btw, `grep '\<sed\>' srcpkgs/*/template | wc -l` says 2918...

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (8 preceding siblings ...)
  2021-11-05 19:02 ` loreb
@ 2021-11-05 19:07 ` loreb
  2021-11-05 19:10 ` loreb
                   ` (4 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 19:07 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1543 bytes --]

From 480f6531f6f89ebe39093f89479ec1fe8a7c9bdf Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] update xiate to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

Also use vsed instead of plain sed.
---
 srcpkgs/xiate/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..106d9c777bb6 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,6 +1,6 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=21.05
 revision=1
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
@@ -9,13 +9,12 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+wrksrc=xiate-v${version}
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
-	sed -i 's;/usr/local;/usr;g' Makefile
+	vsed -i 's;/usr/local;/usr;g' Makefile
 }
 
 post_install() {

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (9 preceding siblings ...)
  2021-11-05 19:07 ` [PR PATCH] [Updated] " loreb
@ 2021-11-05 19:10 ` loreb
  2021-11-05 19:16 ` loreb
                   ` (3 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 19:10 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1570 bytes --]

From 5d794933ce4b0d8ad015577ac2aebdccd8394d20 Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] update xiate to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

Also use vsed instead of plain sed.
---
 srcpkgs/xiate/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..ec5db68ba5bf 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,7 +1,8 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=21.05
 revision=1
+wrksrc=xiate-v${version}
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
 makedepends="vte3-devel"
@@ -9,13 +10,11 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
-	sed -i 's;/usr/local;/usr;g' Makefile
+	vsed -i 's;/usr/local;/usr;g' Makefile
 }
 
 post_install() {

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (10 preceding siblings ...)
  2021-11-05 19:10 ` loreb
@ 2021-11-05 19:16 ` loreb
  2021-11-14  6:58 ` ericonr
                   ` (2 subsequent siblings)
  14 siblings, 0 replies; 16+ messages in thread
From: loreb @ 2021-11-05 19:16 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1571 bytes --]

From 4652b0852a466a5187b2642f0666c3c1a677e2d4 Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] xiate: update to new home and version

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

Also use vsed instead of plain sed.
---
 srcpkgs/xiate/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..ec5db68ba5bf 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,7 +1,8 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=21.05
 revision=1
+wrksrc=xiate-v${version}
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
 makedepends="vte3-devel"
@@ -9,13 +10,11 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
-	sed -i 's;/usr/local;/usr;g' Makefile
+	vsed -i 's;/usr/local;/usr;g' Makefile
 }
 
 post_install() {

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Updated] update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (11 preceding siblings ...)
  2021-11-05 19:16 ` loreb
@ 2021-11-14  6:58 ` ericonr
  2021-11-14  6:58 ` ericonr
  2021-11-14  6:58 ` [PR PATCH] [Merged]: " ericonr
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-14  6:58 UTC (permalink / raw)
  To: ml

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

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

https://github.com/loreb/void-packages xiate-new-home
https://github.com/void-linux/void-packages/pull/33914

update xiate to new home and version
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-xiate-new-home-33914.patch --]
[-- Type: text/x-diff, Size: 1569 bytes --]

From 5f4fbf63e9e86ce80eecad465f5529d10797f8ce Mon Sep 17 00:00:00 2001
From: Lorenzo Beretta <vc.net.loreb@gmail.com>
Date: Fri, 5 Nov 2021 15:39:35 +0100
Subject: [PATCH] xiate: update to 21.05 and new home

This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

Also use vsed instead of plain sed.
---
 srcpkgs/xiate/template | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/xiate/template b/srcpkgs/xiate/template
index 742c53df8a8e..ec5db68ba5bf 100644
--- a/srcpkgs/xiate/template
+++ b/srcpkgs/xiate/template
@@ -1,7 +1,8 @@
 # Template file for 'xiate'
 pkgname=xiate
-version=18.05
+version=21.05
 revision=1
+wrksrc=xiate-v${version}
 build_style=gnu-makefile
 hostmakedepends="pkg-config"
 makedepends="vte3-devel"
@@ -9,13 +10,11 @@ short_desc="Terminal emulator which uses VTE as a backend"
 maintainer="Diogo Leal <diogo@diogoleal.com>"
 license="MIT"
 homepage="https://www.uninformativ.de/projects/xiate/"
-distfiles="https://github.com/vain/xiate/archive/v${version}.tar.gz"
-checksum=2f7317cc71849cdbf31479be224f9ed20fce25d2fa8320885a5294ed2101411b
+distfiles="https://www.uninformativ.de/git/xiate/archives/xiate-v${version}.tar.gz"
+checksum=487dc53f48494e53a1db9aa36a1e04a594b207d7fcda84908e192bb4e706b7cb
 
 pre_configure() {
-	cp config.def.h config.h
-	[ -e ${FILESDIR}/config.h ] && cp ${FILESDIR}/config.h config.h
-	sed -i 's;/usr/local;/usr;g' Makefile
+	vsed -i 's;/usr/local;/usr;g' Makefile
 }
 
 post_install() {

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (12 preceding siblings ...)
  2021-11-14  6:58 ` ericonr
@ 2021-11-14  6:58 ` ericonr
  2021-11-14  6:58 ` [PR PATCH] [Merged]: " ericonr
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-14  6:58 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/33914#issuecomment-968218512

Comment:
I fixed the commit message, and merging.

Thanks!

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: [PR PATCH] [Merged]: update xiate to new home and version
  2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
                   ` (13 preceding siblings ...)
  2021-11-14  6:58 ` ericonr
@ 2021-11-14  6:58 ` ericonr
  14 siblings, 0 replies; 16+ messages in thread
From: ericonr @ 2021-11-14  6:58 UTC (permalink / raw)
  To: ml

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

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

update xiate to new home and version
https://github.com/void-linux/void-packages/pull/33914

Description:
This fixes https://github.com/void-linux/void-packages/issues/3549;
upstream is no longer using config.h, so we can simplify pre_configure.

<!-- 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.)
- [ X] I built this PR locally for my native architecture, (x86_64-musl)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->

I had to set $version to v21.05 (notice the extra 'v') because the tarball extracts to xiate-v21.05;
please let me know if there's a cleaner way.

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2021-11-14  6:58 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-11-05 14:49 [PR PATCH] update xiate to new home and version loreb
2021-11-05 14:58 ` [PR PATCH] [Updated] " loreb
2021-11-05 15:01 ` loreb
2021-11-05 15:08 ` loreb
2021-11-05 16:46 ` [PR REVIEW] " ericonr
2021-11-05 16:46 ` ericonr
2021-11-05 16:46 ` ericonr
2021-11-05 16:46 ` ericonr
2021-11-05 18:59 ` [PR PATCH] [Updated] " loreb
2021-11-05 19:02 ` loreb
2021-11-05 19:07 ` [PR PATCH] [Updated] " loreb
2021-11-05 19:10 ` loreb
2021-11-05 19:16 ` loreb
2021-11-14  6:58 ` ericonr
2021-11-14  6:58 ` ericonr
2021-11-14  6:58 ` [PR PATCH] [Merged]: " ericonr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).