* [PR PATCH] foot: update to 1.9.0.
@ 2021-08-27 12:45 ifreund
2021-08-27 13:44 ` [PR REVIEW] " ericonr
` (12 more replies)
0 siblings, 13 replies; 14+ messages in thread
From: ifreund @ 2021-08-27 12:45 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1581 bytes --]
There is a new pull request by ifreund against master on the void-packages repository
https://github.com/ifreund/void-packages foot
https://github.com/void-linux/void-packages/pull/32703
foot: update to 1.9.0.
<!-- 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/32703.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-foot-32703.patch --]
[-- Type: text/x-diff, Size: 1168 bytes --]
From 684bffd0b208750197b9a2c29b0e986f9db06c6a Mon Sep 17 00:00:00 2001
From: Isaac Freund <ifreund@ifreund.xyz>
Date: Fri, 27 Aug 2021 14:44:35 +0200
Subject: [PATCH] foot: update to 1.9.0.
---
| 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
--git a/srcpkgs/foot/template b/srcpkgs/foot/template
index 3dde8c741f24..d254056d476a 100644
--- a/srcpkgs/foot/template
+++ b/srcpkgs/foot/template
@@ -1,6 +1,6 @@
# Template file for 'foot'
pkgname=foot
-version=1.8.2
+version=1.9.0
revision=1
wrksrc=$pkgname
build_style=meson
@@ -15,7 +15,7 @@ license="MIT"
homepage="https://codeberg.org/dnkl/foot"
changelog="https://codeberg.org/dnkl/foot/src/branch/master/CHANGELOG.md"
distfiles="${homepage}/archive/${version}.tar.gz"
-checksum=50cf5b9f3cc1ebaafa62255eea22395e8267cce21e119bc6f7ecacf11f15dada
+checksum=ddc3f90185ae75badcb2d6abe772d54fd6defebf1ed4111861a9c4afa1c73a4d
post_install() {
rm -f ${DESTDIR}/usr/share/doc/foot/LICENSE
@@ -25,6 +25,6 @@ post_install() {
foot-terminfo_package() {
short_desc+=" - terminfo data"
pkg_install() {
- vmove usr/share/terminfo
+ vmove usr/share/foot/terminfo
}
}
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR REVIEW] foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
@ 2021-08-27 13:44 ` ericonr
2021-08-29 10:23 ` ram02z
` (11 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ericonr @ 2021-08-27 13:44 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 632 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#discussion_r697452400
Comment:
IMO foot gives us hooks in its build system exactly to control such things. We should use `usr/share/terminfo` at least until ncurses makes a release with their version of foot, otherwise installing `foot-terminfo` into a server you ssh into (which is the only reason to have a subpackage at all) won't do anything until you magically know to export TERMINFO somewhere.
It also breaks stupid things (I do) like `doas nvi`, unless I update my config to pass TERMINFO through as well.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR REVIEW] foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
2021-08-27 13:44 ` [PR REVIEW] " ericonr
@ 2021-08-29 10:23 ` ram02z
2021-08-29 10:41 ` ifreund
` (10 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ram02z @ 2021-08-29 10:23 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 261 bytes --]
New review comment by ram02z on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#discussion_r697992339
Comment:
The non standard terminfo location has a bunch of outstanding issues. See https://codeberg.org/dnkl/foot/issues/695
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR REVIEW] foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
2021-08-27 13:44 ` [PR REVIEW] " ericonr
2021-08-29 10:23 ` ram02z
@ 2021-08-29 10:41 ` ifreund
2021-08-29 11:09 ` [PR PATCH] [Updated] " ifreund
` (9 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ifreund @ 2021-08-29 10:41 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 269 bytes --]
New review comment by ifreund on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#discussion_r697994544
Comment:
Yep, from that issue and dnkl's mail to maintainers it looks like upstream now recommends exactly what @ericonr suggested.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR PATCH] [Updated] foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (2 preceding siblings ...)
2021-08-29 10:41 ` ifreund
@ 2021-08-29 11:09 ` ifreund
2021-09-04 17:28 ` ram02z
` (8 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ifreund @ 2021-08-29 11:09 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1586 bytes --]
There is an updated pull request by ifreund against master on the void-packages repository
https://github.com/ifreund/void-packages foot
https://github.com/void-linux/void-packages/pull/32703
foot: update to 1.9.0.
<!-- 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/32703.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-foot-32703.patch --]
[-- Type: text/x-diff, Size: 1810 bytes --]
From 0d0d79709ff0b88a654d0e881a11697e8a7d3618 Mon Sep 17 00:00:00 2001
From: Isaac Freund <ifreund@ifreund.xyz>
Date: Fri, 27 Aug 2021 14:44:35 +0200
Subject: [PATCH] foot: update to 1.9.0.
---
| 15 +++++++++++----
1 file changed, 11 insertions(+), 4 deletions(-)
--git a/srcpkgs/foot/template b/srcpkgs/foot/template
index 3dde8c741f24..5873ca6db29a 100644
--- a/srcpkgs/foot/template
+++ b/srcpkgs/foot/template
@@ -1,10 +1,12 @@
# Template file for 'foot'
pkgname=foot
-version=1.8.2
+version=1.9.0
revision=1
wrksrc=$pkgname
build_style=meson
-configure_args="-Dwerror=false"
+# Tell foot not to set TERMINFO as we install the terminfo to the default
+# location.
+configure_args="-Dwerror=false -Dcustom-terminfo-install-location=no"
hostmakedepends="pkg-config scdoc wayland-devel"
makedepends="fcft-devel libutf8proc-devel libxkbcommon-devel ncurses-devel tllist
wayland-devel wayland-protocols"
@@ -15,7 +17,7 @@ license="MIT"
homepage="https://codeberg.org/dnkl/foot"
changelog="https://codeberg.org/dnkl/foot/src/branch/master/CHANGELOG.md"
distfiles="${homepage}/archive/${version}.tar.gz"
-checksum=50cf5b9f3cc1ebaafa62255eea22395e8267cce21e119bc6f7ecacf11f15dada
+checksum=ddc3f90185ae75badcb2d6abe772d54fd6defebf1ed4111861a9c4afa1c73a4d
post_install() {
rm -f ${DESTDIR}/usr/share/doc/foot/LICENSE
@@ -25,6 +27,11 @@ post_install() {
foot-terminfo_package() {
short_desc+=" - terminfo data"
pkg_install() {
- vmove usr/share/terminfo
+ vmove usr/share/foot/terminfo
+
+ # Install the terminfo to the default location instead. This follows the
+ # new upstream recommendations: https://codeberg.org/dnkl/foot/issues/695
+ vcopy ${PKGDESTDIR}/usr/share/foot/terminfo usr/share/terminfo
+ rm -r ${PKGDESTDIR}/usr/share/foot/terminfo
}
}
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (3 preceding siblings ...)
2021-08-29 11:09 ` [PR PATCH] [Updated] " ifreund
@ 2021-09-04 17:28 ` ram02z
2021-09-12 4:16 ` cinerea0
` (7 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ram02z @ 2021-09-04 17:28 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 193 bytes --]
New comment by ram02z on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-913009814
Comment:
@ericonr sorry for the ping, but can this get merged?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (4 preceding siblings ...)
2021-09-04 17:28 ` ram02z
@ 2021-09-12 4:16 ` cinerea0
2021-09-27 3:34 ` ericonr
` (6 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: cinerea0 @ 2021-09-12 4:16 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 183 bytes --]
New comment by cinerea0 on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-917547938
Comment:
Agreed, this would be good to get merged.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (5 preceding siblings ...)
2021-09-12 4:16 ` cinerea0
@ 2021-09-27 3:34 ` ericonr
2021-09-27 7:36 ` stacyharper
` (5 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: ericonr @ 2021-09-27 3:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 265 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-927500726
Comment:
Seems to be working here locally, but I was under the impression a packaging fix release would be coming? 1.9.1 or whatever?
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (6 preceding siblings ...)
2021-09-27 3:34 ` ericonr
@ 2021-09-27 7:36 ` stacyharper
2021-09-30 18:40 ` cinerea0
` (4 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: stacyharper @ 2021-09-27 7:36 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 413 bytes --]
New comment by stacyharper on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-927610626
Comment:
For additional info, here a thread with @dnkl, the developer behind
foot:
https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/24357#note_177559
The actual mr that get merged into aports:
https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/24845
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (7 preceding siblings ...)
2021-09-27 7:36 ` stacyharper
@ 2021-09-30 18:40 ` cinerea0
2021-10-01 14:55 ` dnkl
` (3 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: cinerea0 @ 2021-09-30 18:40 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 215 bytes --]
New comment by cinerea0 on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-931570995
Comment:
A packaging fix may be coming, but it doesn't look like it's coming soon.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (8 preceding siblings ...)
2021-09-30 18:40 ` cinerea0
@ 2021-10-01 14:55 ` dnkl
2021-10-02 4:44 ` cinerea0
` (2 subsequent siblings)
12 siblings, 0 replies; 14+ messages in thread
From: dnkl @ 2021-10-01 14:55 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 249 bytes --]
New comment by dnkl on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-932301809
Comment:
I can do a 1.9.1 release, if that makes it easier for you. I haven't yet, to avoid unnecessary packaging churn.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (9 preceding siblings ...)
2021-10-01 14:55 ` dnkl
@ 2021-10-02 4:44 ` cinerea0
2021-10-02 21:23 ` [PR PATCH] [Updated] " ifreund
2021-10-02 21:56 ` [PR PATCH] [Merged]: foot: update to 1.9.2 ericonr
12 siblings, 0 replies; 14+ messages in thread
From: cinerea0 @ 2021-10-02 4:44 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 194 bytes --]
New comment by cinerea0 on void-packages repository
https://github.com/void-linux/void-packages/pull/32703#issuecomment-932680918
Comment:
@ifreund It looks like 1.9.1 has just been released.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR PATCH] [Updated] foot: update to 1.9.0.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (10 preceding siblings ...)
2021-10-02 4:44 ` cinerea0
@ 2021-10-02 21:23 ` ifreund
2021-10-02 21:56 ` [PR PATCH] [Merged]: foot: update to 1.9.2 ericonr
12 siblings, 0 replies; 14+ messages in thread
From: ifreund @ 2021-10-02 21:23 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1586 bytes --]
There is an updated pull request by ifreund against master on the void-packages repository
https://github.com/ifreund/void-packages foot
https://github.com/void-linux/void-packages/pull/32703
foot: update to 1.9.0.
<!-- 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/32703.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-foot-32703.patch --]
[-- Type: text/x-diff, Size: 985 bytes --]
From c1190ee51b7306cd7634100fe271d8b58d6ff0bf Mon Sep 17 00:00:00 2001
From: Isaac Freund <ifreund@ifreund.xyz>
Date: Fri, 27 Aug 2021 14:44:35 +0200
Subject: [PATCH] foot: update to 1.9.2.
---
| 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
--git a/srcpkgs/foot/template b/srcpkgs/foot/template
index 3dde8c741f24..fb29953d7d29 100644
--- a/srcpkgs/foot/template
+++ b/srcpkgs/foot/template
@@ -1,6 +1,6 @@
# Template file for 'foot'
pkgname=foot
-version=1.8.2
+version=1.9.2
revision=1
wrksrc=$pkgname
build_style=meson
@@ -15,7 +15,7 @@ license="MIT"
homepage="https://codeberg.org/dnkl/foot"
changelog="https://codeberg.org/dnkl/foot/src/branch/master/CHANGELOG.md"
distfiles="${homepage}/archive/${version}.tar.gz"
-checksum=50cf5b9f3cc1ebaafa62255eea22395e8267cce21e119bc6f7ecacf11f15dada
+checksum=5859d8d7293da4c7c52f45a9756fc7989edc364057e8b86b4e1fbea7bc2b4009
post_install() {
rm -f ${DESTDIR}/usr/share/doc/foot/LICENSE
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PR PATCH] [Merged]: foot: update to 1.9.2.
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
` (11 preceding siblings ...)
2021-10-02 21:23 ` [PR PATCH] [Updated] " ifreund
@ 2021-10-02 21:56 ` ericonr
12 siblings, 0 replies; 14+ messages in thread
From: ericonr @ 2021-10-02 21:56 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1433 bytes --]
There's a merged pull request on the void-packages repository
foot: update to 1.9.2.
https://github.com/void-linux/void-packages/pull/32703
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
-->
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2021-10-02 21:56 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-27 12:45 [PR PATCH] foot: update to 1.9.0 ifreund
2021-08-27 13:44 ` [PR REVIEW] " ericonr
2021-08-29 10:23 ` ram02z
2021-08-29 10:41 ` ifreund
2021-08-29 11:09 ` [PR PATCH] [Updated] " ifreund
2021-09-04 17:28 ` ram02z
2021-09-12 4:16 ` cinerea0
2021-09-27 3:34 ` ericonr
2021-09-27 7:36 ` stacyharper
2021-09-30 18:40 ` cinerea0
2021-10-01 14:55 ` dnkl
2021-10-02 4:44 ` cinerea0
2021-10-02 21:23 ` [PR PATCH] [Updated] " ifreund
2021-10-02 21:56 ` [PR PATCH] [Merged]: foot: update to 1.9.2 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).