unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#73034] [PATCH] gnu: git: Fix perl search-paths of wrapped programs.
@ 2024-09-05  1:55 Simon Tournier
  2024-09-05  2:37 ` [bug#73034] [PATCH v2] " Simon Tournier
                   ` (2 more replies)
  0 siblings, 3 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-05  1:55 UTC (permalink / raw)
  To: 73034; +Cc: Simon Tournier

Fixes a regression introduced in f288604428e9c8f096ec9e0eccf207e079806271,
where the git-send-email and gitweb.cgi wrap-programs had translated into the
"new style" using G-exps.  See <https://issues.guix.gnu.org/73030>.

* gnu/packages/version-control.scm (git)[arguments]<phases>: Restore the
complete list of propagated inputs by modifying the functional composition
order.

Change-Id: Idfa33d9e5519aa9303b10138a166391244636ede
---
 gnu/packages/version-control.scm | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)

Hi,

Since there is no team, I guess, let CC people who submitted and reviewed the
initial patch. ;-)

Well, is this patch allowed to go to directly to master?  Because it triggers
888 rebuilds:

    $ ./pre-inst-env guix refresh -l git | cut -f1 -d':'
    Building the following 411 packages would ensure 888 dependent packages are rebuilt

Cheers,
simon

PS: Sent using this fix. ;-)  But an annoyance remains about Need MIME::Base64
and Authen::SASL; it requires perl-authen-sasl.


diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 1db783b731..d43382368b 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -55,6 +55,7 @@
 ;;; Copyright © 2024 Hilton Chain <hako@ultrarare.space>
 ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
 ;;; Copyright © 2024 Suhail Singh <suhail@bayesians.ca>
