You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2015/03/28 14:16:22 UTC
svn commit: r1669769 - /subversion/branches/1.9.x/STATUS
Author: stefan2
Date: Sat Mar 28 13:16:21 2015
New Revision: 1669769
URL: http://svn.apache.org/r1669769
Log:
* STATUS: Add 1.9 API doccu & declaration fixes r1668598 through r1668608.
Modified:
subversion/branches/1.9.x/STATUS
Modified: subversion/branches/1.9.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1669769&r1=1669768&r2=1669769&view=diff
==============================================================================
--- subversion/branches/1.9.x/STATUS (original)
+++ subversion/branches/1.9.x/STATUS Sat Mar 28 13:16:21 2015
@@ -231,6 +231,54 @@ Other candidate changes:
Votes:
+1: stefan2
+ * r1668598
+ Fixes to the public svn_config API declarations and documentation.
+ Justification:
+ Have a complete, correct and easily accessible API description.
+ None of these changes should affect generated code nor the ABI.
+ Votes:
+ +1: stefan2
+
+ * r1668599, r1668608
+ Fixes to the public svn_io API declarations and documentation.
+ Justification:
+ Have a complete, correct and easily accessible API description.
+ None of these changes should affect generated code nor the ABI.
+ Votes:
+ +1: stefan2
+
+ * r1668600
+ Fixes to the public RA layer API declarations and documentation.
+ Justification:
+ Have a complete, correct and easily accessible API description.
+ None of these changes should affect generated code nor the ABI.
+ Votes:
+ +1: stefan2
+
+ * r1668602
+ Fixes to the public svn_string API declarations and documentation.
+ Justification:
+ Have a complete, correct and easily accessible API description.
+ None of these changes should affect generated code nor the ABI.
+ Votes:
+ +1: stefan2
+
+ * r1668603
+ Fixes to the public svn_wc API declarations and documentation.
+ Justification:
+ Have a complete, correct and easily accessible API description.
+ None of these changes should affect generated code nor the ABI.
+ Votes:
+ +1: stefan2
+
+ * r1668607
+ Fixes to the public svn_diff API declarations and documentation.
+ Justification:
+ Have a complete, correct and easily accessible API description.
+ None of these changes should affect generated code nor the ABI.
+ Votes:
+ +1: stefan2
+
Veto-blocked changes:
=====================
Re: svn commit: r1669769 - /subversion/branches/1.9.x/STATUS
Posted by Stefan Fuhrmann <st...@wandisco.com>.
On Sun, Mar 29, 2015 at 9:49 AM, Daniel Shahaf <d....@daniel.shahaf.name>
wrote:
> stefan2@apache.org wrote on Sat, Mar 28, 2015 at 13:16:22 -0000:
> > Author: stefan2
> > Date: Sat Mar 28 13:16:21 2015
> > New Revision: 1669769
> >
> > URL: http://svn.apache.org/r1669769
> > Log:
> > * STATUS: Add 1.9 API doccu & declaration fixes r1668598 through
> r1668608.
> >
> > Modified:
> > subversion/branches/1.9.x/STATUS
> >
> > Modified: subversion/branches/1.9.x/STATUS
> > URL:
> http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1669769&r1=1669768&r2=1669769&view=diff
> >
> ==============================================================================
> > --- subversion/branches/1.9.x/STATUS (original)
> > +++ subversion/branches/1.9.x/STATUS Sat Mar 28 13:16:21 2015
> > @@ -231,6 +231,54 @@ Other candidate changes:
> > Votes:
> > +1: stefan2
> >
> > + * r1668598
> > + Fixes to the public svn_config API declarations and documentation.
> > + Justification:
> > + Have a complete, correct and easily accessible API description.
> > + None of these changes should affect generated code nor the ABI.
> > + Votes:
> > + +1: stefan2
>
> Voting is not required for documentation fixes, according to
> <
> https://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization
> >.
>
> I think you can just move the doxygen fixes to the approved section.
> They're practically obvious fixes.
>
Good point. That reduces noise for reviewers.
I moved the docstring-only patches to "approved" now.
-- Stefan^2.
Re: svn commit: r1669769 - /subversion/branches/1.9.x/STATUS
Posted by Stefan Fuhrmann <st...@wandisco.com>.
On Sun, Mar 29, 2015 at 9:49 AM, Daniel Shahaf <d....@daniel.shahaf.name>
wrote:
> stefan2@apache.org wrote on Sat, Mar 28, 2015 at 13:16:22 -0000:
> > Author: stefan2
> > Date: Sat Mar 28 13:16:21 2015
> > New Revision: 1669769
> >
> > URL: http://svn.apache.org/r1669769
> > Log:
> > * STATUS: Add 1.9 API doccu & declaration fixes r1668598 through
> r1668608.
> >
> > Modified:
> > subversion/branches/1.9.x/STATUS
> >
> > Modified: subversion/branches/1.9.x/STATUS
> > URL:
> http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1669769&r1=1669768&r2=1669769&view=diff
> >
> ==============================================================================
> > --- subversion/branches/1.9.x/STATUS (original)
> > +++ subversion/branches/1.9.x/STATUS Sat Mar 28 13:16:21 2015
> > @@ -231,6 +231,54 @@ Other candidate changes:
> > Votes:
> > +1: stefan2
> >
> > + * r1668598
> > + Fixes to the public svn_config API declarations and documentation.
> > + Justification:
> > + Have a complete, correct and easily accessible API description.
> > + None of these changes should affect generated code nor the ABI.
> > + Votes:
> > + +1: stefan2
>
> Voting is not required for documentation fixes, according to
> <
> https://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization
> >.
>
> I think you can just move the doxygen fixes to the approved section.
> They're practically obvious fixes.
>
Good point. That reduces noise for reviewers.
I moved the docstring-only patches to "approved" now.
-- Stefan^2.
Re: svn commit: r1669769 - /subversion/branches/1.9.x/STATUS
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
stefan2@apache.org wrote on Sat, Mar 28, 2015 at 13:16:22 -0000:
> Author: stefan2
> Date: Sat Mar 28 13:16:21 2015
> New Revision: 1669769
>
> URL: http://svn.apache.org/r1669769
> Log:
> * STATUS: Add 1.9 API doccu & declaration fixes r1668598 through r1668608.
>
> Modified:
> subversion/branches/1.9.x/STATUS
>
> Modified: subversion/branches/1.9.x/STATUS
> URL: http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1669769&r1=1669768&r2=1669769&view=diff
> ==============================================================================
> --- subversion/branches/1.9.x/STATUS (original)
> +++ subversion/branches/1.9.x/STATUS Sat Mar 28 13:16:21 2015
> @@ -231,6 +231,54 @@ Other candidate changes:
> Votes:
> +1: stefan2
>
> + * r1668598
> + Fixes to the public svn_config API declarations and documentation.
> + Justification:
> + Have a complete, correct and easily accessible API description.
> + None of these changes should affect generated code nor the ABI.
> + Votes:
> + +1: stefan2
Voting is not required for documentation fixes, according to
<https://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization>.
I think you can just move the doxygen fixes to the approved section.
They're practically obvious fixes.
Daniel
Re: svn commit: r1669769 - /subversion/branches/1.9.x/STATUS
Posted by Daniel Shahaf <d....@daniel.shahaf.name>.
stefan2@apache.org wrote on Sat, Mar 28, 2015 at 13:16:22 -0000:
> Author: stefan2
> Date: Sat Mar 28 13:16:21 2015
> New Revision: 1669769
>
> URL: http://svn.apache.org/r1669769
> Log:
> * STATUS: Add 1.9 API doccu & declaration fixes r1668598 through r1668608.
>
> Modified:
> subversion/branches/1.9.x/STATUS
>
> Modified: subversion/branches/1.9.x/STATUS
> URL: http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1669769&r1=1669768&r2=1669769&view=diff
> ==============================================================================
> --- subversion/branches/1.9.x/STATUS (original)
> +++ subversion/branches/1.9.x/STATUS Sat Mar 28 13:16:21 2015
> @@ -231,6 +231,54 @@ Other candidate changes:
> Votes:
> +1: stefan2
>
> + * r1668598
> + Fixes to the public svn_config API declarations and documentation.
> + Justification:
> + Have a complete, correct and easily accessible API description.
> + None of these changes should affect generated code nor the ABI.
> + Votes:
> + +1: stefan2
Voting is not required for documentation fixes, according to
<https://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization>.
I think you can just move the doxygen fixes to the approved section.
They're practically obvious fixes.
Daniel