Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] New package: fx-21.0.0
@ 2022-04-21 16:27 nulldutra
  2022-04-21 16:29 ` [PR REVIEW] " classabbyamp
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: nulldutra @ 2022-04-21 16:27 UTC (permalink / raw)
  To: ml

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

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

https://github.com/nulldutra/void-packages new-package-fx
https://github.com/void-linux/void-packages/pull/36805

New package: fx-21.0.0
https://github.com/antonmedv/fx

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

#### New package
- This new package conforms to the quality requirements: **YES**


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-new-package-fx-36805.patch --]
[-- Type: text/x-diff, Size: 944 bytes --]

From 6775eb3ad0cfb5ca5bee8938359313d140d69d24 Mon Sep 17 00:00:00 2001
From: "Gabriel M. Dutra" <nulldutra@gmail.com>
Date: Thu, 21 Apr 2022 13:20:50 -0300
Subject: [PATCH] New package: fx-21.0.0

---
 srcpkgs/fx/template | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 srcpkgs/fx/template

diff --git a/srcpkgs/fx/template b/srcpkgs/fx/template
new file mode 100644
index 000000000000..56305e90d09e
--- /dev/null
+++ b/srcpkgs/fx/template
@@ -0,0 +1,17 @@
+# Template file for 'fx'
+pkgname=fx
+version=21.0.0
+revision=1
+build_style=go
+go_import_path="github.com/antonmedv/fx"
+hostmakedepends="git"
+short_desc="Terminal JSON viewer"
+maintainer="Gabriel M. Dutra <nulldutra@gmail.com>"
+license="MIT"
+homepage="https://github.com/antonmedv/fx"
+distfiles="$homepage/archive/$version.tar.gz"
+checksum=4711c582ce69c53f5ee8a348d603e0fab5f302859a020609ef33c2ef566f0d68
+
+post_install() {
+	vlicense LICENSE
+}

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

* Re: [PR REVIEW] New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
  2022-04-21 16:29 ` [PR REVIEW] " classabbyamp
