bug-guix
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#24069: [PATCHv2] Re: bug#24069: gcc man page is broken


From: Efraim Flashner
Subject: bug#24069: [PATCHv2] Re: bug#24069: gcc man page is broken
Date: Mon, 2 Oct 2017 12:24:31 +0300
User-agent: Mutt/1.8.3 (2017-05-23)

On Sun, Oct 01, 2017 at 11:54:56AM -0400, Maxim Cournoyer wrote:
> Version 2.
> 
> Removes the unnecessary (ice-9 match) use clause I added while
> experimenting.
> 

> From a044901c0fc51571b8ed6809080029b8701b09e4 Mon Sep 17 00:00:00 2001
> From: Maxim Cournoyer <address@hidden>
> Date: Sun, 1 Oct 2017 09:27:31 -0400
> Subject: [PATCH] gnu: gcc: Fix manual pages.
> 
> Fixes bug #24069.
> 
> * gnu/packages/gcc.scm (gcc-4.7)[native-inputs]: Add perl.
> (gcc-4.9)[native-inputs]: Likewise.
> ---
>  gnu/packages/gcc.scm | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/gnu/packages/gcc.scm b/gnu/packages/gcc.scm
> index 7870d4513..7089b420f 100644
> --- a/gnu/packages/gcc.scm
> +++ b/gnu/packages/gcc.scm
> @@ -155,7 +155,8 @@ where the OS part is overloaded to denote a specific 
> ABI---into GCC
>  
>        ;; GCC < 5 is one of the few packages that doesn't ship .info files.
>        ;; Newer texinfos fail to build the manual, so we use an older one.
> -      (native-inputs `(("texinfo" ,texinfo-5)))
> +      (native-inputs `(("perl" ,perl)   ;for manpages
> +                       ("texinfo" ,texinfo-5)))
>  
>        (arguments
>         `(#:out-of-source? #t
> @@ -370,7 +371,9 @@ Go.  It also includes runtime support libraries for these 
> languages.")
>                  "14l06m7nvcvb0igkbip58x59w3nq6315k6jcz3wr9ch1rn9d44bc"))
>                (patches (search-patches "gcc-arm-bug-71399.patch"
>                                         "gcc-libvtv-runpath.patch"))))
> -    (native-inputs `(("texinfo" ,texinfo)))))
> +    ;; Override inherited texinfo-5 with latest version.
> +    (native-inputs `(("perl" ,perl)   ;for manpages
> +                     ("texinfo" ,texinfo)))))
>  
>  (define-public gcc-5
>    ;; Note: GCC >= 5 ships with .info files but 'make install' fails to 
> install
> -- 
> 2.14.1
> 

I believe this would cause gcc-5 to be rebuilt, which would cause a
world rebuild, so for the moment we also need a native-inputs entry for
gcc-5 making sure that nothing changes, ie: that we're only using
texinfo there, preferably with a note that it should be changed in the
future.  Also, you should take a look at gnu/packages/commencement and
make sure that we're not adding perl as a native-input for gcc-boot0 or
for libstdc++, wherever that happens to live (I don't remember off
hand).

I assume it is OK for us to build the man pages for gcc-final (if it
happens) since this is the gcc used for actually building packages, and
I believe the one pulled in for gcc-toolchain.


-- 
Efraim Flashner   <address@hidden>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

Attachment: signature.asc
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]