[bug#68891] [PATCH] gnu: emacs-org-board: Update to 1138-2.500fe02.

[bug#68891] [PATCH] gnu: emacs-org-board: Update to 1138-2.500fe02.

[Cayetano Santos via Guix-patches via]

* gnu/packages/emacs-xyz.scm (emacs-org-board): Update to 1138-2.500fe02.
---
 gnu/packages/emacs-xyz.scm | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index c7e6852194..3e6d09c509 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -27782,8 +27782,8 @@ (define-public emacs-cpreproc-openvdb
    (license license:gpl3+)))
 
 (define-public emacs-org-board
-  (let ((commit "1393bd46d11a81328ed4fb8471831415a3efe224")
-        (revision "1"))
+  (let ((commit "500fe02bc114e5b535a2eb2ab73954d79428168f")
+        (revision "2"))
     (package
       (name "emacs-org-board")
       (version (git-version "1138" revision commit))
@@ -27795,7 +27795,7 @@ (define-public emacs-org-board
                 (file-name (git-file-name name version))
                 (sha256
                  (base32
-                  "1kryrg988c3sbxyp1sdgc6xdv2iz6kiflpzn2rw4z3l4grzab53b"))))
+                  "1mcghw9mmlyqsgxk97hd57495sfx8dp6ig7qgs5xn2b2k0ygv4vf"))))
       (build-system emacs-build-system)
       (arguments
        (list
@@ -27805,11 +27805,9 @@ (define-public emacs-org-board
               (lambda* (#:key inputs #:allow-other-keys)
                 (emacs-substitute-variables "org-board.el"
                   ("org-board-wget-program"
-                   (search-input-file inputs "/bin/wget"))))))))
-      (inputs
-       (list wget))
+                   (search-input-file inputs "/bin/wget2"))))))))
       (propagated-inputs
-       (list emacs-org emacs-ztree))
+       (list emacs-ztree wget2))
       (home-page "https://github.com/charlesroelli/org-board")
       (synopsis "Bookmarking and web archival system for Org mode")
       (description

base-commit: c692808a5e1413480162be1406186a63bd92651c
-- 
2.41.0

[bug#68891] [PATCH] gnu: emacs-org-board: Update to 1138-2.500fe02.

[Christopher Baines]

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


Cayetano Santos via Guix-patches via <guix-patches@gnu.org> writes:

> * gnu/packages/emacs-xyz.scm (emacs-org-board): Update to 1138-2.500fe02.
> ---
>  gnu/packages/emacs-xyz.scm | 12 +++++-------
>  1 file changed, 5 insertions(+), 7 deletions(-)
>
> diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
> index c7e6852194..3e6d09c509 100644
> --- a/gnu/packages/emacs-xyz.scm
> +++ b/gnu/packages/emacs-xyz.scm
> @@ -27782,8 +27782,8 @@ (define-public emacs-cpreproc-openvdb
>     (license license:gpl3+)))
>  
>  (define-public emacs-org-board
> -  (let ((commit "1393bd46d11a81328ed4fb8471831415a3efe224")
> -        (revision "1"))
> +  (let ((commit "500fe02bc114e5b535a2eb2ab73954d79428168f")
> +        (revision "2"))
>      (package
>        (name "emacs-org-board")
>        (version (git-version "1138" revision commit))
> @@ -27795,7 +27795,7 @@ (define-public emacs-org-board
>                  (file-name (git-file-name name version))
>                  (sha256
>                   (base32
> -                  "1kryrg988c3sbxyp1sdgc6xdv2iz6kiflpzn2rw4z3l4grzab53b"))))
> +                  "1mcghw9mmlyqsgxk97hd57495sfx8dp6ig7qgs5xn2b2k0ygv4vf"))))
>        (build-system emacs-build-system)
>        (arguments
>         (list
> @@ -27805,11 +27805,9 @@ (define-public emacs-org-board
>                (lambda* (#:key inputs #:allow-other-keys)
>                  (emacs-substitute-variables "org-board.el"
>                    ("org-board-wget-program"
> -                   (search-input-file inputs "/bin/wget"))))))))
> -      (inputs
> -       (list wget))
> +                   (search-input-file inputs "/bin/wget2"))))))))
>        (propagated-inputs
> -       (list emacs-org emacs-ztree))
> +       (list emacs-ztree wget2))

Thanks for the patch, these changes around wget/wget2 look confusing to
me though.