@ 2022-04-21 16:29 ` classabbyamp
  2022-04-22 15:59 ` [PR PATCH] [Updated] " nulldutra
                   ` (4 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: classabbyamp @ 2022-04-21 16:29 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/36805#discussion_r855376311

Comment:
why does this need git?

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

* Re: [PR REVIEW] New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
@ 2022-04-21 16:29 ` classabbyamp
  2022-04-21 16:29 ` classabbyamp
                   ` (5 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: classabbyamp @ 2022-04-21 16:29 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/36805#discussion_r855375853

Comment:
expand $homepage here

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

* Re: [PR PATCH] [Updated] New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
  2022-04-21 16:29 ` [PR REVIEW] " classabbyamp
  2022-04-21 16:29 ` classabbyamp
@ 2022-04-22 15:59 ` nulldutra
  2022-04-22 16:02 ` [PR REVIEW] " nulldutra
                   ` (3 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: nulldutra @ 2022-04-22 15:59 UTC (permalink / raw)
  To: ml

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

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

https://github.com/nulldutra/void-packages new-package-fx
https://github.com/void-linux/void-packages/pull/36805

New package: fx-21.0.0
https://github.com/antonmedv/fx

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

#### New package
- This new package conforms to the quality requirements: **YES**


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

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-new-package-fx-36805.patch --]
[-- Type: text/x-diff, Size: 1877 bytes --]

From 6775eb3ad0cfb5ca5bee8938359313d140d69d24 Mon Sep 17 00:00:00 2001
From: "Gabriel M. Dutra" <nulldutra@gmail.com>
Date: Thu, 21 Apr 2022 13:20:50 -0300
Subject: [PATCH 1/2] New package: fx-21.0.0

---
 srcpkgs/fx/template | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 srcpkgs/fx/template

diff --git a/srcpkgs/fx/template b/srcpkgs/fx/template
new file mode 100644
index 000000000000..56305e90d09e
--- /dev/null
+++ b/srcpkgs/fx/template
@@ -0,0 +1,17 @@
+# Template file for 'fx'
+pkgname=fx
+version=21.0.0
+revision=1
+build_style=go
+go_import_path="github.com/antonmedv/fx"
+hostmakedepends="git"
+short_desc="Terminal JSON viewer"
+maintainer="Gabriel M. Dutra <nulldutra@gmail.com>"
+license="MIT"
+homepage="https://github.com/antonmedv/fx"
+distfiles="$homepage/archive/$version.tar.gz"
+checksum=4711c582ce69c53f5ee8a348d603e0fab5f302859a020609ef33c2ef566f0d68
+
+post_install() {
+	vlicense LICENSE
+}

From 6ca7d448a1f59d0d6340ba3b3febd85126ac79a4 Mon Sep 17 00:00:00 2001
From: "Gabriel M. Dutra" <nulldutra@gmail.com>
Date: Fri, 22 Apr 2022 12:59:08 -0300
Subject: [PATCH 2/2] fx: removing git dependency and expand homepage variable

---
 srcpkgs/fx/template | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/srcpkgs/fx/template b/srcpkgs/fx/template
index 56305e90d09e..69c9d066de8a 100644
--- a/srcpkgs/fx/template
+++ b/srcpkgs/fx/template
@@ -4,12 +4,11 @@ version=21.0.0
 revision=1
 build_style=go
 go_import_path="github.com/antonmedv/fx"
-hostmakedepends="git"
 short_desc="Terminal JSON viewer"
 maintainer="Gabriel M. Dutra <nulldutra@gmail.com>"
 license="MIT"
 homepage="https://github.com/antonmedv/fx"
-distfiles="$homepage/archive/$version.tar.gz"
+distfiles="${homepage}/archive/$version.tar.gz"
 checksum=4711c582ce69c53f5ee8a348d603e0fab5f302859a020609ef33c2ef566f0d68
 
 post_install() {

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

* Re: [PR REVIEW] New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
                   ` (2 preceding siblings ...)
  2022-04-22 15:59 ` [PR PATCH] [Updated] " nulldutra
@ 2022-04-22 16:02 ` nulldutra
  2022-04-22 16:02 ` nulldutra
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 8+ messages in thread
From: nulldutra @ 2022-04-22 16:02 UTC (permalink / raw)
  To: ml

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

New review comment by nulldutra on void-packages repository

https://github.com/void-linux/void-packages/pull/36805#discussion_r856374029

Comment:
In other packages, $homepage variable is not expanded. What motivation for this? but, I made the adjustment in the variable.

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

* Re: [PR REVIEW] New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
                   ` (3 preceding siblings ...)
  2022-04-22 16:02 ` [PR REVIEW] " nulldutra
@ 2022-04-22 16:02 ` nulldutra
  2022-04-22 16:26 ` classabbyamp
  2022-04-22 18:35 ` abenson
  6 siblings, 0 replies; 8+ messages in thread
From: nulldutra @ 2022-04-22 16:02 UTC (permalink / raw)
  To: ml

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

New review comment by nulldutra on void-packages repository

https://github.com/void-linux/void-packages/pull/36805#discussion_r856374165

Comment:
Sorry, git is not needed. Removed!

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

* Re: [PR REVIEW] New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
                   ` (4 preceding siblings ...)
  2022-04-22 16:02 ` nulldutra
@ 2022-04-22 16:26 ` classabbyamp
  2022-04-22 18:35 ` abenson
  6 siblings, 0 replies; 8+ messages in thread
From: classabbyamp @ 2022-04-22 16:26 UTC (permalink / raw)
  To: ml

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

New review comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/36805#discussion_r856391752

Comment:
having variables other than $version (like $homepage or $pkgname) in distfiles urls cause more changes to be needed if one changes. some packages did this before and they haven't been changed yet.

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

* Re: New package: fx-21.0.0
  2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
                   ` (5 preceding siblings ...)
  2022-04-22 16:26 ` classabbyamp
@ 2022-04-22 18:35 ` abenson
  6 siblings, 0 replies; 8+ messages in thread
From: abenson @ 2022-04-22 18:35 UTC (permalink / raw)
  To: ml

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

New comment by abenson on void-packages repository

https://github.com/void-linux/void-packages/pull/36805#issuecomment-1106763878

Comment:
[Squash the two into a single commit.](https://www.git-tower.com/learn/git/faq/git-squash)

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

end of thread, other threads:[~2022-04-22 18:35 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-21 16:27 [PR PATCH] New package: fx-21.0.0 nulldutra
2022-04-21 16:29 ` [PR REVIEW] " classabbyamp
2022-04-21 16:29 ` classabbyamp
2022-04-22 15:59 ` [PR PATCH] [Updated] " nulldutra
2022-04-22 16:02 ` [PR REVIEW] " nulldutra
2022-04-22 16:02 ` nulldutra
2022-04-22 16:26 ` classabbyamp
2022-04-22 18:35 ` abenson

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