* [bug#55107] [PATCH] gnu: zsh: Build and install info pages.
@ 2022-04-25 6:40 Andrew Tropin
2022-04-27 3:51 ` Thiago Jung Bauermann via Guix-patches via
[not found] ` <handler.55107.D55107.16512368901526.notifdone@debbugs.gnu.org>
0 siblings, 2 replies; 4+ messages in thread
From: Andrew Tropin @ 2022-04-25 6:40 UTC (permalink / raw)
To: 55107
[-- Attachment #1: Type: text/plain, Size: 1857 bytes --]
* gnu/packages/shells.scm (zsh)[arguments]: Build and install info pages.
---
gnu/packages/shells.scm | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/gnu/packages/shells.scm b/gnu/packages/shells.scm
index 3a48bc7dbf..7158a5bcfb 100644
--- a/gnu/packages/shells.scm
+++ b/gnu/packages/shells.scm
@@ -19,6 +19,7 @@
;;; Copyright © 2020, 2022 Marius Bakke <marius@gnu.org>
;;; Copyright © 2021, 2022 Nicolas Goaziou <mail@nicolasgoaziou.fr>
;;; Copyright © 2021, 2022 Felix Gruber <felgru@posteo.net>
+;;; Copyright © 2022 Andrew Tropin <andrew@trop.in>
;;;
;;; This file is part of GNU Guix.
;;;
@@ -64,6 +65,7 @@ (define-module (gnu packages shells)
#:use-module (gnu packages tls)
#:use-module (gnu packages version-control)
#:use-module (gnu packages xorg)
+ #:use-module (gnu packages texinfo)
#:use-module (guix build-system cargo)
#:use-module (guix build-system cmake)
#:use-module (guix build-system gnu)
@@ -522,9 +524,13 @@ (define-public zsh
(("command -pv") "command -v")
(("command -p") "command ")
(("'command' -p") "'command' "))
- #t)))))
+ #t))
+ (add-after 'build 'make-info
+ (lambda _ (invoke "make" "info")))
+ (add-after 'build 'install-info
+ (lambda _ (invoke "make" "install.info"))))))
(native-inputs (list autoconf))
- (inputs (list ncurses pcre perl))
+ (inputs (list ncurses pcre perl texinfo))
(synopsis "Powerful shell for interactive use and scripting")
(description "The Z shell (zsh) is a Unix shell that can be used
as an interactive login shell and as a powerful command interpreter
--
2.35.1
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 853 bytes --]
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [bug#55107] [PATCH] gnu: zsh: Build and install info pages.
2022-04-25 6:40 [bug#55107] [PATCH] gnu: zsh: Build and install info pages Andrew Tropin
@ 2022-04-27 3:51 ` Thiago Jung Bauermann via Guix-patches via
2022-04-29 12:54 ` bug#55107: " Ludovic Courtès
[not found] ` <handler.55107.D55107.16512368901526.notifdone@debbugs.gnu.org>
1 sibling, 1 reply; 4+ messages in thread
From: Thiago Jung Bauermann via Guix-patches via @ 2022-04-27 3:51 UTC (permalink / raw)
To: Andrew Tropin; +Cc: 55107
Hello Andrew,
This looks great. Just a couple of comments:
Andrew Tropin <andrew@trop.in> writes:
> [[PGP Signed Part:Undecided]]
>
> * gnu/packages/shells.scm (zsh)[arguments]: Build and install info pages.
The changelog should also mention the addition of texinfo to inputs (or
native inputs).
> @@ -522,9 +524,13 @@ (define-public zsh
> (("command -pv") "command -v")
> (("command -p") "command ")
> (("'command' -p") "'command' "))
> - #t)))))
> + #t))
Phases don't need to return #t anymore, so this patch would be a nice
opportunity to remove it here.
> + (add-after 'build 'make-info
> + (lambda _ (invoke "make" "info")))
> + (add-after 'build 'install-info
> + (lambda _ (invoke "make" "install.info"))))))
> (native-inputs (list autoconf))
> - (inputs (list ncurses pcre perl))
> + (inputs (list ncurses pcre perl texinfo))
texinfo is a tool executed during the build process itself, so it should
be listed in native-inputs to allow cross-compilation.
--
Thanks
Thiago
^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#55107: [PATCH] gnu: zsh: Build and install info pages.
2022-04-27 3:51 ` Thiago Jung Bauermann via Guix-patches via
@ 2022-04-29 12:54 ` Ludovic Courtès
0 siblings, 0 replies; 4+ messages in thread
From: Ludovic Courtès @ 2022-04-29 12:54 UTC (permalink / raw)
To: Thiago Jung Bauermann; +Cc: 55107-done, Andrew Tropin
Hi,
Thiago Jung Bauermann <bauermann@kolabnow.com> skribis:
> This looks great. Just a couple of comments:
>
> Andrew Tropin <andrew@trop.in> writes:
>
>> [[PGP Signed Part:Undecided]]
>>
>> * gnu/packages/shells.scm (zsh)[arguments]: Build and install info pages.
Applied with the changes Thiago suggested.
Thanks!
Ludo’.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [bug#55107] closed (Re: bug#55107: [PATCH] gnu: zsh: Build and install info pages.)
[not found] ` <handler.55107.D55107.16512368901526.notifdone@debbugs.gnu.org>
@ 2022-05-02 12:14 ` Andrew Tropin
0 siblings, 0 replies; 4+ messages in thread
From: Andrew Tropin @ 2022-05-02 12:14 UTC (permalink / raw)
To: 55107
[-- Attachment #1: Type: text/plain, Size: 3310 bytes --]
On 2022-04-29 12:55, GNU bug Tracking System wrote:
> Your bug report
>
> #55107: [PATCH] gnu: zsh: Build and install info pages.
>
> which was filed against the guix-patches package, has been closed.
>
> The explanation is attached below, along with your original report.
> If you require more details, please reply to 55107@debbugs.gnu.org.
>
> --
> 55107: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=55107
> GNU Bug Tracking System
> Contact help-debbugs@gnu.org with problems
> From: Ludovic Courtès <ludo@gnu.org>
> Subject: Re: bug#55107: [PATCH] gnu: zsh: Build and install info pages.
> To: Thiago Jung Bauermann <bauermann@kolabnow.com>
> Cc: 55107-done@debbugs.gnu.org, Andrew Tropin <andrew@trop.in>
> Date: Fri, 29 Apr 2022 14:54:34 +0200
>
> Hi,
>
> Thiago Jung Bauermann <bauermann@kolabnow.com> skribis:
>
>> This looks great. Just a couple of comments:
>>
>> Andrew Tropin <andrew@trop.in> writes:
>>
>>> [[PGP Signed Part:Undecided]]
>>>
>>> * gnu/packages/shells.scm (zsh)[arguments]: Build and install info pages.
>
> Applied with the changes Thiago suggested.
Thank you very much!
>
> Thanks!
>
> Ludo’.
>
> From: Andrew Tropin <andrew@trop.in>
> Subject: [PATCH] gnu: zsh: Build and install info pages.
> To: guix-patches@gnu.org
> Date: Mon, 25 Apr 2022 09:40:02 +0300
>
>
> * gnu/packages/shells.scm (zsh)[arguments]: Build and install info pages.
> ---
> gnu/packages/shells.scm | 10 ++++++++--
> 1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/shells.scm b/gnu/packages/shells.scm
> index 3a48bc7dbf..7158a5bcfb 100644
> --- a/gnu/packages/shells.scm
> +++ b/gnu/packages/shells.scm
> @@ -19,6 +19,7 @@
> ;;; Copyright © 2020, 2022 Marius Bakke <marius@gnu.org>
> ;;; Copyright © 2021, 2022 Nicolas Goaziou <mail@nicolasgoaziou.fr>
> ;;; Copyright © 2021, 2022 Felix Gruber <felgru@posteo.net>
> +;;; Copyright © 2022 Andrew Tropin <andrew@trop.in>
> ;;;
> ;;; This file is part of GNU Guix.
> ;;;
> @@ -64,6 +65,7 @@ (define-module (gnu packages shells)
> #:use-module (gnu packages tls)
> #:use-module (gnu packages version-control)
> #:use-module (gnu packages xorg)
> + #:use-module (gnu packages texinfo)
> #:use-module (guix build-system cargo)
> #:use-module (guix build-system cmake)
> #:use-module (guix build-system gnu)
> @@ -522,9 +524,13 @@ (define-public zsh
> (("command -pv") "command -v")
> (("command -p") "command ")
> (("'command' -p") "'command' "))
> - #t)))))
> + #t))
> + (add-after 'build 'make-info
> + (lambda _ (invoke "make" "info")))
> + (add-after 'build 'install-info
> + (lambda _ (invoke "make" "install.info"))))))
> (native-inputs (list autoconf))
> - (inputs (list ncurses pcre perl))
> + (inputs (list ncurses pcre perl texinfo))
> (synopsis "Powerful shell for interactive use and scripting")
> (description "The Z shell (zsh) is a Unix shell that can be used
> as an interactive login shell and as a powerful command interpreter
--
Best regards,
Andrew Tropin
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 853 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-05-02 12:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-25 6:40 [bug#55107] [PATCH] gnu: zsh: Build and install info pages Andrew Tropin
2022-04-27 3:51 ` Thiago Jung Bauermann via Guix-patches via
2022-04-29 12:54 ` bug#55107: " Ludovic Courtès
[not found] ` <handler.55107.D55107.16512368901526.notifdone@debbugs.gnu.org>
2022-05-02 12:14 ` [bug#55107] closed (Re: bug#55107: [PATCH] gnu: zsh: Build and install info pages.) Andrew Tropin
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).