GNU bug report logs - #57326
[PATCH 0/7] gnu: Add hydroxide.

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 57326 in the body.
You can then email your comments to 57326 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: guix-patches <at> gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 0/7] gnu: Add hydroxide.
Date: Sun, 21 Aug 2022 19:33:14 +0000

I packaged Hydroxide to let you know that I packaged Hydroxide. 🙃

https://github.com/emersion/hydroxide

Here's the list of packages in this patchset:

 - go-github-com-emersion-go-imap (required version update)
 - go-github-com-boltdb-bolt
 - go-github-com-emersion-go-bcrypt
 - go-github-com-emersion-go-mbox
 - go-github-com-emersion-go-vcard
 - go-github-com-emersion-go-webdav
 - hydroxide

None of of the dependencies rely on each other, so commit order is irrelevant as long as Hydroxide is last. Several of these patches rely on the Aerc patchset however:

https://issues.guix.gnu.org/55903

`go-github-com-protonmail-go-crypto`, `go-github-com-emersion-go-message`, `go-github-com-emersion-go-smtp` and their dependencies are already packaged for use with Aerc, so this patchset shouldn't be accepted before Aerc is accepted.

Also, some of Hydroxide's dependencies were already included in Guix.

`go-github-com-emersion-go-imap` was already packaged, but I needed to bump the version so the `go-imap-move`[1] and `go-imap-specialuse`[2] extensions were merged. I sent PR #224[3] to avoid having to package those extensions separately.

`go-golang-org-x-crypto`, `go-golang-org-x-sys` and `go-golang-org-x-term` were also already packaged, and only the latter is directly required by Hydroxide.

Finally, about revision numbers, I'm using the latest Hydroxide commit in order to access my PR and to avoid requiring `go-github-com-mattn-go-isatty` (removed in commit 482bf89). I've also used the latest version for `go-github-com-emersion-go-webdav`, since most recent tagged version is behind enough to cause the Hydroxide build to fail.

[1]: https://github.com/emersion/go-imap-move
[2]: https://github.com/emersion/go-imap-specialuse
[3]: https://github.com/emersion/hydroxide/pull/224

Cairn (7):
  gnu: go-github-com-emersion-go-imap: Update to 1.2.1.
  gnu: Add go-github-com-boltdb-bolt.
  gnu: Add go-github-com-emersion-go-bcrypt.
  gnu: Add go-github-com-emersion-go-mbox.
  gnu: Add go-github-com-emersion-go-vcard.
  gnu: Add go-github-com-emersion-go-webdav.
  gnu: Add hydroxide.

 gnu/packages/golang.scm | 125 +++++++++++++++++++++++++++++++++++++++-
 gnu/packages/mail.scm   |  36 ++++++++++++
 2 files changed, 159 insertions(+), 2 deletions(-)

-- 
2.37.2

Message #8 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 1/7] gnu: go-github-com-emersion-go-imap: Update to 1.2.1.
Date: Sun, 21 Aug 2022 23:42:23 +0000

* gnu/packages/golang.scm (go-github-com-emersion-go-imap): Update to 1.2.1.
---
 gnu/packages/golang.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index d2f3dc39df..130e3d12f9 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -6150,7 +6150,7 @@ (define-public go-github-com-ddevault-go-libvterm
 (define-public go-github-com-emersion-go-imap
   (package
     (name "go-github-com-emersion-go-imap")
-    (version "1.0.0")
+    (version "1.2.1")
     (source
       (origin
         (method git-fetch)
@@ -6159,7 +6159,7 @@ (define-public go-github-com-emersion-go-imap
               (commit (string-append "v" version))))
         (sha256
          (base32
-          "1id8j2d0rn9sj8y62xhyygqpk5ygrcl9jlfx92sm1jsvxsm3kywq"))
+          "0ak2ysvfcc9w0g1070msis8x9sh6gzvf0nd65ks594siwbmqddw8"))
         (file-name (git-file-name name version))))
     (build-system go-build-system)
     (arguments
-- 
2.37.2

Message #11 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 2/7] gnu: Add go-github-com-boltdb-bolt.
Date: Sun, 21 Aug 2022 23:42:58 +0000

* gnu/packages/golang.scm (go-github-com-boltdb-bolt): New variable.
---
 gnu/packages/golang.scm | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 130e3d12f9..0f52edbf52 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -8101,6 +8101,31 @@ (define-public go-github-com-zclconf-go-cty
 configuration languages, but other uses may be possible too.")
     (license license:expat)))
 
