From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-return-192766-listarch-gcc=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 34804 invoked by alias); 14 Mar 2017 20:29:29 -0000
Mailing-List: contact gcc-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc/>
List-Post: <mailto:gcc@gcc.gnu.org>
List-Help: <http://gcc.gnu.org/ml/>
Sender: gcc-owner@gcc.gnu.org
Received: (qmail 34791 invoked by uid 89); 14 Mar 2017 20:29:29 -0000
Authentication-Results: sourceware.org; auth=none
X-Virus-Found: No
X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE,SPF_PASS autolearn=ham version=3.3.2 spammy=doo
X-HELO: ainaz.pair.com
Received: from ainaz.pair.com (HELO ainaz.pair.com) (209.68.2.66) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Tue, 14 Mar 2017 20:29:28 +0000
Received: from anthias (vie-188-118-249-200.dsl.sil.at [188.118.249.200])	(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))	(No client certificate requested)	by ainaz.pair.com (Postfix) with ESMTPSA id E7E243F530;	Tue, 14 Mar 2017 16:29:24 -0400 (EDT)
Date: Tue, 14 Mar 2017 20:29:00 -0000
From: Gerald Pfeifer <gerald@pfeifer.com>
To: Palmer Dabbelt <palmer@dabbelt.com>
cc: gcc@gcc.gnu.org
Subject: Re: [PATCH] RISC-V documentation cleanups
In-Reply-To: <20170313173933.5524-2-palmer@dabbelt.com>
Message-ID: <alpine.LSU.2.20.1703142125300.3498@anthias.pfeifer.com>
References: <alpine.LSU.2.20.1703130951510.3498@anthias.pfeifer.com> <20170313173933.5524-1-palmer@dabbelt.com> <20170313173933.5524-2-palmer@dabbelt.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
X-IsSubscribed: yes
X-SW-Source: 2017-03/txt/msg00074.txt.bz2

On Mon, 13 Mar 2017, Palmer Dabbelt wrote:
> A recent mailing list post about install.texi cleanup suggested I 
> take a look at ours, and there were a few problems:
> 
>  * No table of contents entries
>  * Not alphabetically ordered
>  * Missing a note about requiring binutils-2.28

This looks fine with a few changes, thank you.

Perhaps a bit repetitive?  I'm wondering, would it make sense to 
have general riscv64-*-* entries that cover the general items?

> 2017-03-13  Palmer Dabbelt  <palmer@dabbelt.com
> 
>         * doc/install.texi (Specific) <riscv32-*-elf>: Add table of contents
>         link.
>         <riscv32-*-linux>: Likewise.
>         <riscv64-*-elf>: Likewise
>         <riscv64-*-linux>: Likewise.

Here I'd just say

	* doc/install.texi (Specific): Add foo, bar, didl, doo to the
	table of contents.

>         <riscv32-*-elf>: Add a note about requiring binutils-2.28.

Here and in the text, binutils 2.28 (without the dash).

Approved.

Gerald

PS: We usually send patches to the gcc-patches@ list.