+;;; Copyright © 2024 Simon Tournier <zimon.toutoune@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -657,21 +658,24 @@ (define-public git
                       ,(search-path-as-list
                         '("lib/perl5/site_perl")
                         '#$(delete-duplicates
-                            (append-map
-                             (compose last package-transitive-propagated-inputs)
-                             (list (this-package-input "perl-authen-sasl")
-                                   (this-package-input "perl-net-smtp-ssl")
-                                   (this-package-input
-                                    "perl-io-socket-ssl")))))))
+                            (map last
+                                 (append-map
+                                  package-transitive-propagated-inputs
+                                  (list (this-package-input "perl-authen-sasl")
+                                        (this-package-input "perl-net-smtp-ssl")
+                                        (this-package-input
+                                         "perl-io-socket-ssl"))))))))
+
                   ;; Tell 'gitweb.cgi' where perl modules are.
                   (wrap-program (string-append out "/share/gitweb/gitweb.cgi")
                     `("PERL5LIB" ":" prefix
                       ,(search-path-as-list
                         '("lib/perl5/site_perl")
                         '#$(delete-duplicates
-                            (append-map
-                             (compose last package-transitive-propagated-inputs)
-                             (list (this-package-input "perl-cgi")))))))
+                            (map last
+                                 (append-map
+                                  package-transitive-propagated-inputs
+                                  (list (this-package-input "perl-cgi"))))))))
 
                   ;; Tell 'git-submodule' where Perl is.
                   (wrap-program git-sm

base-commit: ed95ddeb1e58c314f2e22b4cd35986042f3e2f21
-- 
2.45.2





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

* [bug#73034] [PATCH v2] gnu: git: Fix perl search-paths of wrapped programs.
  2024-09-05  1:55 [bug#73034] [PATCH] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
@ 2024-09-05  2:37 ` Simon Tournier
  2024-09-05 12:47 ` [bug#73034] [PATCH] " Ashish SHUKLA via Guix-patches via
  2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
  2 siblings, 0 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-05  2:37 UTC (permalink / raw)
  To: 73034; +Cc: Simon Tournier, maxim.cournoyer, liliana.prikler, ludo

Fixes a regression introduced in f288604428e9c8f096ec9e0eccf207e079806271,
where the git-send-email and gitweb.cgi wrap-programs had translated into the
"new style" using G-exps.  See <https://issues.guix.gnu.org/73030>.

* gnu/packages/version-control.scm (git)[arguments]<phases>: Restore the
complete list of propagated inputs by modifying the functional composition
order.

Change-Id: Ia4e5feeae6418a9f098464556b74bc871e761be1
---
 gnu/packages/version-control.scm | 29 ++++++++++++++++++++---------
 1 file changed, 20 insertions(+), 9 deletions(-)

Hi,

Change since v1:

'package-transitive-propagated-inputs' does not return the package itself, so
let manually add it.

Cheers,
simon


diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 1db783b731..b8192035c2 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -55,6 +55,7 @@
 ;;; Copyright © 2024 Hilton Chain <hako@ultrarare.space>
 ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
 ;;; Copyright © 2024 Suhail Singh <suhail@bayesians.ca>
+;;; Copyright © 2024 Simon Tournier <zimon.toutoune@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -657,21 +658,31 @@ (define-public git
                       ,(search-path-as-list
                         '("lib/perl5/site_perl")
                         '#$(delete-duplicates
-                            (append-map
-                             (compose last package-transitive-propagated-inputs)
-                             (list (this-package-input "perl-authen-sasl")
-                                   (this-package-input "perl-net-smtp-ssl")
-                                   (this-package-input
-                                    "perl-io-socket-ssl")))))))
+                            (let ((these-inputs
+                                   (list (this-package-input "perl-authen-sasl")
+                                         (this-package-input "perl-net-smtp-ssl")
+                                         (this-package-input "perl-io-socket-ssl"))))
+                              (append
+                               these-inputs
+                               (map last
+                                    (append-map
+                                     package-transitive-propagated-inputs
+                                     these-inputs))))))))
+
                   ;; Tell 'gitweb.cgi' where perl modules are.
                   (wrap-program (string-append out "/share/gitweb/gitweb.cgi")
                     `("PERL5LIB" ":" prefix
                       ,(search-path-as-list
                         '("lib/perl5/site_perl")
                         '#$(delete-duplicates
-                            (append-map
-                             (compose last package-transitive-propagated-inputs)
-                             (list (this-package-input "perl-cgi")))))))
+                            (let ((these-inputs
+                                   (list (this-package-input "perl-cgi"))))
+                              (append
+                               these-inputs
+                               (map last
+                                    (append-map
+                                     package-transitive-propagated-inputs
+                                     these-inputs))))))))
 
                   ;; Tell 'git-submodule' where Perl is.
                   (wrap-program git-sm

base-commit: ed95ddeb1e58c314f2e22b4cd35986042f3e2f21
-- 
2.45.2





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

* [bug#73034] [PATCH] gnu: git: Fix perl search-paths of wrapped programs.
  2024-09-05  1:55 [bug#73034] [PATCH] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
  2024-09-05  2:37 ` [bug#73034] [PATCH v2] " Simon Tournier
@ 2024-09-05 12:47 ` Ashish SHUKLA via Guix-patches via
  2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
  2 siblings, 0 replies; 12+ messages in thread
From: Ashish SHUKLA via Guix-patches via @ 2024-09-05 12:47 UTC (permalink / raw)
  To: 73034; +Cc: Simon Tournier

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

Hi,

If we're going to apply this patch which would trigger so many rebuilds,
I suggest combining it with git 2.46.0 update, I sent in issue #72413[0]

Also, this patch fixed, git-send-email for me.

[0] https://issues.guix.gnu.org/72413

Thanks!
--
Ashish

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

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

* [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0
  2024-09-05  1:55 [bug#73034] [PATCH] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
  2024-09-05  2:37 ` [bug#73034] [PATCH v2] " Simon Tournier
  2024-09-05 12:47 ` [bug#73034] [PATCH] " Ashish SHUKLA via Guix-patches via
@ 2024-09-05 15:34 ` Simon Tournier
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 1/3] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
                     ` (3 more replies)
  2 siblings, 4 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-05 15:34 UTC (permalink / raw)
  To: 73034; +Cc: Simon Tournier, maxim.cournoyer, liliana.prikler, ludo

Hi,

Since changes to the package Git imply large rebuilds, this series integrates
submission #72413.

Change since v2: Restore the ’origin’ of git-manpages under native-inputs.

Commit f288604428e9c8f096ec9e0eccf207e079806271 moved the origin inside the phase.

--8<---------------cut here---------------start------------->8---
+          (add-after 'split 'install-man-pages
+            (lambda _
+              (let ((man (string-append #$output "/share/man")))
+                (mkdir-p man)
+                (with-directory-excursion man
+                  (invoke
+                   "tar" "xvf"
+                   #$(origin
+                       (method url-fetch)
+                       (uri (string-append
+                             "mirror://kernel.org/software/scm/git/"
+                             "git-manpages-" version ".tar.xz"))
+                       (sha256
+                        (base32
+                         "0xsqakgy0s60zpa13ilj6zj420kdh8pf4v3nrp1nziwj8ja4qymw"))))))))
--8<---------------cut here---------------end--------------->8---

And this appears to me a bad idea, because 1. it makes harder to know what are
the inputs and more importantly 2. it is hidden from procedure
’package-direct-sources’, which means it will not be archived.

Cheers,
simon


1:  https://issues.guix.gnu.org/72413

Ashish SHUKLA (1):
  gnu: git: Update to 2.46.0

Simon Tournier (2):
  gnu: git: Fix perl search-paths of wrapped programs.
  gnu: git: Move git-manpages origin from phases to native-inputs.

 gnu/packages/version-control.scm | 77 +++++++++++++++++++-------------
 1 file changed, 47 insertions(+), 30 deletions(-)


base-commit: 7d2ced8d6d9c38327592d312376d59a8c37fc160
-- 
2.45.2





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

* [bug#73034] [PATCH v3 1/3] gnu: git: Fix perl search-paths of wrapped programs.
  2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
@ 2024-09-05 15:34   ` Simon Tournier
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 2/3] gnu: git: Update to 2.46.0 Simon Tournier
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-05 15:34 UTC (permalink / raw)
  To: 73034; +Cc: Simon Tournier

Fixes a regression introduced in f288604428e9c8f096ec9e0eccf207e079806271,
where the git-send-email and gitweb.cgi wrap-programs had translated into the
"new style" using G-exps.  See <https://issues.guix.gnu.org/73030>.

* gnu/packages/version-control.scm (git)[arguments]<phases>: Restore the
complete list of propagated inputs by modifying the functional composition
order.

Change-Id: Ia4e5feeae6418a9f098464556b74bc871e761be1
---
 gnu/packages/version-control.scm | 29 ++++++++++++++++++++---------
 1 file changed, 20 insertions(+), 9 deletions(-)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 1db783b731..b8192035c2 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -55,6 +55,7 @@
 ;;; Copyright © 2024 Hilton Chain <hako@ultrarare.space>
 ;;; Copyright © 2023 Zheng Junjie <873216071@qq.com>
 ;;; Copyright © 2024 Suhail Singh <suhail@bayesians.ca>
+;;; Copyright © 2024 Simon Tournier <zimon.toutoune@gmail.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -657,21 +658,31 @@ (define-public git
                       ,(search-path-as-list
                         '("lib/perl5/site_perl")
                         '#$(delete-duplicates
-                            (append-map
-                             (compose last package-transitive-propagated-inputs)
-                             (list (this-package-input "perl-authen-sasl")
-                                   (this-package-input "perl-net-smtp-ssl")
-                                   (this-package-input
-                                    "perl-io-socket-ssl")))))))
+                            (let ((these-inputs
+                                   (list (this-package-input "perl-authen-sasl")
+                                         (this-package-input "perl-net-smtp-ssl")
+                                         (this-package-input "perl-io-socket-ssl"))))
+                              (append
+                               these-inputs
+                               (map last
+                                    (append-map
+                                     package-transitive-propagated-inputs
+                                     these-inputs))))))))
+
                   ;; Tell 'gitweb.cgi' where perl modules are.
                   (wrap-program (string-append out "/share/gitweb/gitweb.cgi")
                     `("PERL5LIB" ":" prefix
                       ,(search-path-as-list
                         '("lib/perl5/site_perl")
                         '#$(delete-duplicates
-                            (append-map
-                             (compose last package-transitive-propagated-inputs)
-                             (list (this-package-input "perl-cgi")))))))
+                            (let ((these-inputs
+                                   (list (this-package-input "perl-cgi"))))
+                              (append
+                               these-inputs
+                               (map last
+                                    (append-map
+                                     package-transitive-propagated-inputs
+                                     these-inputs))))))))
 
                   ;; Tell 'git-submodule' where Perl is.
                   (wrap-program git-sm
-- 
2.45.2





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

* [bug#73034] [PATCH v3 2/3] gnu: git: Update to 2.46.0
  2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 1/3] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
@ 2024-09-05 15:34   ` Simon Tournier
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 3/3] gnu: git: Move git-manpages origin from phases to native-inputs Simon Tournier
  2024-09-06  4:17   ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Maxim Cournoyer
  3 siblings, 0 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-05 15:34 UTC (permalink / raw)
  To: 73034; +Cc: Ashish SHUKLA

From: Ashish SHUKLA <ashish.is@lostca.se>

* gnu/packages/version-control.scm (git-minimal): Update to 2.46.0.
  (git)[arguments]<phases>{install-man-pages} Update hash for the
  git-manpages.

Change-Id: I686825ffba828e88e37ff156fab23fd43155d210
---
 gnu/packages/version-control.scm | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index b8192035c2..9a4c2fdbdc 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -249,14 +249,14 @@ (define git-cross-configure-flags
 (define-public git-minimal
   (package
     (name "git-minimal")
-    (version "2.45.2")
+    (version "2.46.0")
     (source (origin
              (method url-fetch)
              (uri (string-append "mirror://kernel.org/software/scm/git/git-"
                                  version ".tar.xz"))
              (sha256
               (base32
-               "1nws1vjgj54sv32wxl1h3n1jkcpabqv7a605hhafsby0n5zfigsi"))))
+               "15bzq9m6c033qiz5q5gw1nqw4m452vvqax30wbms6z4bl9i384kz"))))
     (build-system gnu-build-system)
     (arguments
      (list
@@ -703,7 +703,7 @@ (define-public git
                                ".tar.xz"))
                          (sha256
                           (base32
-                           "1pqrp46kwbxycqld39027ph1cvkq9am156y3sswn6w2khsg30f09"))))))))))))
+                           "1lvvhzypllbyd8j6m0p9qgd3gqg10gch9s7lqif8vr9n80fqn4fw"))))))))))))
     (native-inputs
      (modify-inputs (package-native-inputs git-minimal)
        ;; For subtree documentation.
-- 
2.45.2





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

* [bug#73034] [PATCH v3 3/3] gnu: git: Move git-manpages origin from phases to native-inputs.
  2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 1/3] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 2/3] gnu: git: Update to 2.46.0 Simon Tournier
@ 2024-09-05 15:34   ` Simon Tournier
  2024-09-06  4:17   ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Maxim Cournoyer
  3 siblings, 0 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-05 15:34 UTC (permalink / raw)
  To: 73034; +Cc: Simon Tournier

The move from native-inputs to the install-man-pages phase had been done by
f288604428e9c8f096ec9e0eccf207e079806271.

* gnu/packages/version-control.scm (git)[arguments]<phases>: Move git-manpages
origin from here...
[native-inputs]: ...to here.

Change-Id: I9fa543f7410bce6eb8a82d275eab81a1ea86ecc0
---
 gnu/packages/version-control.scm | 44 ++++++++++++++++++--------------
 1 file changed, 25 insertions(+), 19 deletions(-)

diff --git a/gnu/packages/version-control.scm b/gnu/packages/version-control.scm
index 9a4c2fdbdc..3e8f042d18 100644
--- a/gnu/packages/version-control.scm
+++ b/gnu/packages/version-control.scm
@@ -695,26 +695,32 @@ (define-public git
                   (with-directory-excursion man
                     (invoke
                      "tar" "xvf"
-                     #$(origin
-                         (method url-fetch)
-                         (uri (string-append
-                               "mirror://kernel.org/software/scm/git/"
-                               "git-manpages-" (package-version this-package)
-                               ".tar.xz"))
-                         (sha256
-                          (base32
-                           "1lvvhzypllbyd8j6m0p9qgd3gqg10gch9s7lqif8vr9n80fqn4fw"))))))))))))
+                     #$(this-package-native-input "git-manpages"))))))))))
     (native-inputs
-     (modify-inputs (package-native-inputs git-minimal)
-       ;; For subtree documentation.
-       (append asciidoc
-               docbook2x
-               docbook-xml-4.5
-               docbook-xsl
-               libxslt
-               pkg-config
-               texinfo
-               xmlto)))
+     (cons
+      ;; To build the man pages from the git sources, we would need a dependency
+      ;; on a full XML tool chain, and building it actually takes ages.  So we
+      ;; use this lazy approach and use released tarball.
+      `("git-manpages"
+       ,(origin
+           (method url-fetch)
+           (uri (string-append
+                 "mirror://kernel.org/software/scm/git/git-manpages-"
+                 (package-version this-package) ".tar.xz"))
+           (sha256
+            (base32
+             "1lvvhzypllbyd8j6m0p9qgd3gqg10gch9s7lqif8vr9n80fqn4fw"))))
+
+      (modify-inputs (package-native-inputs git-minimal)
+        ;; For subtree documentation.
+        (append asciidoc
+                docbook2x
+                docbook-xml-4.5
+                docbook-xsl
+                libxslt
+                pkg-config
+                texinfo
+                xmlto))))
     (inputs
      (modify-inputs (package-inputs git-minimal)
        (append bash-minimal             ;for wrap-program
-- 
2.45.2





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

* [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0
  2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
                     ` (2 preceding siblings ...)
  2024-09-05 15:34   ` [bug#73034] [PATCH v3 3/3] gnu: git: Move git-manpages origin from phases to native-inputs Simon Tournier
@ 2024-09-06  4:17   ` Maxim Cournoyer
  2024-09-06 10:31     ` Simon Tournier
  3 siblings, 1 reply; 12+ messages in thread
From: Maxim Cournoyer @ 2024-09-06  4:17 UTC (permalink / raw)
  To: Simon Tournier; +Cc: ludo, 73034, liliana.prikler

Hi,

Simon Tournier <zimon.toutoune@gmail.com> writes:

> Hi,
>
> Since changes to the package Git imply large rebuilds, this series integrates
> submission #72413.
>
> Change since v2: Restore the ’origin’ of git-manpages under native-inputs.
>
> Commit f288604428e9c8f096ec9e0eccf207e079806271 moved the origin inside the phase.
>
> +          (add-after 'split 'install-man-pages
> +            (lambda _
> +              (let ((man (string-append #$output "/share/man")))
> +                (mkdir-p man)
> +                (with-directory-excursion man
> +                  (invoke
> +                   "tar" "xvf"
> +                   #$(origin
> +                       (method url-fetch)
> +                       (uri (string-append
> +                             "mirror://kernel.org/software/scm/git/"
> +                             "git-manpages-" version ".tar.xz"))
> +                       (sha256
> +                        (base32
> +                         "0xsqakgy0s60zpa13ilj6zj420kdh8pf4v3nrp1nziwj8ja4qymw"))))))))
>
> And this appears to me a bad idea, because 1. it makes harder to know what are
> the inputs and more importantly 2. it is hidden from procedure
> ’package-direct-sources’, which means it will not be archived.

I believe your argument 1. is going to affect any label-free package
definitions needing additional origins copied in, so the issue is bigger
than just this commit, in my opinion.

About 2; perhaps it'd be preferable to build the doc from source, if
that doesn't introduce cycles or too large of a native inputs graph.

-- 
Thanks,
Maxim




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

* [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0
  2024-09-06  4:17   ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Maxim Cournoyer
@ 2024-09-06 10:31     ` Simon Tournier
  2024-09-06 15:53       ` Simon Tournier
  0 siblings, 1 reply; 12+ messages in thread
From: Simon Tournier @ 2024-09-06 10:31 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: ludo, 73034, liliana.prikler

Hi Maxim,

On Fri, 06 Sep 2024 at 13:17, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:

>> And this appears to me a bad idea, because 1. it makes harder to know what are
>> the inputs and more importantly 2. it is hidden from procedure
>> ’package-direct-sources’, which means it will not be archived.
>
> I believe your argument 1. is going to affect any label-free package
> definitions needing additional origins copied in, so the issue is bigger
> than just this commit, in my opinion.

Well, is it not changing the scope of the work being reviewed?

If not, sorry, I do not understand what you mean.  Could you explain more?

In my views, what comes from the outside should be listed under inputs,
native-inputs or propagated-inputs.  I mean, that’s somehow the
principle from functional paradigm.  Putting an ’origin’ inside an
arguments is somehow a way to get around that principle, IMHO.

For instance, packages farstream, gnulib-checkout, smithforth,
gnome-recipes and dmd-bootstrap should also be fixed.

If you mean that it’s not easy to fix, from my understanding, it changes
the scope of the work being reviewed but let take the opportunity to
discuss. :-)

Currently it’s not possible to write something like:

--8<---------------cut here---------------start------------->8---
(native-inputs (append
                    (list
                     `("foo"
                       ,(origin
                          (method git-fetch)
                          (uri (git-reference
                                (url "https://somewhere.org/plop")
                                (commit (string-append "v" version))))
                          (file-name (git-file-name "plip" version))
                          (sha256
                           (base32
                            "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx")))))
                    (list bar baz and other)))