+(define-public go-github-com-boltdb-bolt
+  (package
+    (name "go-github-com-boltdb-bolt")
+    (version "1.3.1")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/boltdb/bolt")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "0z7j06lijfi4y30ggf2znak2zf2srv2m6c68ar712wd2ys44qb3r"))))
+    (build-system go-build-system)
+    (arguments
+     `(#:tests? #f ;Undefined test flag causes fail
+       #:import-path "github.com/boltdb/bolt"))
+    (home-page "https://github.com/boltdb/bolt")
+    (synopsis "Embedded key/value database for Go")
+    (description "Bolt is a pure Go key/value store inspired by Howard Chu's
+LMDB project.  The goal of the project is to provide a simple, fast, and
+reliable  database for projects that don't require a full database server
+such as Postgres or MySQL.")
+    (license license:expat)))
+
 (define-public go-etcd-io-bbolt
   (package
     (name "go-etcd-io-bbolt")
-- 
2.37.2

Message #14 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 3/7] gnu: Add go-github-com-emersion-go-bcrypt.
Date: Sun, 21 Aug 2022 23:43:21 +0000

* gnu/packages/golang.scm (go-github-com-emersion-go-bcrypt): New variable.
---
 gnu/packages/golang.scm | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 0f52edbf52..60c014a3c1 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -9939,3 +9939,29 @@ (define-public go-github-com-sourcegraph-jsonrpc2
     (description
      "Package jsonrpc2 provides a Go implementation of JSON-RPC 2.0.")
     (license license:expat)))
+
+(define-public go-github-com-emersion-go-bcrypt
+  (let ((commit "6e724a1baa633595ee7629cc82f284f8a5fcb10a")
+         (revision "0"))
+    (package
+      (name "go-github-com-emersion-go-bcrypt")
+      (version (git-version "0.0.0" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/emersion/go-bcrypt")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "1pchrgs05w30iqbh4d6iys4wvlyajsdwchp5mkf59amgsbyjaqgm"))))
+      (build-system go-build-system)
+      (arguments
+        `(#:import-path "github.com/emersion/go-bcrypt"))
+      (inputs (list go-golang-org-x-crypto))
+      (home-page "https://github.com/emersion/go-bcrypt")
+      (synopsis "Go implementation of bcrypt adaptive hashing algorithm")
+      (description "Provos and Mazières's bcrypt algorithm for password
+hashing implemented in Go.  This modification adds the
+GenerateFromPasswordAndSalt function.")
+      (license license:bsd-3))))
-- 
2.37.2

Message #17 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 4/7] gnu: Add go-github-com-emersion-go-mbox.
Date: Sun, 21 Aug 2022 23:43:40 +0000

* gnu/packages/golang.scm (go-github-com-emersion-go-mbox): 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 60c014a3c1..9bad453402 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -9965,3 +9965,25 @@ (define-public go-github-com-emersion-go-bcrypt
 hashing implemented in Go.  This modification adds the
 GenerateFromPasswordAndSalt function.")
       (license license:bsd-3))))
