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]

PATCH for Re: [PING] PR 28322 unknown -Wno-* options should cause warnings, not errors


Hej Peter,

I know it's been a while, but I had queued your message to actually
improve our documentation.

Please find my first attempt at this below. I have committed this patch, 
so it now shows up at http://gcc.gnu.org/contribute.html.  If you have
any thoughts or suggestions for improvement, please let me know!

(The part relevant to your request is the last part of the patch, the
rest moves and shortens the description of "without further approval").

Gerald

On Wed, 9 Jul 2008, Peter Maydell wrote:
>> This patch is OK.
> Thanks. What's the process for causing the patch to actually get applied
> to SVN? http://gcc.gnu.org/contribute.html isn't clear on the point for
> the case where the patch submitter doesn't have write-after-approval.

Index: contribute.html
===================================================================
RCS file: /cvs/gcc/wwwdocs/htdocs/contribute.html,v
retrieving revision 1.70
diff -u -3 -p -r1.70 contribute.html
--- contribute.html	6 Aug 2008 21:57:16 -0000	1.70
+++ contribute.html	26 Jan 2009 00:20:48 -0000
@@ -276,20 +276,24 @@ changing.)  For further information on t
 the <a href="svn.html">Anonymous read-only SVN access</a> and <a
 href="svnwrite.html">Read-write SVN access</a> pages.</p>
 
+<p>(Everything listed here still applies if you can check in the patch
+without further approval under the <a
+href="svnwrite.html#policies">GCC write access policies</a>, except
+that ChangeLog entries may be included as part of the patch and diffs
+representing totally new files may be omitted (especially if large).)</p> 
+
+<h3>Pinging patches, Getting patches applied</h3>
+
 <p>If you do not receive a response to a patch that you have submitted
 within two weeks or so, it may be a good idea to chase it by sending a
 follow-up email to the same list(s).  Patches can occasionally fall through
 the cracks.  Please be sure to include a brief summary of the patch and the
 URL of the entry in the mailing list archive of the original submission.</p>
 
-<p>Everything listed here still applies if you can check in the patch
-without further approval under the <a
-href="svnwrite.html#policies">GCC write access policies</a>, except
-that ChangeLog entries may be included as part of the patch since
-no-one else will need to apply it to the tree later and diffs representing
-totally new files may be omitted (especially if large) since they can be
-accessed directly from the repository.</p>
-
+<p>If you do not have write access and a patch of yours has been approved,
+but not committed, please advise the approver of that fact.  You may want
+to point out lack of write access in your initial submission, too.</p>
+ 
 
 <h2><a name="announce">Announcing Changes (to our Users)</a></h2>
 


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