Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] newsboat: update to 2.22.1.
@ 2021-01-13 20:57 subsonik
  2021-01-13 21:28 ` Minoru
                   ` (7 more replies)
  0 siblings, 8 replies; 19+ messages in thread
From: subsonik @ 2021-01-13 20:57 UTC (permalink / raw)
  To: ml

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

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

https://github.com/subsonik/void-packages newsboat
https://github.com/void-linux/void-packages/pull/27902

newsboat: update to 2.22.1.


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

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

From 61a9e33e0f4269cd9e95e6f15efb03bbc06d80b1 Mon Sep 17 00:00:00 2001
From: John Rowley <hey@johnrowley.me>
Date: Wed, 13 Jan 2021 21:55:35 +0100
Subject: [PATCH] newsboat: update to 2.22.1.

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

diff --git a/srcpkgs/newsboat/template b/srcpkgs/newsboat/template
index e0ebdeeea50..d9fc235194c 100644
--- a/srcpkgs/newsboat/template
+++ b/srcpkgs/newsboat/template
@@ -1,6 +1,6 @@
 # Template file for 'newsboat'
 pkgname=newsboat
-version=2.21
+version=2.22.1
 revision=1
 build_style=configure
 build_helper="rust"
@@ -16,7 +16,7 @@ license="MIT"
 homepage="https://newsboat.org/"
 changelog="https://raw.githubusercontent.com/newsboat/newsboat/master/CHANGELOG.md"
 distfiles="https://newsboat.org/releases/${version}/newsboat-${version}.tar.xz"
