diff mbox

DOC patch: about gengtype & plugins

Message ID 20110706191703.7446773a8ae678a5efa23ccc@starynkevitch.net
State New
Headers show

Commit Message

Basile Starynkevitch July 6, 2011, 5:17 p.m. UTC
Hello All,

The attached documentation patch is nearly trivial, I was tempted to apply it without review.

### gcc/ChangeLog entry ###
2011-07-06  Basile Starynkevitch  <basile@starynkevitch.net>

	* doc/plugins.texi (Building GCC plugins): gengtype needs its 
        gtype.state

### end gcc/ChangeLog entry ###

Ok?

Comments

Laurynas Biveinis July 15, 2011, 1:12 p.m. UTC | #1
2011/7/6 Basile Starynkevitch <basile@starynkevitch.net>:
>        * doc/plugins.texi (Building GCC plugins): gengtype needs its
>        gtype.state

Replace "than" in the patch with "as".

I assume it passes make info?

OK with that change, thank you.
Gerald Pfeifer July 16, 2011, 11:48 a.m. UTC | #2
On Fri, 15 Jul 2011, Laurynas Biveinis wrote:
> 2011/7/6 Basile Starynkevitch <basile@starynkevitch.net>:
>>        * doc/plugins.texi (Building GCC plugins): gengtype needs its
>>        gtype.state
> Replace "than" in the patch with "as".
> 
> I assume it passes make info?
> 
> OK with that change, thank you.

And a full stop at the end of the ChangeLog entry.  And perhaps 
"corresponding" instead of "its" in the ChangeLog?
 
Gerald
diff mbox

Patch

Index: gcc/doc/plugins.texi
===================================================================
--- gcc/doc/plugins.texi	(revision 175923)
+++ gcc/doc/plugins.texi	(working copy)
@@ -446,6 +446,6 @@  A single source file plugin may be built with @cod
 plugin.so}, using backquote shell syntax to query the @file{plugin}
 directory.
 
-Plugins needing to use @command{gengtype} require a GCC build
-directory for the same version of GCC that they will be linked
-against.
+When a plugin needs to use @command{gengtype}, be sure that both
+@file{gengtype} and @file{gtype.state} have the same version than the
+GCC for which the plugin is built.