Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] Tuba: update to 0.3.2
@ 2023-05-22  2:34 chrysos349
  2023-05-23  7:48 ` [PR REVIEW] " paper42
                   ` (23 more replies)
  0 siblings, 24 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-22  2:34 UTC (permalink / raw)
  To: ml

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

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

https://github.com/chrysos349/void-packages Tuba
https://github.com/void-linux/void-packages/pull/44024

Tuba: update to 0.3.2
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64-LIBC)

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

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

From 84fdf7973a2a5027bf325ea332c913072118f623 Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 22 May 2023 05:32:53 +0300
Subject: [PATCH] Tuba: update to 0.3.2

---
 srcpkgs/Tuba/template | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/Tuba/template b/srcpkgs/Tuba/template
index fc2974bde2e0..3df3485ffa91 100644
--- a/srcpkgs/Tuba/template
+++ b/srcpkgs/Tuba/template
@@ -1,14 +1,21 @@
 # Template file for 'Tuba'
 pkgname=Tuba
-version=0.2.0
+version=0.3.2
 revision=1
 build_style=meson
 configure_args="-Ddistro=true"
-hostmakedepends="pkg-config gettext desktop-file-utils vala glib-devel"
-makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel"
+hostmakedepends="desktop-file-utils gettext pkg-config"
+makedepends="gtksourceview5-devel libadwaita-devel libgee-devel libsecret-devel
+ libwebp-devel"
+depends="webp-pixbuf-loader"
+checkdepends="appstream-glib"
 short_desc="Browse the fediverse"
 maintainer="Michal Vasilek <michal@vasilek.cz>"
 license="GPL-3.0-only"
