diff mbox

Remove stray '@' from install.texi (was Re: [PATCH] Delete GCJ)

Message ID 1480468255.7673.135.camel@redhat.com
State New
Headers show

Commit Message

David Malcolm Nov. 30, 2016, 1:10 a.m. UTC 
On Tue, 2016-11-29 at 14:23 -0700, Jeff Law wrote:
> On 11/21/2016 04:23 PM, Matthias Klose wrote:

> > On 21.11.2016 18:16, Rainer Orth wrote:

> > > Hi Matthias,

> > > 

> > > > ahh, didn't see that :-/ Now fixed, is this clearer now?

> > > > 

> > > > The options @option{--with-target-bdw-gc-include} and

> > > > @option{--with-target-bdw-gc-lib} must always specified

> > > > together for

> > >                                                ^ be

> > 

> > thanks to all sorting out the documentation issues. Now attaching

> > the updated

> > diff. Ok to commit?

> > 

> > Matthias

> > 

> > 


[...]

> > gcc/

> > 

> > 2016-11-19  Matthias Klose  <doko@ubuntu.com>

> > 

> >         * doc/install.texi: Document configure options --enable

> > -objc-gc

> >         and --with-target-bdw-gc.


[...]

r242985 seems to have broken the build, for me at least (with texinfo
5.1):

../../src/gcc/doc/install.texi:2199: use braces to give a command as an argument to @=
make[2]: *** [doc/gccinstall.info] Error 1

The attached patch fixes it.

OK to commit?
Dave

Comments

Sandra Loosemore Nov. 30, 2016, 1:20 a.m. UTC | #1 
On 11/29/2016 06:10 PM, David Malcolm wrote:
>[snip]

>

> r242985 seems to have broken the build, for me at least (with texinfo

> 5.1):

>

> ../../src/gcc/doc/install.texi:2199: use braces to give a command as an argument to @=

> make[2]: *** [doc/gccinstall.info] Error 1

>

> The attached patch fixes it.

>

> OK to commit?


OK.  (This is so trivial it would qualify under the obvious patch rule 
anyway.)

-Sandra
David Malcolm Nov. 30, 2016, 1:26 a.m. UTC | #2 
On Tue, 2016-11-29 at 18:20 -0700, Sandra Loosemore wrote:
> On 11/29/2016 06:10 PM, David Malcolm wrote:

> > [snip]

> > 

> > r242985 seems to have broken the build, for me at least (with

> > texinfo

> > 5.1):

> > 

> > ../../src/gcc/doc/install.texi:2199: use braces to give a command

> > as an argument to @=

> > make[2]: *** [doc/gccinstall.info] Error 1

> > 

> > The attached patch fixes it.

> > 

> > OK to commit?

> 

> OK.  (This is so trivial it would qualify under the obvious patch

> rule 

> anyway.)

> 

> -Sandra


My texinfo skills aren't as strong as yours, so I was half-wondering if
this was some syntax I wasn't aware of, or maybe a version issue. 

Thanks for the confirmation; committed to trunk as r242991.

Dave
diff mbox

Patch

From 60ec369e17c898fb3cdcbca43ed77d450a30d074 Mon Sep 17 00:00:00 2001
From: David Malcolm <dmalcolm@redhat.com>
Date: Tue, 29 Nov 2016 20:38:53 -0500
Subject: [PATCH] Remove stray character from install.texi

gcc/ChangeLog:
	* doc/install.texi (--with-target-bdw-gc): Remove stray '@'.
---
 gcc/doc/install.texi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi
index 5d96e5f..140ff80 100644
--- a/gcc/doc/install.texi
+++ b/gcc/doc/install.texi
@@ -2196,7 +2196,7 @@  continues.
 @itemx --with-target-bdw-gc-lib=@var{list}
 Specify search directories for the garbage collector header files and
 libraries. @var{list} is a comma separated list of key value pairs of the
-form @samp{@var{multilibdir}@=@var{path}}, where the default multilib key
+form @samp{@var{multilibdir}=@var{path}}, where the default multilib key
 is named as @samp{.} (dot), or is omitted (e.g.
 @samp{--with-target-bdw-gc=/opt/bdw-gc,32=/opt-bdw-gc32}).
 
-- 
1.8.5.3