--8<---------------cut here---------------end--------------->8---

It appears to me that something is lacking: inputs-append. ;-)  We could
have a macro or a procedure that does the dance, allowing to mix both
“styles”.


> About 2; perhaps it'd be preferable to build the doc from source, if
> that doesn't introduce cycles or too large of a native inputs graph.

This is out of the scope, IMHO.  Yes, I agree: it might be preferable
but while waiting, it appears to me even more preferable to not have a
package that hides all its sources. ;-)

Cheers,
simon




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

* [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0
  2024-09-06 10:31     ` Simon Tournier
@ 2024-09-06 15:53       ` Simon Tournier
  2024-09-08 12:10         ` bug#73034: " Maxim Cournoyer
  0 siblings, 1 reply; 12+ messages in thread
From: Simon Tournier @ 2024-09-06 15:53 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: ludo, 73034, liliana.prikler

Re,

On Fri, 06 Sep 2024 at 12:31, Simon Tournier <zimon.toutoune@gmail.com> wrote:

>> I believe your argument 1. is going to affect any label-free package
>> definitions needing additional origins copied in, so the issue is bigger
>> than just this commit, in my opinion.

[...]

>                                        from my understanding, it changes
> the scope of the work being reviewed but let take the opportunity to
> discuss. :-)

See submission #73073 <https://issues.guix.gnu.org/73073>.

Cheers,
simon




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

* bug#73034: [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0
  2024-09-06 15:53       ` Simon Tournier
@ 2024-09-08 12:10         ` Maxim Cournoyer
  2024-09-09 17:50           ` [bug#73034] " Simon Tournier
  0 siblings, 1 reply; 12+ messages in thread
From: Maxim Cournoyer @ 2024-09-08 12:10 UTC (permalink / raw)
  To: Simon Tournier; +Cc: ludo, 73034-done, liliana.prikler

Hi Simon,

Simon Tournier <zimon.toutoune@gmail.com> writes:

> Re,
>
> On Fri, 06 Sep 2024 at 12:31, Simon Tournier <zimon.toutoune@gmail.com> wrote:
>
>>> I believe your argument 1. is going to affect any label-free package
>>> definitions needing additional origins copied in, so the issue is bigger
>>> than just this commit, in my opinion.
>
> [...]
>
>>                                        from my understanding, it changes
>> the scope of the work being reviewed but let take the opportunity to
>> discuss. :-)
>
> See submission #73073 <https://issues.guix.gnu.org/73073>.

Thanks for following up with concrete ideas.  It seems the discussion
still has some ways to go before we reach consensus there, but at least
it seems clear we do not want to go back to reintroducing labels, so
I've applied 1/3 and 3/3 but left out 2/3.

Thank you for fixing my botched wrapped-program phase translation :-).
We can now use 'git send-email' and friends easily again.

