This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] install.texi: Correct mention of Fortran in "full distribution" desc.


Gerald Pfeifer wrote:
On Thu, 12 Oct 2006, Steve Kargl wrote:
I'd be interested in opinions on whether or not I should add a parenthetical note saying "(Fortran 77 in GCC 3.4 and earlier, Fortran 95 in GCC 4.0 and later)" after "Fortran" in the list, to cover the changeover from g77 to gfortran. Comments?

I don't think the parenthetical comment is needed. The patch looks OK to me, but I probably can't approve it.

One think we like to achieve with install.texi, as far as possible without completely bending over, is to keep it applicable to older releases as well (which is where the form of the Ada references comes from, I believe).


In this specific instance, I tend to agree with Steve in favor of omitting
the parenthetical note (not the least because GNU Fortran is going to be
more than Fortran 95, IIUC).

Please make sure not to exceed line lengths of 72 characters (which the
current patch seems to do) and commit, Brooks.

72 characters is the appropriate limit? In the original patch, I was going by the surrounding text, which (in the comment lines with rows of asterisks, in particular) seemed to be limiting at 79 characters -- most of the surrounding lines are in the 72-79 character range....


In any case, I've revised this to limit the paragraph to lines of no more than 72 characters, and added a couple of "the"s that seemed to want adding. Committed as attached.

- Brooks
Index: install.texi
===================================================================
--- install.texi	(revision 117891)
+++ install.texi	(working copy)
@@ -460,11 +460,11 @@
 Please refer to the @uref{http://gcc.gnu.org/releases.html,,releases web page}
 for information on how to obtain GCC@.
 
-The full distribution includes the C, C++, Objective-C, Fortran 77, Fortran
-(in case of GCC 4.0 and later), Java, and Ada (in case of GCC 3.1 and later)
-compilers.  The full distribution also includes runtime libraries for C++,
-Objective-C, Fortran 77, Fortran, and Java.  In GCC 3.0 and later versions,
-GNU compiler testsuites are also included in the full distribution.
+The full distribution includes the C, C++, Objective-C, Fortran, Java,
+and Ada (in the case of GCC 3.1 and later) compilers.  The full
+distribution also includes runtime libraries for C++, Objective-C,
+Fortran, and Java.  In GCC 3.0 and later versions, the GNU compiler
+testsuites are also included in the full distribution.
 
 If you choose to download specific components, you must download the core
 GCC distribution plus any language specific distributions you wish to

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]