Add missing ChangeLog entry

Add missing ChangeLog entry

[Ian Lance Taylor]

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

I noticed that we have a --with-specs option in gcc/configure.ac, added
in revision 155208 with this e-mail message:
http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html

Nathan forgot to commit the ChangeLog entry, so I have committed this
patch to ChangeLog-2009.  (Patches to ChangeLog files do not themselves
require ChangeLog entries.)

More seriously, this option is not documented in gcc/doc/install.texi,
as pointed out by Gerald in
http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00947.html .  We've had
discussions circling around this idea in the past, and I was surprised
to discover that it has already been committed.

Nathan, would you be willing to write some docs for it?  Thanks.

Ian



[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-diff, Size: 568 bytes --]

Index: gcc/ChangeLog-2009
===================================================================
--- gcc/ChangeLog-2009	(revision 174368)
+++ gcc/ChangeLog-2009	(working copy)
@@ -474,6 +474,13 @@
 	* intl.c (get_spaces): New.
 	* intl.h (get_spaces): New.
 
+2009-12-14  Mark Mitchell  <mark@codesourcery.com>
+
+	* configure.ac (--with-specs): New option.
+	* configure: Regenerated.
+	* gcc.c (driver_self_specs): Include CONFIGURE_SPECS.
+	* Makefile.in (DRIVER_DEFINES): Add -DCONFIGURE_SPECS.
+
 2009-12-14  Jakub Jelinek  <jakub@redhat.com>
 
 	PR bootstrap/42369

Re: Add missing ChangeLog entry

[Nathan Sidwell]

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

On 06/01/11 15:32, Ian Lance Taylor wrote:
> I noticed that we have a --with-specs option in gcc/configure.ac, added
> in revision 155208 with this e-mail message:
> http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html

sorry about that.  How's the attachd documentation?


-- 
Nathan Sidwell


[-- Attachment #2: doc.patch --]
[-- Type: text/x-patch, Size: 733 bytes --]

2011-06-02  Nathan Sidwell  <nathan@codesourcery.com>

	* doc/install.texi (Options specification): Document --with-specs.

Index: doc/install.texi
===================================================================
--- doc/install.texi	(revision 174559)
+++ doc/install.texi	(working copy)
@@ -771,6 +771,12 @@
 on other configuration options, and differs between cross and native
 configurations.
 
+@item --with-specs=@var{specs}
+Specify additional command line driver SPECS.  This can be useful if
+you to turn on a non-standard feature by default without modifying the
+compiler's source code, for instance
+@option{--with-specs=%@{!fcommon:%@{!fno-common:-fno-common@}@}}.
+
 @end table
 
 @item --program-prefix=@var{prefix}

Re: Add missing ChangeLog entry

[Basile Starynkevitch]

On Thu, 02 Jun 2011 11:12:12 +0100
Nathan Sidwell <nathan@codesourcery.com> wrote:

> On 06/01/11 15:32, Ian Lance Taylor wrote:
> > I noticed that we have a --with-specs option in gcc/configure.ac, added
> > in revision 155208 with this e-mail message:
> > http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html
> 
> sorry about that.  How's the attachd documentation?

[...]
+@item --with-specs=@var{specs}
+Specify additional command line driver SPECS.  This can be useful if
+you to turn on a non-standard feature by default without modifying the
+compiler's source code, for instance
+@option{--with-specs=%@{!fcommon:%@{!fno-common:-fno-common@}@}}.

I am not a native English speaker, and my english is bad. But perhaps it should be
"this can be useful if you *want* to turn on"
I feel that the 'want' word is missing...

Regards.
-- 
Basile STARYNKEVITCH         http://starynkevitch.net/Basile/
email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359
8, rue de la Faiencerie, 92340 Bourg La Reine, France
*** opinions {are only mine, sont seulement les miennes} ***

Re: Add missing ChangeLog entry

[Ian Lance Taylor]

Nathan Sidwell <nathan@codesourcery.com> writes:

> On 06/01/11 15:32, Ian Lance Taylor wrote:
>> I noticed that we have a --with-specs option in gcc/configure.ac, added
>> in revision 155208 with this e-mail message:
>> http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html
>
> sorry about that.  How's the attachd documentation?

Works for me, but please add an @xref{Spec Files} (you might need a
document in there too, not sure) as a pointer to where the spec format
is documented.

Thanks.

Ian

Re: Add missing ChangeLog entry

[Nathan Sidwell]

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

On 06/02/11 18:13, Ian Lance Taylor wrote:

> Works for me, but please add an @xref{Spec Files} (you might need a
> document in there too, not sure) as a pointer to where the spec format
> is documented.

ok, this is what I committed, cribbing from the rs6000 xref in the same file.

nathan

-- 
Nathan Sidwell


[-- Attachment #2: all.diff --]
[-- Type: text/plain, Size: 952 bytes --]

2011-06-03  Nathan Sidwell  <nathan@codesourcery.com>

	* doc/install.texi (Options specification): Document --with-specs.

Index: doc/install.texi
===================================================================
--- doc/install.texi	(revision 174597)
+++ doc/install.texi	(working copy)
@@ -771,6 +771,19 @@
 on other configuration options, and differs between cross and native
 configurations.
 
+@item --with-specs=@var{specs}
+Specify additional command line driver SPECS.
+This can be useful if you need to turn on a non-standard feature by
+default without modifying the compiler's source code, for instance
+@option{--with-specs=%@{!fcommon:%@{!fno-common:-fno-common@}@}}.
+@ifnothtml
+@xref{Spec Files,, Specifying subprocesses and the switches to pass to them,
+gcc, Using the GNU Compiler Collection (GCC)},
+@end ifnothtml
+@ifhtml
+See ``Spec Files'' in the main manual
+@end ifhtml
+
 @end table
 
 @item --program-prefix=@var{prefix}