The reason for the emacs-substitute-variables bit is to allow wget to
just be an input rather than a propagated input, but you're making wget2
a propagated input here. This change and the motivation behind it is
also missing from the commit message.

Thanks,

Chris

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 987 bytes --]

[bug#68891] [PATCH] gnu: emacs-org-board: Update to 1138-2.500fe02.

[Cayetano Santos via Guix-patches via]

>mar. 13 févr. 2024 at 11:21, Christopher Baines <mail@cbaines.net> wrote:

> Thanks for the patch, these changes around wget/wget2 look confusing to
> me though.
>
> The reason for the emacs-substitute-variables bit is to allow wget to
> just be an input rather than a propagated input, but you're making wget2
> a propagated input here. This change and the motivation behind it is
> also missing from the commit message.

When one installs emacs-org-board, wget2-2.1.0 gets installed along with
it as for emacs-substitute-variables, as you say, and the variable
org-board-wget-program is bound to this hardcoded value.

Once one decides to upgrade wget2 to something different (say, in a
different profile), this logic breaks. User needs to manually bind
org-board-wget-program to whatever wget2 is after upgrading. At this
point, wget2 becomes a native-input requirement.

Please, correct me if I’m wrong.

Cayetano

[bug#68891] [PATCH] gnu: emacs-org-board: Update to 1138-2.500fe02.

[Nicolas Goaziou via Guix-patches via]

Hello,

Cayetano Santos via Guix-patches via <guix-patches@gnu.org> writes:

> When one installs emacs-org-board, wget2-2.1.0 gets installed along with
> it as for emacs-substitute-variables, as you say, and the variable
> org-board-wget-program is bound to this hardcoded value.

Correct.

> Once one decides to upgrade wget2 to something different (say, in a
> different profile), this logic breaks. User needs to manually bind
> org-board-wget-program to whatever wget2 is after upgrading. At this
> point, wget2 becomes a native-input requirement.

This is not correct. No action is required from the user.
`org-board-wget-program' still points to the wget program defined as an
input to emacs-org-board, and will still be used at run time.

Of course, the user still can point `org-board-wget-program' to their
new and shiny wget binary, but this is not required.

I also suggest to keep wget (or wget2) as an input and use
`emacs-substitute-variables'.

Regards,
-- 
Nicolas Goaziou

[bug#68891] [PATCH v1] gnu: emacs-org-board: Update to 1138-2.500fe02.

[Cayetano Santos via Guix-patches via]

* gnu/packages/emacs-xyz.scm (emacs-org-board): Update to 1138-2.500fe02.

Change-Id: Ifb8abdb02170a2183c5a023604c1f0d427781759
---

Update to latest.

- emacs-org is part of emacs now, no need to pull latest

 gnu/packages/emacs-xyz.scm | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index b3517b7bd9..5efbd543ac 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -29399,8 +29399,8 @@ (define-public emacs-cpreproc-openvdb
    (license license:gpl3+)))
 
 (define-public emacs-org-board
-  (let ((commit "1393bd46d11a81328ed4fb8471831415a3efe224")
-        (revision "1"))
+  (let ((commit "500fe02bc114e5b535a2eb2ab73954d79428168f")
+        (revision "2"))
     (package
       (name "emacs-org-board")
       (version (git-version "1138" revision commit))
@@ -29412,7 +29412,7 @@ (define-public emacs-org-board
                 (file-name (git-file-name name version))
                 (sha256
                  (base32
-                  "1kryrg988c3sbxyp1sdgc6xdv2iz6kiflpzn2rw4z3l4grzab53b"))))
+                  "1mcghw9mmlyqsgxk97hd57495sfx8dp6ig7qgs5xn2b2k0ygv4vf"))))
       (build-system emacs-build-system)
       (arguments
        (list
@@ -29426,7 +29426,7 @@ (define-public emacs-org-board
       (inputs
        (list wget))
       (propagated-inputs
-       (list emacs-org emacs-ztree))
+       (list emacs-ztree))
       (home-page "https://github.com/charlesroelli/org-board")
       (synopsis "Bookmarking and web archival system for Org mode")
       (description

base-commit: 40625c213ad2007f70117518a7cc290d367e652a
--
2.46.0

[bug#68891] Control message for bug #68891

[Cayetano Santos via Guix-patches via]

user guix
usertag 68891 + patch-hackers-review-list
thanks

--
Cayetano Santos