-homepage="https://tuba.geopjr.dev/"
-distfiles="https://github.com/GeopJr/Tuba/archive/refs/tags/v${version}.tar.gz"
-checksum=c698206bbbd06a1ce5a6e8cc9e6590411dc1cce1dde87fb0e34d8e35432ee2b1
+homepage="https://github.com/GeopJr/Tuba"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=1458b343f069efca00b0b5da49b9eb4b529e49de3d701da06d9529828be19cf9
+
+post_install() {
+	ln -s dev.geopjr.Tuba ${DESTDIR}/usr/bin/tuba
+}

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
  2023-05-23  7:48 ` [PR REVIEW] " paper42
@ 2023-05-23  7:48 ` paper42
  2023-05-23  7:48 ` paper42
                   ` (21 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23  7:48 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1201706840

Comment:
please don't randomly change the order, just add or remove what's necessary 

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
@ 2023-05-23  7:48 ` paper42
  2023-05-23  7:48 ` paper42
                   ` (22 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23  7:48 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1201708018

Comment:
why?

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
  2023-05-23  7:48 ` [PR REVIEW] " paper42
  2023-05-23  7:48 ` paper42
@ 2023-05-23  7:48 ` paper42
  2023-05-23 11:08 ` [PR PATCH] [Updated] " chrysos349
                   ` (20 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23  7:48 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1201707398

Comment:
no, revert to the previous version

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

* Re: [PR PATCH] [Updated] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (2 preceding siblings ...)
  2023-05-23  7:48 ` paper42
@ 2023-05-23 11:08 ` chrysos349
  2023-05-23 11:21 ` chrysos349
                   ` (19 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:08 UTC (permalink / raw)
  To: ml

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

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

https://github.com/chrysos349/void-packages Tuba
https://github.com/void-linux/void-packages/pull/44024

Tuba: update to 0.3.2
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64-LIBC)

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

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

From 76398371f247ffba070524032c97e9ea1f6c8bdf Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 22 May 2023 05:32:53 +0300
Subject: [PATCH] Tuba: update to 0.3.2

---
 srcpkgs/Tuba/template | 19 +++++++++++++------
 1 file changed, 13 insertions(+), 6 deletions(-)

diff --git a/srcpkgs/Tuba/template b/srcpkgs/Tuba/template
index fc2974bde2e0..3df3485ffa91 100644
--- a/srcpkgs/Tuba/template
+++ b/srcpkgs/Tuba/template
@@ -1,14 +1,21 @@
 # Template file for 'Tuba'
 pkgname=Tuba
-version=0.2.0
+version=0.3.2
 revision=1
 build_style=meson
 configure_args="-Ddistro=true"
-hostmakedepends="pkg-config gettext desktop-file-utils vala glib-devel"
-makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel"
+hostmakedepends="desktop-file-utils gettext pkg-config"
+makedepends="gtksourceview5-devel libadwaita-devel libgee-devel libsecret-devel
+ libwebp-devel"
+depends="webp-pixbuf-loader"
+checkdepends="appstream-glib"
 short_desc="Browse the fediverse"
 maintainer="Michal Vasilek <michal@vasilek.cz>"
 license="GPL-3.0-only"
-homepage="https://tuba.geopjr.dev/"
-distfiles="https://github.com/GeopJr/Tuba/archive/refs/tags/v${version}.tar.gz"
-checksum=c698206bbbd06a1ce5a6e8cc9e6590411dc1cce1dde87fb0e34d8e35432ee2b1
+homepage="https://github.com/GeopJr/Tuba"
+distfiles="${homepage}/archive/v${version}.tar.gz"
+checksum=1458b343f069efca00b0b5da49b9eb4b529e49de3d701da06d9529828be19cf9
+
+post_install() {
+	ln -s dev.geopjr.Tuba ${DESTDIR}/usr/bin/tuba
+}

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

* Re: [PR PATCH] [Updated] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (3 preceding siblings ...)
  2023-05-23 11:08 ` [PR PATCH] [Updated] " chrysos349
@ 2023-05-23 11:21 ` chrysos349
  2023-05-23 11:27 ` [PR REVIEW] " chrysos349
                   ` (18 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:21 UTC (permalink / raw)
  To: ml

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

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

https://github.com/chrysos349/void-packages Tuba
https://github.com/void-linux/void-packages/pull/44024

Tuba: update to 0.3.2
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64-LIBC)

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

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

From 46997dd801f4859f560123f2be56d230b10b902a Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 22 May 2023 05:32:53 +0300
Subject: [PATCH] Tuba: update to 0.3.2

---
 srcpkgs/Tuba/template | 15 +++++++++++----
 1 file changed, 11 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/Tuba/template b/srcpkgs/Tuba/template
index fc2974bde2e0..10be9d84a394 100644
--- a/srcpkgs/Tuba/template
+++ b/srcpkgs/Tuba/template
@@ -1,14 +1,21 @@
 # Template file for 'Tuba'
 pkgname=Tuba
-version=0.2.0
+version=0.3.2
 revision=1
 build_style=meson
 configure_args="-Ddistro=true"
-hostmakedepends="pkg-config gettext desktop-file-utils vala glib-devel"
-makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel"
+hostmakedepends="pkg-config gettext desktop-file-utils"
+makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel
+ libwebp-devel"
+depends="webp-pixbuf-loader"
+checkdepends="appstream-glib"
 short_desc="Browse the fediverse"
 maintainer="Michal Vasilek <michal@vasilek.cz>"
 license="GPL-3.0-only"
 homepage="https://tuba.geopjr.dev/"
 distfiles="https://github.com/GeopJr/Tuba/archive/refs/tags/v${version}.tar.gz"
-checksum=c698206bbbd06a1ce5a6e8cc9e6590411dc1cce1dde87fb0e34d8e35432ee2b1
+checksum=1458b343f069efca00b0b5da49b9eb4b529e49de3d701da06d9529828be19cf9
+
+post_install() {
+	ln -s dev.geopjr.Tuba ${DESTDIR}/usr/bin/tuba
+}

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (4 preceding siblings ...)
  2023-05-23 11:21 ` chrysos349
@ 2023-05-23 11:27 ` chrysos349
  2023-05-23 11:28 ` chrysos349
                   ` (17 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:27 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202126177

Comment:
Thanks for your feedback, @paper42 ! It turns out I messed with your template. Changes reverted.
`Vala` and `glib-devel` are deps of `libadwaita-devel`. So I removed them, correct?

BTW, the change is not random. The deps were sorted alphabetically. Am I not allowed to do this? Or is it easier to review changes when things stay the same?

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (5 preceding siblings ...)
  2023-05-23 11:27 ` [PR REVIEW] " chrysos349
@ 2023-05-23 11:28 ` chrysos349
  2023-05-23 11:30 ` chrysos349
                   ` (16 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:28 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202128492

Comment:
Reverted. Could you explain why? Is the old version of a `homepage` variable more preferable? Is the old version of a `distfiles` variable more preferable?

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (6 preceding siblings ...)
  2023-05-23 11:28 ` chrysos349
@ 2023-05-23 11:30 ` chrysos349
  2023-05-23 11:38 ` chrysos349
                   ` (15 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:30 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202130037

Comment:
Easier to launch from the terminal. Saw it in the AUR template, liked it. Is this a bad practice?

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

* Re: Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (7 preceding siblings ...)
  2023-05-23 11:30 ` chrysos349
@ 2023-05-23 11:38 ` chrysos349
  2023-05-23 11:46 ` [PR REVIEW] " chrysos349
                   ` (14 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:38 UTC (permalink / raw)
  To: ml

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

New comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#issuecomment-1559120150

Comment:
@paper42 Again, thanks for all your valuable feedback!

I read through [CONTRIBUTING.md](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md) and [Manual.md](https://github.com/void-linux/void-packages/blob/master/Manual.md) . I guess I need some more pointers in regards to other (unwritten) rules.

All I want is to help maintaining packages in my fav distro. That's all!

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (8 preceding siblings ...)
  2023-05-23 11:38 ` chrysos349
@ 2023-05-23 11:46 ` chrysos349
  2023-05-23 16:06 ` [PR PATCH] [Updated] " chrysos349
                   ` (13 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 11:46 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202126177

Comment:
Thanks for your feedback, @paper42 ! It turns out I messed with your template. Changes reverted.
`vala` and `glib-devel` are deps of `libadwaita-devel`. So I removed them, correct?

BTW, the change is not random. The deps were sorted alphabetically. Am I not allowed to do this? Or is it easier to review changes when things stay the same?

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

* Re: [PR PATCH] [Updated] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (9 preceding siblings ...)
  2023-05-23 11:46 ` [PR REVIEW] " chrysos349
@ 2023-05-23 16:06 ` chrysos349
  2023-05-23 16:07 ` [PR REVIEW] " chrysos349
                   ` (12 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 16:06 UTC (permalink / raw)
  To: ml

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

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

https://github.com/chrysos349/void-packages Tuba
https://github.com/void-linux/void-packages/pull/44024

Tuba: update to 0.3.2
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64-LIBC)

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

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

From dc57e6f5a4bdefedbc98a341cef28420c6407b76 Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 22 May 2023 05:32:53 +0300
Subject: [PATCH] Tuba: update to 0.3.2

---
 srcpkgs/Tuba/template | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/Tuba/template b/srcpkgs/Tuba/template
index fc2974bde2e0..0ec2767d4f70 100644
--- a/srcpkgs/Tuba/template
+++ b/srcpkgs/Tuba/template
@@ -1,14 +1,21 @@
 # Template file for 'Tuba'
 pkgname=Tuba
-version=0.2.0
+version=0.3.2
 revision=1
 build_style=meson
 configure_args="-Ddistro=true"
 hostmakedepends="pkg-config gettext desktop-file-utils vala glib-devel"
-makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel"
+makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel
+ libwebp-devel"
+depends="webp-pixbuf-loader"
+checkdepends="appstream-glib"
 short_desc="Browse the fediverse"
 maintainer="Michal Vasilek <michal@vasilek.cz>"
 license="GPL-3.0-only"
 homepage="https://tuba.geopjr.dev/"
 distfiles="https://github.com/GeopJr/Tuba/archive/refs/tags/v${version}.tar.gz"
-checksum=c698206bbbd06a1ce5a6e8cc9e6590411dc1cce1dde87fb0e34d8e35432ee2b1
+checksum=1458b343f069efca00b0b5da49b9eb4b529e49de3d701da06d9529828be19cf9
+
+post_install() {
+	ln -s dev.geopjr.Tuba ${DESTDIR}/usr/bin/tuba
+}

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (10 preceding siblings ...)
  2023-05-23 16:06 ` [PR PATCH] [Updated] " chrysos349
@ 2023-05-23 16:07 ` chrysos349
  2023-05-23 18:50 ` paper42
                   ` (11 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 16:07 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202632614

Comment:
I get it now. `vala` and `glibc-devel` are required for cross-compilation.

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (11 preceding siblings ...)
  2023-05-23 16:07 ` [PR REVIEW] " chrysos349
@ 2023-05-23 18:50 ` paper42
  2023-05-23 18:50 ` paper42
                   ` (10 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23 18:50 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202853867

Comment:
> BTW, the change is not random. The deps were sorted alphabetically. Am I not allowed to do this? Or is it easier to review changes when things stay the same?

Please don't do lint changes like this just for the sake of changing it. Do them only if you know this is a convention (and this is not).

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (12 preceding siblings ...)
  2023-05-23 18:50 ` paper42
@ 2023-05-23 18:50 ` paper42
  2023-05-23 18:50 ` paper42
                   ` (9 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23 18:50 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202853907

Comment:
homepage is supposed to be the homepage of the program, https://tuba.geopjr.dev/ is much much better for regular users than a github repository. Also, we are trying to avoid using $homepage in distfiles because it's an unnecessary variable for something that doesn't change and can mess with some scripts. As in the previous case, please don't apply these unnecessary fixes unless you know this style is preferred and keep the diff nice, small, clean and well review-able.

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (13 preceding siblings ...)
  2023-05-23 18:50 ` paper42
@ 2023-05-23 18:50 ` paper42
  2023-05-23 21:05 ` [PR PATCH] [Updated] " chrysos349
                   ` (8 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23 18:50 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1202854010

Comment:
I am against this

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

* Re: [PR PATCH] [Updated] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (14 preceding siblings ...)
  2023-05-23 18:50 ` paper42
@ 2023-05-23 21:05 ` chrysos349
  2023-05-23 21:10 ` [PR REVIEW] " chrysos349
                   ` (7 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 21:05 UTC (permalink / raw)
  To: ml

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

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

https://github.com/chrysos349/void-packages Tuba
https://github.com/void-linux/void-packages/pull/44024

Tuba: update to 0.3.2
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, (x86_64-LIBC)

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

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

From abdfb80936559a14081e0d49e97af9efdc0901fd Mon Sep 17 00:00:00 2001
From: chrysos349 <chrysostom349@gmail.com>
Date: Mon, 22 May 2023 05:32:53 +0300
Subject: [PATCH] Tuba: update to 0.3.2

---
 srcpkgs/Tuba/template | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/srcpkgs/Tuba/template b/srcpkgs/Tuba/template
index fc2974bde2e0..18cf6d24e3b2 100644
--- a/srcpkgs/Tuba/template
+++ b/srcpkgs/Tuba/template
@@ -1,14 +1,17 @@
 # Template file for 'Tuba'
 pkgname=Tuba
-version=0.2.0
+version=0.3.2
 revision=1
 build_style=meson
 configure_args="-Ddistro=true"
 hostmakedepends="pkg-config gettext desktop-file-utils vala glib-devel"
-makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel"
+makedepends="libadwaita-devel libgee-devel libsecret-devel gtksourceview5-devel
+ libwebp-devel"
+depends="webp-pixbuf-loader"
+checkdepends="appstream-glib"
 short_desc="Browse the fediverse"
 maintainer="Michal Vasilek <michal@vasilek.cz>"
 license="GPL-3.0-only"
 homepage="https://tuba.geopjr.dev/"
 distfiles="https://github.com/GeopJr/Tuba/archive/refs/tags/v${version}.tar.gz"
-checksum=c698206bbbd06a1ce5a6e8cc9e6590411dc1cce1dde87fb0e34d8e35432ee2b1
+checksum=1458b343f069efca00b0b5da49b9eb4b529e49de3d701da06d9529828be19cf9

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (15 preceding siblings ...)
  2023-05-23 21:05 ` [PR PATCH] [Updated] " chrysos349
@ 2023-05-23 21:10 ` chrysos349
  2023-05-23 21:12 ` chrysos349
                   ` (6 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 21:10 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1203016574

Comment:
This is useful! I will keep the diff small and clean in the future!

P.S. Doesn't this piece of information need to be in [CONTRIBUTING.md](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md)?

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (16 preceding siblings ...)
  2023-05-23 21:10 ` [PR REVIEW] " chrysos349
@ 2023-05-23 21:12 ` chrysos349
  2023-05-23 23:38 ` paper42
                   ` (5 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: chrysos349 @ 2023-05-23 21:12 UTC (permalink / raw)
  To: ml

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

New review comment by chrysos349 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1203018776

Comment:
Removed.

Please clarify. Are you against this in your templates? Or are you (as a member of VoidLinux) against such practice in any templates?

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (17 preceding siblings ...)
  2023-05-23 21:12 ` chrysos349
@ 2023-05-23 23:38 ` paper42
  2023-05-23 23:40 ` paper42
                   ` (4 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23 23:38 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1203157217

Comment:
I am personally against us (a distribution) creating a symlink for a graphical program just so it is easier to launch from CLI. We have to choose the same symlink name as all other distributions, make sure it doesn't conflict with any other package and somehow solve conflicts in that case. If people want to have an alias, they can set it, but launching a graphical application should be usually done with a graphical launcher which parses uses standard .desktop files instead of plain commands.

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (18 preceding siblings ...)
  2023-05-23 23:38 ` paper42
@ 2023-05-23 23:40 ` paper42
  2023-05-23 23:58 ` paper42
                   ` (3 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23 23:40 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1203159013

Comment:
I am not sure what exactly should be added there, keeping commits clean and not doing unrelated or unnecessary changes is just a basic git best practice.

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

* Re: [PR REVIEW] Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (19 preceding siblings ...)
  2023-05-23 23:40 ` paper42
@ 2023-05-23 23:58 ` paper42
  2023-06-04 10:52 ` Bnyro
                   ` (2 subsequent siblings)
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-05-23 23:58 UTC (permalink / raw)
  To: ml

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

New review comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#discussion_r1203157217

Comment:
I am personally against us (a distribution) creating a symlink for a graphical program just so it is easier to launch from CLI. We would have to choose the same symlink name as all other distributions (otherwise it would be very painful and confusing), make sure it doesn't conflict with any other package and somehow solve conflicts in that case. If people want to have an alias, they can set it on their own, but launching a graphical application should be usually done with a graphical launcher which parses uses standard .desktop files instead of plain commands.

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

* Re: Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (20 preceding siblings ...)
  2023-05-23 23:58 ` paper42
@ 2023-06-04 10:52 ` Bnyro
  2023-06-04 11:11 ` paper42
  2023-06-04 11:12 ` [PR PATCH] [Merged]: " paper42
  23 siblings, 0 replies; 27+ messages in thread
From: Bnyro @ 2023-06-04 10:52 UTC (permalink / raw)
  To: ml

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

New comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#issuecomment-1575517319

Comment:
Tested and everything seems to run fine on my machine!
Since `webp-pixbuf-loader` is only an optional dependency for webp support, I'm not sure if it should be added as a dependency :thinking:

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

* Re: Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (21 preceding siblings ...)
  2023-06-04 10:52 ` Bnyro
@ 2023-06-04 11:11 ` paper42
  2023-06-04 11:12 ` [PR PATCH] [Merged]: " paper42
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-06-04 11:11 UTC (permalink / raw)
  To: ml

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

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44024#issuecomment-1575524975

Comment:
> Since `webp-pixbuf-loader` is only an optional dependency for webp support, I'm not sure if it should be added as a dependency thinking

I think it's ok to add it in this case because some servers only apparently use webp and without this dependency, they wouldn't work.

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

* Re: [PR PATCH] [Merged]: Tuba: update to 0.3.2
  2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
                   ` (22 preceding siblings ...)
  2023-06-04 11:11 ` paper42
@ 2023-06-04 11:12 ` paper42
  23 siblings, 0 replies; 27+ messages in thread
From: paper42 @ 2023-06-04 11:12 UTC (permalink / raw)
  To: ml

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

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

Tuba: update to 0.3.2
https://github.com/void-linux/void-packages/pull/44024

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

#### Local build testing
- I built this PR locally for my native architecture, (x86_64-LIBC)

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

* Re: Tuba: update to 0.3.2
  2023-06-04 10:11 [PR PATCH] " Bnyro
  2023-06-04 10:25 ` paper42
@ 2023-06-04 10:27 ` Bnyro
  1 sibling, 0 replies; 27+ messages in thread
From: Bnyro @ 2023-06-04 10:27 UTC (permalink / raw)
  To: ml

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

New comment by Bnyro on void-packages repository

https://github.com/void-linux/void-packages/pull/44253#issuecomment-1575507988

Comment:
My bad!
I'll do that, sure!

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

* Re: Tuba: update to 0.3.2
  2023-06-04 10:11 [PR PATCH] " Bnyro
@ 2023-06-04 10:25 ` paper42
  2023-06-04 10:27 ` Bnyro
  1 sibling, 0 replies; 27+ messages in thread
From: paper42 @ 2023-06-04 10:25 UTC (permalink / raw)
  To: ml

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

New comment by paper42 on void-packages repository

https://github.com/void-linux/void-packages/pull/44253#issuecomment-1575507131

Comment:
duplicate of #44024, could you test that PR and comment your findings there?

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

end of thread, other threads:[~2023-06-04 11:12 UTC | newest]

Thread overview: 27+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-22  2:34 [PR PATCH] Tuba: update to 0.3.2 chrysos349
2023-05-23  7:48 ` [PR REVIEW] " paper42
2023-05-23  7:48 ` paper42
2023-05-23  7:48 ` paper42
2023-05-23 11:08 ` [PR PATCH] [Updated] " chrysos349
2023-05-23 11:21 ` chrysos349
2023-05-23 11:27 ` [PR REVIEW] " chrysos349
2023-05-23 11:28 ` chrysos349
2023-05-23 11:30 ` chrysos349
2023-05-23 11:38 ` chrysos349
2023-05-23 11:46 ` [PR REVIEW] " chrysos349
2023-05-23 16:06 ` [PR PATCH] [Updated] " chrysos349
2023-05-23 16:07 ` [PR REVIEW] " chrysos349
2023-05-23 18:50 ` paper42
2023-05-23 18:50 ` paper42
2023-05-23 18:50 ` paper42
2023-05-23 21:05 ` [PR PATCH] [Updated] " chrysos349
2023-05-23 21:10 ` [PR REVIEW] " chrysos349
2023-05-23 21:12 ` chrysos349
2023-05-23 23:38 ` paper42
2023-05-23 23:40 ` paper42
2023-05-23 23:58 ` paper42
2023-06-04 10:52 ` Bnyro
2023-06-04 11:11 ` paper42
2023-06-04 11:12 ` [PR PATCH] [Merged]: " paper42
2023-06-04 10:11 [PR PATCH] " Bnyro
2023-06-04 10:25 ` paper42
2023-06-04 10:27 ` Bnyro

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