all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Efraim Flashner <efraim@flashner.co.il>
To: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Cc: 24069@debbugs.gnu.org
Subject: bug#24069: [PATCHv2] Re: bug#24069: gcc man page is broken
Date: Mon, 2 Oct 2017 12:24:31 +0300	[thread overview]
Message-ID: <20171002092431.GP26375@macbook42.flashner.co.il> (raw)
In-Reply-To: <87vajy6ey7.fsf_-_@gmail.com>

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

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 <maxim.cournoyer@gmail.com>
> 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   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

  reply	other threads:[~2017-10-02  9:25 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-25 20:23 bug#24069: gcc man page is broken Danny Milosavljevic
2017-03-30 22:22 ` Branson, Joshua A
2017-09-30 20:36   ` Maxim Cournoyer
2017-10-01 13:39     ` bug#24069: [PATCH] " Maxim Cournoyer
2017-10-01 15:54       ` bug#24069: [PATCHv2] " Maxim Cournoyer
2017-10-02  9:24         ` Efraim Flashner [this message]
2017-10-02 14:53           ` Ludovic Courtès
2017-10-02 20:15             ` Maxim Cournoyer
2017-10-03  2:28               ` bug#24069: [PATCHv3] " Maxim Cournoyer
2017-10-03 23:44                 ` bug#24069: [PATCHv4] " Maxim Cournoyer
2017-10-31 15:16                   ` Ludovic Courtès

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20171002092431.GP26375@macbook42.flashner.co.il \
    --to=efraim@flashner.co.il \
    --cc=24069@debbugs.gnu.org \
    --cc=maxim.cournoyer@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.