-checksum=0c46b3dd46bb578dd6dd4915db4cfdffb4352ab258f251080ad14655c75a9c31
+checksum=8920f41cc53776b67c0e85ad1696b0967f6ac248f3b8913d977942c15d75e690
 
 do_check() {
 	make test

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
@ 2021-01-13 21:28 ` Minoru
  2021-01-13 22:01 ` subsonik
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 19+ messages in thread
From: Minoru @ 2021-01-13 21:28 UTC (permalink / raw)
  To: ml

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

New comment by Minoru on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-759750365

Comment:
Hi, Newsboat maintainer again :wave: Please see https://github.com/tibequadorian/void-packages/pull/5#issuecomment-758023098 and https://github.com/tibequadorian/void-packages/pull/5#issuecomment-758023098 (someone else was preparing a PR with an update, and I ended up commenting in their fork).

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
  2021-01-13 21:28 ` Minoru
@ 2021-01-13 22:01 ` subsonik
  2021-01-20  0:19 ` ericonr
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 19+ messages in thread
From: subsonik @ 2021-01-13 22:01 UTC (permalink / raw)
  To: ml

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

New comment by subsonik on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-759765730

Comment:
Thanks for the input @Minoru :)

I've ran the test suite on my machine using an x86_64 chroot and everything passed as an unprivileged user. The x86_64-musl tests fail, as previously noted, but I can confirm that the app itself is working fine on my machine (x86_64-musl).

I'm inexperienced with Void packaging/Travis, unfortunately, so I'm not sure if I can get the tests to run on Travis as an unprivileged user.

I think someone more knowledgeable will need to weigh in here, but in my opinion, besides the musl test failures and Travis running the test suite as a privileged user, this update is good to go. Maybe the test phase should be skipped?

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
  2021-01-13 21:28 ` Minoru
  2021-01-13 22:01 ` subsonik
@ 2021-01-20  0:19 ` ericonr
  2021-01-20  2:03 ` tibequadorian
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-01-20  0:19 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-763236473

Comment:
@tibequadorian do you think the changes from `post_patch` in https://github.com/tibequadorian/void-packages/pull/5/files are necessary?

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
                   ` (2 preceding siblings ...)
  2021-01-20  0:19 ` ericonr
@ 2021-01-20  2:03 ` tibequadorian
  2021-01-20  2:03 ` tibequadorian
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-01-20  2:03 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-763276139

Comment:
> @tibequadorian do you think the changes from `post_patch` in https://github.com/tibequadorian/void-packages/pull/5/files are necessary?

On @Minoru suggestion in https://github.com/tibequadorian/void-packages/pull/5#discussion_r555127025 I set `make_check_target=ci-check` and removed current `do_check` from the template. This fails with "Error opening terminal: unknown."
However the `ci-check` target in the Makefile looks quite similar to what was in the `do_check` function, so I kinda merged the idea in `post_patch()` and checks were indeed successful, but the CI still fails.

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
                   ` (3 preceding siblings ...)
  2021-01-20  2:03 ` tibequadorian
@ 2021-01-20  2:03 ` tibequadorian
  2021-01-20  2:04 ` tibequadorian
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-01-20  2:03 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-763276139

Comment:
> @tibequadorian do you think the changes from `post_patch` in https://github.com/tibequadorian/void-packages/pull/5/files are necessary?

On @Minoru suggestion in https://github.com/tibequadorian/void-packages/pull/5#discussion_r555127025 I set `make_check_target=ci-check` and removed current `do_check` from the template. This fails with "Error opening terminal: unknown."
However the `ci-check` target in the Makefile looks quite similar to what was in the `do_check` function, so I kinda merged the idea in `post_patch` and checks were indeed successful, but the CI still fails.

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
                   ` (4 preceding siblings ...)
  2021-01-20  2:03 ` tibequadorian
@ 2021-01-20  2:04 ` tibequadorian
  2021-01-20  2:06 ` tibequadorian
  2021-02-18  2:51 ` [PR PATCH] [Closed]: " ericonr
  7 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-01-20  2:04 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-763276139

Comment:
> @tibequadorian do you think the changes from `post_patch` in https://github.com/tibequadorian/void-packages/pull/5/files are necessary?

On @Minoru suggestion in https://github.com/tibequadorian/void-packages/pull/5#discussion_r555127025 I set `make_check_target=ci-check` and removed current `do_check` from the template. This fails with "Error opening terminal: unknown."
However the `ci-check` target in the Makefile looks quite similar to what was in the `do_check` function, so I kinda merged the idea in `post_patch` and checks were indeed successful on my machine, but the CI still fails.

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

* Re: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
                   ` (5 preceding siblings ...)
  2021-01-20  2:04 ` tibequadorian
@ 2021-01-20  2:06 ` tibequadorian
  2021-02-18  2:51 ` [PR PATCH] [Closed]: " ericonr
  7 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-01-20  2:06 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/27902#issuecomment-763276139

Comment:
> @tibequadorian do you think the changes from `post_patch` in https://github.com/tibequadorian/void-packages/pull/5/files are necessary?

On @Minoru suggestion in https://github.com/tibequadorian/void-packages/pull/5#discussion_r555127025 I set `make_check_target=ci-check` and removed current `do_check` from the template. This fails with "Error opening terminal: unknown."
However parts of the `ci-check` target in the Makefile look quite similar to what was in the `do_check` function, so I kinda merged the idea in `post_patch` and checks were indeed successful on my machine, but the CI still fails.

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

* Re: [PR PATCH] [Closed]: newsboat: update to 2.22.1.
  2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
                   ` (6 preceding siblings ...)
  2021-01-20  2:06 ` tibequadorian
@ 2021-02-18  2:51 ` ericonr
  7 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-02-18  2:51 UTC (permalink / raw)
  To: ml

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

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

newsboat: update to 2.22.1.
https://github.com/void-linux/void-packages/pull/27902

Description:


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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (8 preceding siblings ...)
  2021-02-18  1:16 ` tibequadorian
@ 2021-02-18  1:17 ` tibequadorian
  9 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-02-18  1:17 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780965076

Comment:
> Maybe `export TERM=$TERM` ?

isnt that the same as `export TERM`? 
edit: apparently it isnt

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (7 preceding siblings ...)
  2021-02-18  1:14 ` tibequadorian
@ 2021-02-18  1:16 ` tibequadorian
  2021-02-18  1:17 ` tibequadorian
  9 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-02-18  1:16 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780965076

Comment:
> Maybe `export TERM=$TERM` ?

isnt that the same as `export TERM`? 
edit: apparently it isnt

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (6 preceding siblings ...)
  2021-02-18  1:06 ` ericonr
@ 2021-02-18  1:14 ` tibequadorian
  2021-02-18  1:16 ` tibequadorian
  2021-02-18  1:17 ` tibequadorian
  9 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-02-18  1:14 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780965076

Comment:
> Maybe `export TERM=$TERM` ?

isnt that the same as `export TERM`?

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (5 preceding siblings ...)
  2021-02-18  1:06 ` ericonr
@ 2021-02-18  1:06 ` ericonr
  2021-02-18  1:14 ` tibequadorian
                   ` (2 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-02-18  1:06 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780961013

Comment:
In that case dumb would have been fine :)

I didn't know there was a default, so xterm felt default enough to me.

Maybe `export TERM=$TERM` ?

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (4 preceding siblings ...)
  2021-02-17 23:29 ` tibequadorian
@ 2021-02-18  1:06 ` ericonr
  2021-02-18  1:06 ` ericonr
                   ` (3 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-02-18  1:06 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780961013

Comment:
In that case dumb would have been fine :)

I didn't know there was a default, so xterm felt default enough to me.

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (3 preceding siblings ...)
  2021-02-17 23:29 ` tibequadorian
@ 2021-02-17 23:29 ` tibequadorian
  2021-02-18  1:06 ` ericonr
                   ` (4 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-02-17 23:29 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780922399

Comment:
> export TERM=xterm

When using vsed it evaluates for me to TERM=dumb

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
                   ` (2 preceding siblings ...)
  2021-02-17 20:17 ` ericonr
@ 2021-02-17 23:29 ` tibequadorian
  2021-02-17 23:29 ` tibequadorian
                   ` (5 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: tibequadorian @ 2021-02-17 23:29 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780922399

Comment:
> export TERM=xterm
When using vsed it evaluates for me to TERM=dumb

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
  2021-02-17 14:49 ` ericonr
  2021-02-17 19:46 ` Minoru
@ 2021-02-17 20:17 ` ericonr
  2021-02-17 23:29 ` tibequadorian
                   ` (6 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-02-17 20:17 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780826083

Comment:
I prefer 

```
diff --git a/srcpkgs/newsboat/template b/srcpkgs/newsboat/template
index e1a4429a62..7960b5d411 100644
--- a/srcpkgs/newsboat/template
+++ b/srcpkgs/newsboat/template
@@ -22,8 +22,8 @@ checksum=8920f41cc53776b67c0e85ad1696b0967f6ac248f3b8913d977942c15d75e690
 # tests fail when run by superuser
 make_check=extended
 
-post_patch() {
-	vsed -i Makefile -e "s|\./test|TERM=$TERM \./test|g"
+pre_check() {
+	export TERM=xterm
 }
 
 post_install() {
```

instead of `vsed`. Also add a comment about the tests failing on musl with a link to the issue.

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
  2021-02-17 14:49 ` ericonr
@ 2021-02-17 19:46 ` Minoru
  2021-02-17 20:17 ` ericonr
                   ` (7 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: Minoru @ 2021-02-17 19:46 UTC (permalink / raw)
  To: ml

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

New comment by Minoru on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780806866

Comment:
Hi, Newsboat maintainer here :wave: Tests on musl [have been broken since at least 2.21](https://github.com/newsboat/newsboat/issues/1216), and it looks like no-one affected is interested in looking into that, while I keep distracting myself with other stuff, so I don't think this should be a blocker to this package either.

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

* Re: newsboat: update to 2.22.1.
  2021-02-16 22:09 [PR PATCH] " tibequadorian
@ 2021-02-17 14:49 ` ericonr
  2021-02-17 19:46 ` Minoru
                   ` (8 subsequent siblings)
  9 siblings, 0 replies; 19+ messages in thread
From: ericonr @ 2021-02-17 14:49 UTC (permalink / raw)
  To: ml

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/28801#issuecomment-780607261

Comment:
Tests do run on glibc, but seem to fail on musl: https://gist.github.com/ericonr/74f59140ac7e26a0ac122888d4c22cb5

I'm running tests for the older version, and if it's also broken there I don't think it should be a blocker. Would you mind reporting this to upstream, though?

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

end of thread, other threads:[~2021-02-18  2:51 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-13 20:57 [PR PATCH] newsboat: update to 2.22.1 subsonik
2021-01-13 21:28 ` Minoru
2021-01-13 22:01 ` subsonik
2021-01-20  0:19 ` ericonr
2021-01-20  2:03 ` tibequadorian
2021-01-20  2:03 ` tibequadorian
2021-01-20  2:04 ` tibequadorian
2021-01-20  2:06 ` tibequadorian
2021-02-18  2:51 ` [PR PATCH] [Closed]: " ericonr
2021-02-16 22:09 [PR PATCH] " tibequadorian
2021-02-17 14:49 ` ericonr
2021-02-17 19:46 ` Minoru
2021-02-17 20:17 ` ericonr
2021-02-17 23:29 ` tibequadorian
2021-02-17 23:29 ` tibequadorian
2021-02-18  1:06 ` ericonr
2021-02-18  1:06 ` ericonr
2021-02-18  1:14 ` tibequadorian
2021-02-18  1:16 ` tibequadorian
2021-02-18  1:17 ` tibequadorian

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).