-- 
Thanks,
Maxim




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

* [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0
  2024-09-08 12:10         ` bug#73034: " Maxim Cournoyer
@ 2024-09-09 17:50           ` Simon Tournier
  0 siblings, 0 replies; 12+ messages in thread
From: Simon Tournier @ 2024-09-09 17:50 UTC (permalink / raw)
  To: Maxim Cournoyer; +Cc: ludo, 73034-done, liliana.prikler

Hi Maxim,

On Sun, 08 Sep 2024 at 21:10, Maxim Cournoyer <maxim.cournoyer@gmail.com> wrote:

> We can now use 'git send-email' and friends easily again.

Cool!  Thank you.

Cheers,
simon




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

end of thread, other threads:[~2024-09-09 19:07 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-05  1:55 [bug#73034] [PATCH] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
2024-09-05  2:37 ` [bug#73034] [PATCH v2] " Simon Tournier
2024-09-05 12:47 ` [bug#73034] [PATCH] " Ashish SHUKLA via Guix-patches via
2024-09-05 15:34 ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Simon Tournier
2024-09-05 15:34   ` [bug#73034] [PATCH v3 1/3] gnu: git: Fix perl search-paths of wrapped programs Simon Tournier
2024-09-05 15:34   ` [bug#73034] [PATCH v3 2/3] gnu: git: Update to 2.46.0 Simon Tournier
2024-09-05 15:34   ` [bug#73034] [PATCH v3 3/3] gnu: git: Move git-manpages origin from phases to native-inputs Simon Tournier
2024-09-06  4:17   ` [bug#73034] [PATCH v3 0/3] Fix annoyances of Git and update to 2.46.0 Maxim Cournoyer
2024-09-06 10:31     ` Simon Tournier
2024-09-06 15:53       ` Simon Tournier
2024-09-08 12:10         ` bug#73034: " Maxim Cournoyer
2024-09-09 17:50           ` [bug#73034] " Simon Tournier

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