unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0.
@ 2023-03-20 22:47 Sergey Trofimov
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
  0 siblings, 2 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-20 22:47 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

This is a series of patches to update fdroidcl to 0.7.0. I have built it
locally (x86_64) and verified that the application works correctly.

Sergey Trofimov (4):
  gnu: Add go-github-com-schollz-progressbar-v3.
  gnu: Add go-github-com-mitchellh-colorstring.
  gnu: Add go-github-com-k0kubun-go-ansi.
  gnu: fdroidcl: Update to 0.7.0.

 gnu/packages/android.scm | 14 ++++----
 gnu/packages/golang.scm  | 76 ++++++++++++++++++++++++++++++++++++++++
 2 files changed, 83 insertions(+), 7 deletions(-)

-- 
2.39.2





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

* [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3.
  2023-03-20 22:47 [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Sergey Trofimov
@ 2023-03-20 22:51 ` Sergey Trofimov
  2023-03-20 22:51   ` [bug#62310] [PATCH 2/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
                     ` (4 more replies)
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
  1 sibling, 5 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-20 22:51 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/golang.scm (go-github-com-schollz-progressbar-v3):
New variable.
---
 gnu/packages/golang.scm | 32 ++++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index c75fdb4a31..e9187a14e9 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -11304,6 +11304,38 @@ (define-public go-sigs-k8s-io-yaml
 @url{https://github.com/ghodss/yaml,ghodss/yaml}.")
     (license (list license:expat license:bsd-3))))
 
+(define-public go-github-com-schollz-progressbar-v3
+  (package
+    (name "go-github-com-schollz-progressbar-v3")
+    (version "3.13.1")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/schollz/progressbar")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1hjahr5r52i7w6iyvl3rpzr46iignhfdh4694fl7m2b4gkaw9gd6"))))
+    (build-system go-build-system)
+    (arguments
+     '(#:import-path "github.com/schollz/progressbar/v3"
+       #:tests? #f))
+    (propagated-inputs (list go-golang-org-x-term
+                             go-github-com-stretchr-testify
+                             go-github-com-mitchellh-colorstring
+                             go-github-com-mattn-go-runewidth
+                             go-github-com-mattn-go-isatty
+                             go-github-com-k0kubun-go-ansi
+                             go-github-com-davecgh-go-spew))
+    (home-page "https://github.com/schollz/progressbar")
+    (synopsis "Simple cross-platform progress bar.")
+    (description
+     "This package provides a very simple thread-safe progress bar which should work
+on every OS without problems.  In order to be OS agnostic support of multi-line
+outputs is not provided.")
+    (license license:expat)))
+
 (define-public go-git-sr-ht-emersion-go-scfg
   (package
     (name "go-git-sr-ht-emersion-go-scfg")
-- 
2.39.2





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

* [bug#62310] [PATCH 2/4] gnu: Add go-github-com-mitchellh-colorstring.
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
@ 2023-03-20 22:51   ` Sergey Trofimov
  2023-03-21  1:47     ` [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer
  2023-03-20 22:51   ` [bug#62310] [PATCH 3/4] gnu: Add go-github-com-k0kubun-go-ansi Sergey Trofimov
                     ` (3 subsequent siblings)
  4 siblings, 1 reply; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-20 22:51 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/golang.scm (go-github-com-mitchellh-colorstring):
New variable.
---
 gnu/packages/golang.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index e9187a14e9..002475093a 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -11336,6 +11336,28 @@ (define-public go-github-com-schollz-progressbar-v3
 outputs is not provided.")
     (license license:expat)))
 
+(define-public go-github-com-mitchellh-colorstring
+  (package
+    (name "go-github-com-mitchellh-colorstring")
+    (version "0.0.0-20190213212951-d06e56a500db")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/mitchellh/colorstring")
+                    (commit (go-version->git-ref version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1d2mi5ziszfzdgaz8dg4b6sxa63nw1jnsvffacqxky6yz9m623kn"))))
+    (build-system go-build-system)
+    (arguments
+     '(#:import-path "github.com/mitchellh/colorstring"))
+    (home-page "https://github.com/mitchellh/colorstring")
+    (synopsis "Functions to colorize strings for terminal output.")
+    (description
+     "Colorstring provides functions for colorizing strings for terminal output.")
+    (license license:expat)))
+
 (define-public go-git-sr-ht-emersion-go-scfg
   (package
     (name "go-git-sr-ht-emersion-go-scfg")
-- 
2.39.2





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

* [bug#62310] [PATCH 3/4] gnu: Add go-github-com-k0kubun-go-ansi.
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
  2023-03-20 22:51   ` [bug#62310] [PATCH 2/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
@ 2023-03-20 22:51   ` Sergey Trofimov
  2023-03-21  1:55     ` [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer
  2023-03-20 22:51   ` [bug#62310] [PATCH] " Sergey Trofimov
                     ` (2 subsequent siblings)
  4 siblings, 1 reply; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-20 22:51 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/golang.scm (go-github-com-k0kubun-go-ansi):
New variable.
---
 gnu/packages/golang.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 002475093a..b831a89872 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -11358,6 +11358,28 @@ (define-public go-github-com-mitchellh-colorstring
      "Colorstring provides functions for colorizing strings for terminal output.")
     (license license:expat)))
 
+(define-public go-github-com-k0kubun-go-ansi
+  (package
+    (name "go-github-com-k0kubun-go-ansi")
+    (version "0.0.0-20180517002512-3bf9e2903213")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/k0kubun/go-ansi")
+                    (commit (go-version->git-ref version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "117afax4l268rbswf02icbgxncmd1pk2abkz7cv26iyszi8l26dq"))))
+    (build-system go-build-system)
+    (arguments
+     '(#:import-path "github.com/k0kubun/go-ansi"))
+    (home-page "https://github.com/k0kubun/go-ansi")
+    (synopsis "Go library for ANSI escape sequences.")
+    (description
+     "Windows-portable ANSI escape sequence utility for Go language")
+    (license license:expat)))
+
 (define-public go-git-sr-ht-emersion-go-scfg
   (package
     (name "go-git-sr-ht-emersion-go-scfg")
-- 
2.39.2





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

* [bug#62310] [PATCH] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
  2023-03-20 22:51   ` [bug#62310] [PATCH 2/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
  2023-03-20 22:51   ` [bug#62310] [PATCH 3/4] gnu: Add go-github-com-k0kubun-go-ansi Sergey Trofimov
@ 2023-03-20 22:51   ` Sergey Trofimov
  2023-03-21  1:58     ` [bug#62310] [PATCH 0/4] " Maxim Cournoyer
  2023-03-21  1:43   ` Maxim Cournoyer
  2023-03-21  1:46   ` Maxim Cournoyer
  4 siblings, 1 reply; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-20 22:51 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/android.scm (fdroidcl): Update to 0.7.0.
---
 gnu/packages/android.scm | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/android.scm b/gnu/packages/android.scm
index 881b75d763..9d67dca0cd 100644
--- a/gnu/packages/android.scm
+++ b/gnu/packages/android.scm
@@ -1195,25 +1195,25 @@ (define-public fdroidserver
 (define-public fdroidcl
   (package
     (name "fdroidcl")
-    (version "0.5.0")
+    (version "0.7.0")
     (source (origin
               (method git-fetch)
               (uri (git-reference
-                     (url "https://github.com/mvdan/fdroidcl")
-                     (commit (string-append "v" version))))
+                    (url "https://github.com/mvdan/fdroidcl")
+                    (commit (string-append "v" version))))
               (file-name (git-file-name name version))
               (sha256
-               (base32 "1rxcdyy2j34z0ql9d62w7ivsch9xihjnpb1z9kgy9q46vl8zhhy0"))))
+               (base32
+                "1s3fszlyyab9gbdrg52zcbafsb1mgb770skg7b3gj7f3pzfnra5n"))))
     (build-system go-build-system)
     (arguments
      `(#:import-path "mvdan.cc/fdroidcl"
-       #:tests? #f  ; TODO: Inputs missing.
+       #:tests? #f ;requires internet access
        #:install-source? #f))
-    (inputs
-     (list go-github-com-kr-pretty))
-    ;(native-inputs
-    ; `(("go-github-com-rogpeppe-go-internal-testscript"
-    ;    ,go-github-com-rogpeppe-go-internal-testscript)))
+    (inputs (list go-github-com-kr-pretty go-github-com-schollz-progressbar-v3))
+    ;; these are for tests, but the tests are disabled
+    ;; (native-inputs
+    ;; (list go-github-com-rogpeppe-go-internal go-gopkg-in-check-v1))
     (synopsis "F-Droid desktop client")
     (description
      "While the Android client integrates with the system with regular update
-- 
2.39.2





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

* [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
                     ` (2 preceding siblings ...)
  2023-03-20 22:51   ` [bug#62310] [PATCH] " Sergey Trofimov
@ 2023-03-21  1:43   ` Maxim Cournoyer
  2023-03-21  1:46   ` Maxim Cournoyer
  4 siblings, 0 replies; 16+ messages in thread
From: Maxim Cournoyer @ 2023-03-21  1:43 UTC (permalink / raw)
  To: Sergey Trofimov; +Cc: 62310

Hi!

Sergey Trofimov <sarg@sarg.org.ru> writes:

> * gnu/packages/golang.scm (go-github-com-schollz-progressbar-v3):
> New variable.
> ---
>  gnu/packages/golang.scm | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>
> diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
> index c75fdb4a31..e9187a14e9 100644
> --- a/gnu/packages/golang.scm
> +++ b/gnu/packages/golang.scm
> @@ -11304,6 +11304,38 @@ (define-public go-sigs-k8s-io-yaml
>  @url{https://github.com/ghodss/yaml,ghodss/yaml}.")
>      (license (list license:expat license:bsd-3))))
>  
> +(define-public go-github-com-schollz-progressbar-v3
> +  (package
> +    (name "go-github-com-schollz-progressbar-v3")
> +    (version "3.13.1")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/schollz/progressbar")
> +                    (commit (string-append "v" version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "1hjahr5r52i7w6iyvl3rpzr46iignhfdh4694fl7m2b4gkaw9gd6"))))
> +    (build-system go-build-system)
> +    (arguments
> +     '(#:import-path "github.com/schollz/progressbar/v3"
> +       #:tests? #f))

There seems to be a simple test included; it'd be preferable to run it.
If there are reasons against running it, they should be explained in a
comment.

> +    (propagated-inputs (list go-golang-org-x-term
> +                             go-github-com-stretchr-testify
> +                             go-github-com-mitchellh-colorstring
> +                             go-github-com-mattn-go-runewidth
> +                             go-github-com-mattn-go-isatty
> +                             go-github-com-k0kubun-go-ansi
> +                             go-github-com-davecgh-go-spew))
> +    (home-page "https://github.com/schollz/progressbar")
> +    (synopsis "Simple cross-platform progress bar.")
> +    (description
> +     "This package provides a very simple thread-safe progress bar which should work
> +on every OS without problems.  In order to be OS agnostic support of multi-line
> +outputs is not provided.")
> +    (license license:expat)))

Cross-platform or Windows-compatible or this kind of qualities are not
useful information for Guix users, as Guix can only run on top of
GNU(/Linux) systems anyway, so I'd remove such text from the
description.

Could you please send a v2?

-- 
Thanks,
Maxim




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

* [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
                     ` (3 preceding siblings ...)
  2023-03-21  1:43   ` Maxim Cournoyer
@ 2023-03-21  1:46   ` Maxim Cournoyer
  4 siblings, 0 replies; 16+ messages in thread
From: Maxim Cournoyer @ 2023-03-21  1:46 UTC (permalink / raw)
  To: Sergey Trofimov; +Cc: 62310

Hi,

Sorry, I forgot a couple comments;

Sergey Trofimov <sarg@sarg.org.ru> writes:

> * gnu/packages/golang.scm (go-github-com-schollz-progressbar-v3):
> New variable.

Nitpick; but it'd be more conventional to break the line this:

* gnu/packages/golang.scm
(go-github-com-schollz-progressbar-v3): New variable.

The essence of it is explained in info '(standards) Style of Change
Logs'.

> ---
>  gnu/packages/golang.scm | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>
> diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
> index c75fdb4a31..e9187a14e9 100644
> --- a/gnu/packages/golang.scm
> +++ b/gnu/packages/golang.scm
> @@ -11304,6 +11304,38 @@ (define-public go-sigs-k8s-io-yaml
>  @url{https://github.com/ghodss/yaml,ghodss/yaml}.")
>      (license (list license:expat license:bsd-3))))
>  
> +(define-public go-github-com-schollz-progressbar-v3
> +  (package
> +    (name "go-github-com-schollz-progressbar-v3")
> +    (version "3.13.1")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/schollz/progressbar")
> +                    (commit (string-append "v" version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "1hjahr5r52i7w6iyvl3rpzr46iignhfdh4694fl7m2b4gkaw9gd6"))))
> +    (build-system go-build-system)
> +    (arguments
> +     '(#:import-path "github.com/schollz/progressbar/v3"
> +       #:tests? #f))
> +    (propagated-inputs (list go-golang-org-x-term
> +                             go-github-com-stretchr-testify
> +                             go-github-com-mitchellh-colorstring
> +                             go-github-com-mattn-go-runewidth
> +                             go-github-com-mattn-go-isatty
> +                             go-github-com-k0kubun-go-ansi
> +                             go-github-com-davecgh-go-spew))
> +    (home-page "https://github.com/schollz/progressbar")
> +    (synopsis "Simple cross-platform progress bar.")

synopsis should not end with a trailing period (they are not complete
sentence).  I think 'guix lint' should complain about that.

-- 
Thanks,
Maxim




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

* [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:51   ` [bug#62310] [PATCH 2/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
@ 2023-03-21  1:47     ` Maxim Cournoyer
  0 siblings, 0 replies; 16+ messages in thread
From: Maxim Cournoyer @ 2023-03-21  1:47 UTC (permalink / raw)
  To: Sergey Trofimov; +Cc: 62310

Hi!

Sergey Trofimov <sarg@sarg.org.ru> writes:

> * gnu/packages/golang.scm (go-github-com-mitchellh-colorstring):
> New variable.
> ---
>  gnu/packages/golang.scm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
> index e9187a14e9..002475093a 100644
> --- a/gnu/packages/golang.scm
> +++ b/gnu/packages/golang.scm
> @@ -11336,6 +11336,28 @@ (define-public go-github-com-schollz-progressbar-v3
>  outputs is not provided.")
>      (license license:expat)))
>  
> +(define-public go-github-com-mitchellh-colorstring
> +  (package
> +    (name "go-github-com-mitchellh-colorstring")
> +    (version "0.0.0-20190213212951-d06e56a500db")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/mitchellh/colorstring")
> +                    (commit (go-version->git-ref version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "1d2mi5ziszfzdgaz8dg4b6sxa63nw1jnsvffacqxky6yz9m623kn"))))
> +    (build-system go-build-system)
> +    (arguments
> +     '(#:import-path "github.com/mitchellh/colorstring"))
> +    (home-page "https://github.com/mitchellh/colorstring")
> +    (synopsis "Functions to colorize strings for terminal output.")

Here also, the trailing period can be removed.

-- 
Thanks,
Maxim




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

* [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:51   ` [bug#62310] [PATCH 3/4] gnu: Add go-github-com-k0kubun-go-ansi Sergey Trofimov
@ 2023-03-21  1:55     ` Maxim Cournoyer
  0 siblings, 0 replies; 16+ messages in thread
From: Maxim Cournoyer @ 2023-03-21  1:55 UTC (permalink / raw)
  To: Sergey Trofimov; +Cc: 62310

Sergey Trofimov <sarg@sarg.org.ru> writes:

> * gnu/packages/golang.scm (go-github-com-k0kubun-go-ansi):
> New variable.
> ---
>  gnu/packages/golang.scm | 22 ++++++++++++++++++++++
>  1 file changed, 22 insertions(+)
>
> diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
> index 002475093a..b831a89872 100644
> --- a/gnu/packages/golang.scm
> +++ b/gnu/packages/golang.scm
> @@ -11358,6 +11358,28 @@ (define-public go-github-com-mitchellh-colorstring
>       "Colorstring provides functions for colorizing strings for terminal output.")
>      (license license:expat)))
>  
> +(define-public go-github-com-k0kubun-go-ansi
> +  (package
> +    (name "go-github-com-k0kubun-go-ansi")
> +    (version "0.0.0-20180517002512-3bf9e2903213")
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/k0kubun/go-ansi")
> +                    (commit (go-version->git-ref version))))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +                "117afax4l268rbswf02icbgxncmd1pk2abkz7cv26iyszi8l26dq"))))
> +    (build-system go-build-system)
> +    (arguments
> +     '(#:import-path "github.com/k0kubun/go-ansi"))
> +    (home-page "https://github.com/k0kubun/go-ansi")
> +    (synopsis "Go library for ANSI escape sequences.")

No trailing period for synopsis,

> +    (description
> +     "Windows-portable ANSI escape sequence utility for Go language")

But you need one for the description.  The project page projects a
better one, which reads "This library converts ANSI escape sequences to
Windows API calls on Windows environments." -- I'd use it for the
description.  It makes me wonder though, if it's of any use to Guix
users, since Guix is not going to produce Windows Go binaries that could
run natively on Windows... Could you test if the package that wants this
can be built without it?  It may well be optional.

-- 
Thanks,
Maxim




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

* [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:51   ` [bug#62310] [PATCH] " Sergey Trofimov
@ 2023-03-21  1:58     ` Maxim Cournoyer
  0 siblings, 0 replies; 16+ messages in thread
From: Maxim Cournoyer @ 2023-03-21  1:58 UTC (permalink / raw)
  To: Sergey Trofimov; +Cc: 62310

Hi!

Sergey Trofimov <sarg@sarg.org.ru> writes:

> * gnu/packages/android.scm (fdroidcl): Update to 0.7.0.
> ---
>  gnu/packages/android.scm | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/gnu/packages/android.scm b/gnu/packages/android.scm
> index 881b75d763..9d67dca0cd 100644
> --- a/gnu/packages/android.scm
> +++ b/gnu/packages/android.scm
> @@ -1195,25 +1195,25 @@ (define-public fdroidserver
>  (define-public fdroidcl
>    (package
>      (name "fdroidcl")
> -    (version "0.5.0")
> +    (version "0.7.0")
>      (source (origin
>                (method git-fetch)
>                (uri (git-reference
> -                     (url "https://github.com/mvdan/fdroidcl")
> -                     (commit (string-append "v" version))))
> +                    (url "https://github.com/mvdan/fdroidcl")
> +                    (commit (string-append "v" version))))
>                (file-name (git-file-name name version))
>                (sha256
> -               (base32 "1rxcdyy2j34z0ql9d62w7ivsch9xihjnpb1z9kgy9q46vl8zhhy0"))))
> +               (base32
> +                "1s3fszlyyab9gbdrg52zcbafsb1mgb770skg7b3gj7f3pzfnra5n"))))
>      (build-system go-build-system)
>      (arguments
>       `(#:import-path "mvdan.cc/fdroidcl"
> -       #:tests? #f  ; TODO: Inputs missing.
> +       #:tests? #f ;requires internet access

Perhaps, ";tests require internet access"

>         #:install-source? #f))
> -    (inputs
> -     (list go-github-com-kr-pretty))
> -    ;(native-inputs
> -    ; `(("go-github-com-rogpeppe-go-internal-testscript"
> -    ;    ,go-github-com-rogpeppe-go-internal-testscript)))
> +    (inputs (list go-github-com-kr-pretty go-github-com-schollz-progressbar-v3))
> +    ;; these are for tests, but the tests are disabled
> +    ;; (native-inputs
> +    ;; (list go-github-com-rogpeppe-go-internal go-gopkg-in-check-v1))

I think I'd remove the native-inputs completely instead of keeping dead
code around.

Thanks for the contribution!  I look forward to v2.

-- 
Maxim




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

* [bug#62310] [PATCH v2 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-20 22:47 [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Sergey Trofimov
  2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
@ 2023-03-21  8:04 ` Sergey Trofimov
  2023-03-21  8:15   ` [bug#62310] [PATCH v2 1/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
                     ` (4 more replies)
  1 sibling, 5 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-21  8:04 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov, maxim.cournoyer

I have addressed Maxim's comments and made the following changes:
* enabled tests.short for progressbar-v3 (full suite requires internet access)
* removed optional go-ansi dependency
* reworded synopsysises/descriptions, fixed formatting
* moved check-synopsis-style to %local-checkers, as I usually `lint -n` because of a very slow CVE linter

Sergey Trofimov (4):
  gnu: Add go-github-com-schollz-progressbar-v3.
  gnu: Add go-github-com-mitchellh-colorstring.
  gnu: fdroidcl: Update to 0.7.0.
  guix: Put check-synopsis-style to the local checkers list.

 gnu/packages/android.scm | 17 +++++------
 gnu/packages/golang.scm  | 63 ++++++++++++++++++++++++++++++++++++++++
 guix/lint.scm            |  9 +++---
 3 files changed, 75 insertions(+), 14 deletions(-)

-- 
2.39.2





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

* [bug#62310] [PATCH v2 1/4] gnu: Add go-github-com-mitchellh-colorstring.
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
@ 2023-03-21  8:15   ` Sergey Trofimov
  2023-03-21  8:16   ` [bug#62310] [PATCH v2 2/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
                     ` (3 subsequent siblings)
  4 siblings, 0 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-21  8:15 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/golang.scm
(go-github-com-mitchellh-colorstring): New variable.
---
 gnu/packages/golang.scm | 22 ++++++++++++++++++++++
 1 file changed, 22 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index c75fdb4a31..3ac4863e85 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -11304,6 +11304,28 @@ (define-public go-sigs-k8s-io-yaml
 @url{https://github.com/ghodss/yaml,ghodss/yaml}.")
     (license (list license:expat license:bsd-3))))
 
+(define-public go-github-com-mitchellh-colorstring
+  (package
+    (name "go-github-com-mitchellh-colorstring")
+    (version "0.0.0-20190213212951-d06e56a500db")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/mitchellh/colorstring")
+                    (commit (go-version->git-ref version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1d2mi5ziszfzdgaz8dg4b6sxa63nw1jnsvffacqxky6yz9m623kn"))))
+    (build-system go-build-system)
+    (arguments
+     '(#:import-path "github.com/mitchellh/colorstring"))
+    (home-page "https://github.com/mitchellh/colorstring")
+    (synopsis "Functions to colorize strings for terminal output")
+    (description
+     "Colorstring provides functions for colorizing strings for terminal output.")
+    (license license:expat)))
+
 (define-public go-git-sr-ht-emersion-go-scfg
   (package
     (name "go-git-sr-ht-emersion-go-scfg")
-- 
2.39.2





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

* [bug#62310] [PATCH v2 2/4] gnu: Add go-github-com-schollz-progressbar-v3.
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
  2023-03-21  8:15   ` [bug#62310] [PATCH v2 1/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
@ 2023-03-21  8:16   ` Sergey Trofimov
  2023-03-21  8:16   ` [bug#62310] [PATCH v2 3/4] gnu: fdroidcl: Update to 0.7.0 Sergey Trofimov
                     ` (2 subsequent siblings)
  4 siblings, 0 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-21  8:16 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/golang.scm
(go-github-com-schollz-progressbar-v3): New variable.
---
 gnu/packages/golang.scm | 41 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 3ac4863e85..8f0e202d71 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -11326,6 +11326,47 @@ (define-public go-github-com-mitchellh-colorstring
      "Colorstring provides functions for colorizing strings for terminal output.")
     (license license:expat)))
 
+(define-public go-github-com-schollz-progressbar-v3
+  (package
+    (name "go-github-com-schollz-progressbar-v3")
+    (version "3.13.1")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/schollz/progressbar")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1hjahr5r52i7w6iyvl3rpzr46iignhfdh4694fl7m2b4gkaw9gd6"))))
+    (build-system go-build-system)
+    (arguments
+     (list #:import-path "github.com/schollz/progressbar/v3"
+           #:phases
+           #~(modify-phases %standard-phases
+               (replace 'check
+                 (lambda* (#:key tests? import-path #:allow-other-keys)
+                   (when tests?
+                     ;; tests in the full suite require internet access
+                     (invoke "go" "test" "-test.short" import-path))
+                   #t)))))
+    (propagated-inputs
+     (list go-golang-org-x-term
+           go-github-com-stretchr-testify
+           go-github-com-mitchellh-colorstring
+           go-github-com-mattn-go-runewidth
+           go-github-com-mattn-go-isatty
+           go-github-com-davecgh-go-spew))
+    (home-page "https://github.com/schollz/progressbar")
+    (synopsis "Simple CLI progress bar")
+    (description
+     "This package provides a very simple thread-safe progress bar.  The
+@code{progressbar} implements an @code{io.Writer} so it can automatically detect the
+number of bytes written to a stream, so you can use it as a progressbar for an
+@code{io.Reader}.  When progressbar length is undetermined a customizable spinner is
+shown.")
+    (license license:expat)))
+
 (define-public go-git-sr-ht-emersion-go-scfg
   (package
     (name "go-git-sr-ht-emersion-go-scfg")
-- 
2.39.2





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

* [bug#62310] [PATCH v2 3/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
  2023-03-21  8:15   ` [bug#62310] [PATCH v2 1/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
  2023-03-21  8:16   ` [bug#62310] [PATCH v2 2/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
@ 2023-03-21  8:16   ` Sergey Trofimov
  2023-03-21  8:16   ` [bug#62310] [PATCH v2 4/4] guix: Put check-synopsis-style to the local checkers list Sergey Trofimov
  2023-03-21 18:09   ` bug#62310: [PATCH v2 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer
  4 siblings, 0 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-21  8:16 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* gnu/packages/android.scm (fdroidcl): Update to 0.7.0.
---
 gnu/packages/android.scm | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/gnu/packages/android.scm b/gnu/packages/android.scm
index 881b75d763..a960a73683 100644
--- a/gnu/packages/android.scm
+++ b/gnu/packages/android.scm
@@ -1195,25 +1195,22 @@ (define-public fdroidserver
 (define-public fdroidcl
   (package
     (name "fdroidcl")
-    (version "0.5.0")
+    (version "0.7.0")
     (source (origin
               (method git-fetch)
               (uri (git-reference
-                     (url "https://github.com/mvdan/fdroidcl")
-                     (commit (string-append "v" version))))
+                    (url "https://github.com/mvdan/fdroidcl")
+                    (commit (string-append "v" version))))
               (file-name (git-file-name name version))
               (sha256
-               (base32 "1rxcdyy2j34z0ql9d62w7ivsch9xihjnpb1z9kgy9q46vl8zhhy0"))))
+               (base32
+                "1s3fszlyyab9gbdrg52zcbafsb1mgb770skg7b3gj7f3pzfnra5n"))))
     (build-system go-build-system)
     (arguments
      `(#:import-path "mvdan.cc/fdroidcl"
-       #:tests? #f  ; TODO: Inputs missing.
+       #:tests? #f ;requires internet access
        #:install-source? #f))
-    (inputs
-     (list go-github-com-kr-pretty))
-    ;(native-inputs
-    ; `(("go-github-com-rogpeppe-go-internal-testscript"
-    ;    ,go-github-com-rogpeppe-go-internal-testscript)))
+    (inputs (list go-github-com-kr-pretty go-github-com-schollz-progressbar-v3))
     (synopsis "F-Droid desktop client")
     (description
      "While the Android client integrates with the system with regular update
-- 
2.39.2





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

* [bug#62310] [PATCH v2 4/4] guix: Put check-synopsis-style to the local checkers list.
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
                     ` (2 preceding siblings ...)
  2023-03-21  8:16   ` [bug#62310] [PATCH v2 3/4] gnu: fdroidcl: Update to 0.7.0 Sergey Trofimov
@ 2023-03-21  8:16   ` Sergey Trofimov
  2023-03-21 18:09   ` bug#62310: [PATCH v2 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer
  4 siblings, 0 replies; 16+ messages in thread
From: Sergey Trofimov @ 2023-03-21  8:16 UTC (permalink / raw)
  To: 62310; +Cc: Sergey Trofimov

* guix/lint.scm (check-synopsis-style): Move to %local-checkers.
---
 guix/lint.scm | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/guix/lint.scm b/guix/lint.scm
index a1a27873d9..3ed7fd6e4d 100644
--- a/guix/lint.scm
+++ b/guix/lint.scm
@@ -1862,6 +1862,10 @@ (define %local-checkers
      (name        'description)
      (description "Validate package descriptions")
      (check       check-description-style))
+   (lint-checker
+     (name        'synopsis)
+     (description "Validate package synopses")
+     (check       check-synopsis-style))
    (lint-checker
      (name        'inputs-should-be-native)
      (description "Identify inputs that should be native inputs")
@@ -1926,10 +1930,7 @@ (define %local-checkers
 
 (define %network-dependent-checkers
   (list
-   (lint-checker
-     (name        'synopsis)
-     (description "Validate package synopses")
-     (check       check-synopsis-style))
+
    (lint-checker
      (name        'gnu-description)
      (description "Validate synopsis & description of GNU packages")
-- 
2.39.2





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

* bug#62310: [PATCH v2 0/4] gnu: fdroidcl: Update to 0.7.0.
  2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
                     ` (3 preceding siblings ...)
  2023-03-21  8:16   ` [bug#62310] [PATCH v2 4/4] guix: Put check-synopsis-style to the local checkers list Sergey Trofimov
@ 2023-03-21 18:09   ` Maxim Cournoyer
  4 siblings, 0 replies; 16+ messages in thread
From: Maxim Cournoyer @ 2023-03-21 18:09 UTC (permalink / raw)
  To: Sergey Trofimov; +Cc: 62310-done

Hi,

Sergey Trofimov <sarg@sarg.org.ru> writes:

> I have addressed Maxim's comments and made the following changes:
> * enabled tests.short for progressbar-v3 (full suite requires internet access)
> * removed optional go-ansi dependency
> * reworded synopsysises/descriptions, fixed formatting
> * moved check-synopsis-style to %local-checkers, as I usually `lint -n` because of a very slow CVE lin

Thanks for the lightning fast response!

I've retouched progressbar-v3 cosmetics a tiny bit:

--8<---------------cut here---------------start------------->8---
@@ -11358,13 +11358,13 @@ (define-public go-github-com-schollz-progressbar-v3
            go-github-com-mattn-go-isatty
            go-github-com-davecgh-go-spew))
     (home-page "https://github.com/schollz/progressbar")
-    (synopsis "Simple CLI progress bar")
+    (synopsis "Simple command-line interface (CLI) progress bar")
     (description
      "This package provides a very simple thread-safe progress bar.  The
-@code{progressbar} implements an @code{io.Writer} so it can automatically detect the
-number of bytes written to a stream, so you can use it as a progressbar for an
-@code{io.Reader}.  When progressbar length is undetermined a customizable spinner is
-shown.")
+@code{progressbar} implements an @code{io.Writer} so it can automatically
+detect the number of bytes written to a stream, so you can use it as a
+@code{progressbar} for an @code{io.Reader}.  When @code{progressbar}'s length
+is undetermined, a customizable spinner is shown.")
     (license license:expat)))
--8<---------------cut here---------------end--------------->8---

And installed the series.

Thank you!

-- 
Maxim




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

end of thread, other threads:[~2023-03-21 18:10 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-20 22:47 [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Sergey Trofimov
2023-03-20 22:51 ` [bug#62310] [PATCH 1/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
2023-03-20 22:51   ` [bug#62310] [PATCH 2/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
2023-03-21  1:47     ` [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer
2023-03-20 22:51   ` [bug#62310] [PATCH 3/4] gnu: Add go-github-com-k0kubun-go-ansi Sergey Trofimov
2023-03-21  1:55     ` [bug#62310] [PATCH 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer
2023-03-20 22:51   ` [bug#62310] [PATCH] " Sergey Trofimov
2023-03-21  1:58     ` [bug#62310] [PATCH 0/4] " Maxim Cournoyer
2023-03-21  1:43   ` Maxim Cournoyer
2023-03-21  1:46   ` Maxim Cournoyer
2023-03-21  8:04 ` [bug#62310] [PATCH v2 " Sergey Trofimov
2023-03-21  8:15   ` [bug#62310] [PATCH v2 1/4] gnu: Add go-github-com-mitchellh-colorstring Sergey Trofimov
2023-03-21  8:16   ` [bug#62310] [PATCH v2 2/4] gnu: Add go-github-com-schollz-progressbar-v3 Sergey Trofimov
2023-03-21  8:16   ` [bug#62310] [PATCH v2 3/4] gnu: fdroidcl: Update to 0.7.0 Sergey Trofimov
2023-03-21  8:16   ` [bug#62310] [PATCH v2 4/4] guix: Put check-synopsis-style to the local checkers list Sergey Trofimov
2023-03-21 18:09   ` bug#62310: [PATCH v2 0/4] gnu: fdroidcl: Update to 0.7.0 Maxim Cournoyer

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

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 read-only IMAP folder(s) and NNTP newsgroup(s).