+
+(define-public go-github-com-emersion-go-mbox
+  (package
+    (name "go-github-com-emersion-go-mbox")
+    (version "1.0.3")
+    (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/emersion/go-mbox")
+                    (commit (string-append "v" version))))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "0vnadh2khx7sxn0irrd8gz8ra02x7ij0q8zglq3rqffqil06nliv"))))
+    (build-system go-build-system)
+    (arguments
+     `(#:import-path "github.com/emersion/go-mbox"))
+    (home-page "https://github.com/emersion/go-mbox")
+    (synopsis "Go library for the mbox file format")
+    (description "This package parses the mbox file format into messages
+and formats messages into mbox files.")
+    (license license:expat)))
-- 
2.37.2

Message #20 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 5/7] gnu: Add go-github-com-emersion-go-vcard.
Date: Sun, 21 Aug 2022 23:43:59 +0000

* gnu/packages/golang.scm (go-github-com-emersion-go-vcard): New variable.
---
 gnu/packages/golang.scm | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 9bad453402..3501dfe412 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -9987,3 +9987,27 @@ (define-public go-github-com-emersion-go-mbox
     (description "This package parses the mbox file format into messages
 and formats messages into mbox files.")
     (license license:expat)))
+
+(define-public go-github-com-emersion-go-vcard
+  (let ((commit "d4056df0ec4a571d6d42a8a116f6ee8a8b40d569")
+         (revision "0"))
+    (package
+      (name "go-github-com-emersion-go-vcard")
+      (version (git-version "0.0.0" revision commit))
+      (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/emersion/go-vcard")
+                    (commit commit)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "04yjzg0fgc6ldf603mkh99k1p1fp30ni0j1141abi1jv04sdk919"))))
+      (build-system go-build-system)
+      (arguments
+        `(#:import-path "github.com/emersion/go-vcard"))
+      (inputs (list))
+      (home-page "https://github.com/emersion/go-vcard")
+      (synopsis "Go library for the vCard file format")
+      (description "This package lets you parse and format vCard in Go.")
+      (license license:expat))))
-- 
2.37.2

Message #23 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 6/7] gnu: Add go-github-com-emersion-go-webdav.
Date: Sun, 21 Aug 2022 23:44:13 +0000

* gnu/packages/golang.scm (go-github-com-emersion-go-webdav): New variable.
---
 gnu/packages/golang.scm | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
index 3501dfe412..c2a2c5f2dd 100644
--- a/gnu/packages/golang.scm
+++ b/gnu/packages/golang.scm
@@ -10011,3 +10011,27 @@ (define-public go-github-com-emersion-go-vcard
       (synopsis "Go library for the vCard file format")
       (description "This package lets you parse and format vCard in Go.")
       (license license:expat))))
+
+ (define-public go-github-com-emersion-go-webdav
+   (let ((commit "4a3cd0510f590534e8d1d69c70f736f6ca9762b3")
+         (revision "52"))
+   (package
+     (name "go-github-com-emersion-go-webdav")
+     (version (git-version "0.3.1" revision commit))
+     (source (origin
+              (method git-fetch)
+              (uri (git-reference
+                    (url "https://github.com/emersion/go-webdav")
+                    (commit commit)))
+              (file-name (git-file-name name version))
+              (sha256
+               (base32
+                "1alnsfs7scd3k9z6s974mnfvxc8c2vw95m9sii40l2702q64n4cl"))))
+     (build-system go-build-system)
+     (arguments
+      `(#:import-path "github.com/emersion/go-webdav"))
+     (home-page "https://github.com/emersion/go-webdav")
+     (synopsis "Go library for WebDAV, CalDAV and CardDAV")
+     (description "This package lets you use distributed authoring and
+versioning (DAV) standards in Go.")
+     (license license:expat))))
-- 
2.37.2

Message #26 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: [PATCH 7/7] gnu: Add hydroxide.
Date: Sun, 21 Aug 2022 23:44:31 +0000

* gnu/packages/mail.scm (hydroxide): New variable.
---
 gnu/packages/mail.scm | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)

diff --git a/gnu/packages/mail.scm b/gnu/packages/mail.scm
index 646902b4db..45023c0f7c 100644
--- a/gnu/packages/mail.scm
+++ b/gnu/packages/mail.scm
@@ -4675,3 +4675,39 @@ (define-public smtpmail
 mailserver on their machine.  It enables these users to send their mail over a
 remote SMTP server.")
     (license license:gpl2+)))
+
+(define-public hydroxide
+  (let ((commit "4c32801c34b75696003ef534fc718d86787f8048")
+         (revision "8"))
+    (package
+      (name "hydroxide")
+      (version (git-version "0.2.23" revision commit))
+      (source (origin
+                (method git-fetch)
+                (uri (git-reference
+                      (url "https://github.com/emersion/hydroxide")
+                      (commit commit)))
+                (file-name (git-file-name name version))
+                (sha256
+                 (base32
+                  "111hjm4ba7c9x897sh8vycqw3p61kh664i72rnw906pdyj1npjq6"))))
+      (build-system go-build-system)
+      (arguments
+        `(#:import-path "github.com/emersion/hydroxide/cmd/hydroxide"
+          #:unpack-path "github.com/emersion/hydroxide"))
+      (inputs (list go-github-com-protonmail-go-crypto
+                     go-github-com-boltdb-bolt
+                     go-github-com-emersion-go-bcrypt
+                     go-github-com-emersion-go-imap
+                     go-github-com-emersion-go-mbox
+                     go-github-com-emersion-go-message
+                     go-github-com-emersion-go-smtp
+                     go-github-com-emersion-go-vcard
+                     go-github-com-emersion-go-webdav
+                     go-golang-org-x-term))
+      (home-page "https://github.com/emersion/hydroxide")
+      (synopsis "Third-party Proton Mail bridge supporting CardDAV, IMAP and SMTP")
+      (description "This CLI tool is an alternative to the official
+Proton Mail Bridge.  It allows you to use your preferred e-mail
+clients and git-send-email with Proton Mail.")
+      (license license:expat))))
-- 
2.37.2

Message #29 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Olivier Dion <olivier.dion <at> polymtl.ca>
To: Cairn via Guix-patches via <guix-patches <at> gnu.org>, 57326 <at> debbugs.gnu.org
Cc: Cairn <cairn <at> pm.me>
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Sat, 27 Aug 2022 21:05:07 -0400

On Sun, 21 Aug 2022, Cairn via Guix-patches via <guix-patches <at> gnu.org> wrote:
> I packaged Hydroxide to let you know that I packaged Hydroxide. 🙃
>
> https://github.com/emersion/hydroxide

I was going to start working on this today.  Thanks for this
contribution!

-- 
Olivier Dion
oldiob.dev

Message #35 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: Olivier Dion <olivier.dion <at> polymtl.ca>
Cc: 57326 <at> debbugs.gnu.org, Cairn via Guix-patches via <guix-patches <at> gnu.org>
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Sun, 28 Aug 2022 01:13:57 +0000

[Message part 1 (text/plain, inline)]

> I was going to start working on this today. Thanks for this
> contribution!

Glad to know someone else finds this useful! =)

[signature.asc (application/pgp-signature, attachment)]

Message #41 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 1/7] gnu: go-github-com-emersion-go-imap:
 Update to 1.2.1.
Date: Sun, 28 Aug 2022 08:41:13 +0100

On Mon Aug 22, 2022 at 12:42 AM BST, Cairn via Guix-patches via wrote:
> * gnu/packages/golang.scm (go-github-com-emersion-go-imap): Update to 1.2.1.
> ---
>  gnu/packages/golang.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/golang.scm b/gnu/packages/golang.scm
> index d2f3dc39df..130e3d12f9 100644
> --- a/gnu/packages/golang.scm
> +++ b/gnu/packages/golang.scm
> @@ -6150,7 +6150,7 @@ (define-public go-github-com-ddevault-go-libvterm
>  (define-public go-github-com-emersion-go-imap
>    (package
>      (name "go-github-com-emersion-go-imap")
> -    (version "1.0.0")
> +    (version "1.2.1")
>      (source
>        (origin
>          (method git-fetch)
> @@ -6159,7 +6159,7 @@ (define-public go-github-com-emersion-go-imap
>                (commit (string-append "v" version))))
>          (sha256
>           (base32
> -          "1id8j2d0rn9sj8y62xhyygqpk5ygrcl9jlfx92sm1jsvxsm3kywq"))
> +          "0ak2ysvfcc9w0g1070msis8x9sh6gzvf0nd65ks594siwbmqddw8"))
>          (file-name (git-file-name name version))))
>      (build-system go-build-system)
>      (arguments
> -- 
> 2.37.2

LGTM

    -- (

Message #44 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 2/7] gnu: Add go-github-com-boltdb-bolt.
Date: Sun, 28 Aug 2022 08:44:22 +0100

On Mon Aug 22, 2022 at 12:42 AM BST, Cairn via Guix-patches via wrote:
> +    (arguments
> +     `(#:tests? #f ;Undefined test flag causes fail
> +       #:import-path "github.com/boltdb/bolt"))

fail -> failure. Also, might be better as:

  (arguments
   (list #:import-path "github.com/boltdb/bolt"
         ;; undefined test flag causes test failure
         #:tests? #f))

> +    (description "Bolt is a pure Go key/value store inspired by Howard Chu's
> +LMDB project.  The goal of the project is to provide a simple, fast, and
> +reliable  database for projects that don't require a full database server
> +such as Postgres or MySQL.")

And a slight formatting nit,

  (description
   "yadda yadda")

is preferred over

  (description "yadda yadda")

    -- (

Message #47 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 3/7] gnu: Add go-github-com-emersion-go-bcrypt.
Date: Sun, 28 Aug 2022 08:47:47 +0100

On Mon Aug 22, 2022 at 12:43 AM BST, Cairn via Guix-patches via wrote:
> +  (let ((commit "6e724a1baa633595ee7629cc82f284f8a5fcb10a")
> +         (revision "0"))
           ^
wonky indentation here

> +      (build-system go-build-system)
> +      (arguments
> +        `(#:import-path "github.com/emersion/go-bcrypt"))

  (arguments
   (list #:import-path "github.com/emersion/go-bcrypt"))

> +      (inputs (list go-golang-org-x-crypto))

inputs -> propagated-inputs (sadly necessary due to the way the
go-build-system works)

> +      (description "Provos and Mazières's bcrypt algorithm for password
> +hashing implemented in Go.  This modification adds the
> +GenerateFromPasswordAndSalt function.")

possibly better as:

  (description
   "This package provides an implementation of the @code{bcrypt} password
  hash algorithm.  This fork adds the @code{GenerateFromPasswordAndSalt}
  function."

    -- (

Message #50 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 4/7] gnu: Add go-github-com-emersion-go-mbox.
Date: Sun, 28 Aug 2022 08:51:17 +0100

On Mon Aug 22, 2022 at 12:43 AM BST, Cairn via Guix-patches via wrote:
> +    (arguments
> +     `(#:import-path "github.com/emersion/go-mbox"))

Convert this to list style.

> +    (description "This package parses the mbox file format into messages
> +and formats messages into mbox files.")

  (description
   "yadda yadda")

    -- (

Message #53 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 5/7] gnu: Add go-github-com-emersion-go-vcard.
Date: Sun, 28 Aug 2022 08:52:22 +0100

On Mon Aug 22, 2022 at 12:43 AM BST, Cairn via Guix-patches via wrote:
> +      (arguments
> +        `(#:import-path "github.com/emersion/go-vcard"))

Again, list style.

> +      (inputs (list))

No need for this field.

> +      (home-page "https://github.com/emersion/go-vcard")
> +      (synopsis "Go library for the vCard file format")
> +      (description "This package lets you parse and format vCard in Go.")

Description formatting here, too. Also, IMO "parse and format vCard" sounds
a little weird.

    -- (

Message #56 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 6/7] gnu: Add go-github-com-emersion-go-webdav.
Date: Sun, 28 Aug 2022 08:53:26 +0100

On Mon Aug 22, 2022 at 12:44 AM BST, Cairn via Guix-patches via wrote:
> + (define-public go-github-com-emersion-go-webdav
> +   (let ((commit "4a3cd0510f590534e8d1d69c70f736f6ca9762b3")
> +         (revision "52"))

Change to "0", since it's the first version of this package in Guix.

> +     (arguments
> +      `(#:import-path "github.com/emersion/go-webdav"))

Arguments -> list-style :)

> +     (description "This package lets you use distributed authoring and
> +versioning (DAV) standards in Go.")

Do description formatting here, too.

    -- (

Message #59 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 7/7] gnu: Add hydroxide.
Date: Sun, 28 Aug 2022 08:56:17 +0100

On Mon Aug 22, 2022 at 12:44 AM BST, Cairn via Guix-patches via wrote:
> +(define-public hydroxide
> +  (let ((commit "4c32801c34b75696003ef534fc718d86787f8048")
> +         (revision "8"))

Change to revision 0.

> +      (arguments
> +        `(#:import-path "github.com/emersion/hydroxide/cmd/hydroxide"
> +          #:unpack-path "github.com/emersion/hydroxide"))

Use list style.

> +      (inputs (list go-github-com-protonmail-go-crypto
> +                     go-github-com-boltdb-bolt
> +                     go-github-com-emersion-go-bcrypt
> +                     go-github-com-emersion-go-imap
> +                     go-github-com-emersion-go-mbox
> +                     go-github-com-emersion-go-message
> +                     go-github-com-emersion-go-smtp
> +                     go-github-com-emersion-go-vcard
> +                     go-github-com-emersion-go-webdav
> +                     go-golang-org-x-term))

Convert to something like:

  (inputs
   (list go-github-com-protonmail-go-crypto
         ...))

> +      (description "This CLI tool is an alternative to the official
> +Proton Mail Bridge.  It allows you to use your preferred e-mail
> +clients and git-send-email with Proton Mail.")

  (description
   "This package provides an alternative client for the ProtonMail
  Bridge.  It allows you to use SMTP and IMAP email clients with a
  ProtonMail account.")

    -- (

Message #62 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>, <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] [PATCH 7/7] gnu: Add hydroxide.
Date: Sun, 28 Aug 2022 08:59:22 +0100

On Mon Aug 22, 2022 at 12:44 AM BST, Cairn via Guix-patches via wrote:
> +  (let ((commit "4c32801c34b75696003ef534fc718d86787f8048")

Please add a comment explaining why you're using the latest commit.

> +         (revision "8"))

           ^ Wonky indentation.

    -- (

Message #65 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: "(" <paren <at> disroot.org>
Cc: 57326 <at> debbugs.gnu.org
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Mon, 29 Aug 2022 00:23:28 +0000

[Message part 1 (text/plain, inline)]

Thanks for all the feedback (. I'll implement it all soon and send the set again. I have just a couple questions:

> inputs -> propagated-inputs (sadly necessary due to the way the
> go-build-system works)

Could you explain why this is necessary? It doesn't seem to require it to build or run, at least for me. And do you mean for every package, or just this particular case?

> Description formatting here, too. Also, IMO "parse and format vCard" sounds
> a little weird.

Sounds fine to me. Also, I'm just borrowing the description from the GitHub page. If you do think it should be changed though, what would be a good change?

[signature.asc (application/pgp-signature, attachment)]

Message #68 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Olivier Dion <olivier.dion <at> polymtl.ca>
To: Cairn <cairn <at> pm.me>, "(" <paren <at> disroot.org>
Cc: 57326 <at> debbugs.gnu.org
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Thu, 08 Sep 2022 10:18:52 -0400

On Mon, 29 Aug 2022, Cairn via Guix-patches via <guix-patches <at> gnu.org> wrote:
> Thanks for all the feedback (. I'll implement it all soon and send the
> set again. I have just a couple questions:

While thing are getting fixed.  Did you manage to configure a proton
mail account to hydroxide correctly?  If so, I would use your current
patch locally for setting up my mail account.

-- 
Olivier Dion
oldiob.dev

Message #71 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Cairn" <cairn <at> pm.me>
Cc: 57326 <at> debbugs.gnu.org
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Sat, 10 Sep 2022 08:51:53 +0100

Hi Cairn, sorry for the delay!

On Mon Aug 29, 2022 at 1:23 AM BST, Cairn wrote:
> Could you explain why this is necessary? It doesn't seem to require it to build or run, at least for me. And do you mean for every package, or just this particular case?

This is because Go libraries (though not programs) are stored as source code in the
output directory and later fetched and rebuilt when you build one of their dependents.

So, if go-foo requires go-bar requires go-baz, go-bar will be rebuilt whenever go-foo
is built. But if go-bar doesn't use propagated-inputs, the go-baz dependency of go-bar
will be absent from the build environment.

In this case, we're talking about go-github-com-emersion-go-bcrypt depending on the
go-golang-org-x-crypto package; since x-crypto is probably already propagated in the
hydroxide build environment, as it's quite a common package, you don't see any issues;
but were you to write a package that solely depended on go-bcrypt, it'd fail to build
because x-crypto wasn't in the build environment.

> Sounds fine to me. Also, I'm just borrowing the description from the GitHub page. If you do think it should be changed though, what would be a good change?

IMO, the use of `vCard' as the object of that phrase without any further explanation
of what `vCard' is sounds a wee bit strange, so maybe something like:

  parse and format the vCard file format

Assuming vCard is a file format.

    -- (

Message #74 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: "(" <paren <at> disroot.org>
Cc: 57326 <at> debbugs.gnu.org
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Sat, 10 Sep 2022 09:32:02 +0000

[Message part 1 (text/plain, inline)]

Thanks for the responses (. They both make sense. I'll start implementing these changes.

[signature.asc (application/pgp-signature, attachment)]

Message #77 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: Olivier Dion <olivier.dion <at> polymtl.ca>
Cc: 57326 <at> debbugs.gnu.org
Subject: Re: [bug#57326] [PATCH 0/7] gnu: Add hydroxide.
Date: Sat, 10 Sep 2022 09:39:00 +0000

[Message part 1 (text/plain, inline)]

> While thing are getting fixed. Did you manage to configure a proton
> mail account to hydroxide correctly? If so, I would use your current
> patch locally for setting up my mail account.
> 

> --
> Olivier Dion
> oldiob.dev

Yep! The SMTP server worked at least; I don't have a use case for IMAP. You can either try this or podiki's Peroxide (which you can ask about on the IRC I think). I'm gonna be working on V2 of Hydroxide in order to make it more compatible with Peroxide for whenever it gets sent as a patchset.

[signature.asc (application/pgp-signature, attachment)]

Message #80 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Cairn <cairn <at> pm.me>
To: "57326 <at> debbugs.gnu.org" <57326 <at> debbugs.gnu.org>
Subject: Delays
Date: Fri, 14 Oct 2022 23:04:59 +0000

[Message part 1 (text/plain, inline)]

I need to wait to send V2 until this issue is resolved:

https://github.com/emersion/hydroxide/issues/230

Otherwise the program's useless.

[signature.asc (application/pgp-signature, attachment)]

Message #83 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Tyler Wolf <tyler <at> twolf.io>
To: "57326 <at> debbugs.gnu.org" <57326 <at> debbugs.gnu.org>
Subject: (No Subject)
Date: Sun, 27 Nov 2022 16:31:14 +0000

[Message part 1 (text/plain, inline)]

This issue appears to be fixable with the following: https://github.com/emersion/hydroxide/issues/179#issuecomment-1236208357

Is there a way to create a build with this fix? (maybe a fork will have to be used) It should work.

[Message part 2 (text/html, inline)]

Message #86 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: "(" <paren <at> disroot.org>
To: "Tyler Wolf" <tyler <at> twolf.io>, "57326 <at> debbugs.gnu.org"
 <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] (No Subject)
Date: Sun, 27 Nov 2022 16:36:12 +0000

[Message part 1 (text/plain, inline)]

On Sun Nov 27, 2022 at 4:31 PM GMT, Tyler Wolf via Guix-patches via wrote:
> Is there a way to create a build with this fix? (maybe a fork will have to be used) It should work.

You'd use the ``patches'' field of ``origin'' and the ``search-patches'' procedure,
after turning that fix into a Git patch.

    -- (

[signature.asc (application/pgp-signature, inline)]

Message #89 received at 57326 <at> debbugs.gnu.org (full text, mbox):

From: Olivier Dion <olivier.dion <at> polymtl.ca>
To: "(" <paren <at> disroot.org>, Tyler Wolf <tyler <at> twolf.io>,
 "57326 <at> debbugs.gnu.org" <57326 <at> debbugs.gnu.org>
Subject: Re: [bug#57326] (No Subject)
Date: Fri, 06 Jan 2023 14:49:22 -0500

Hi,

The issue got a fix and has been merged
<https://github.com/emersion/hydroxide/pull/232>.

-- 
Olivier Dion
oldiob.dev

Message #94 received at 57326-done <at> debbugs.gnu.org (full text, mbox):

From: Sharlatan Hellseher <sharlatanus <at> gmail.com>
To: 57326-done <at> debbugs.gnu.org
Subject: [PATCH 0/7] gnu: Add hydroxide.
Date: Sun, 28 Jul 2024 23:02:19 +0100

[Message part 1 (text/plain, inline)]

Hi,

Hydroxide <at> v0.2.29 was pushed to go-team as part of work from
<https://issues.guix.gnu.org/71780>.

Closing this issues as has been succeeded by #71780.

--
Oleg

[signature.asc (application/pgp-signature, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.