STATUS - subversion - Git at Google

       * * * * * * * * * * * * * * * * * * * * * * * * * * * *
       *                                                     *
       *  THIS RELEASE STREAM IS OPEN TO BUG FIXES.          *
       *                                                     *
       * * * * * * * * * * * * * * * * * * * * * * * * * * * *

 This file tracks the status of releases in the 1.8.x line.

 See http://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization
 for details on how release lines and voting work, what kinds of bugs can
 delay a release, etc.

 Status of 1.8.5:

 Candidate changes:
 ==================

  * r1469519, r1469520, r1481848, r1481870
    Silence compiler warnings, tighten scopes and improve code.
    Justification:
      Clean builds make it more likely to notice real compiler warnings.
    Branch: ^/subversion/branches/1.8.x-r1469519-and-friends
    Votes:
      +1: blair, brane
      -0: philip (see dev thread: false alarm, trades one warning for another)

  * r1498000
    Improve error handling for inherited property requests by using the
    transition based xml parser in libsvn_ra_serf.
    Justification:
      This maintenance work which usually doesn't belong in a patch release
      makes it possible to properly detect if this request should be restarted
      on non-chunk-enabled proxies.
    Votes:
      +1: rhuijben, ivan

  * r1496127
    Fix occasional failure of checkout_tests.py 12.
    Justification:
      Spurious FAILs confuse people testing the release.
    Votes:
      +1: philip, rhuijben
      -0: danielsh (loses functionality; patch sent with alternative fix)
      stsp: where exactly is the alternative patch?
      danielsh: http://mid.gmane.org/20130705040533.GL3244@lp-shahaf.local

  * r1501293, r1501338, r1501360, r1501533
    Fix issue #4390, "parallel nested checkout not possible with wc-ng"
    Justification:
      Regression from 1.6. User complained. See issue for details.
    Branch:
      ^/subversion/branches/1.8.x-issue4390
    Depends:
      Nomination conflicts with the branch and merges cleanly without it.
      Branch should probably be removed from the nomination (it was only needed
      before r1503552).
    Votes:
      +1: stsp, rhuijben
      -0: ivan, brane (not regression from 1.7.x,
                       destructive change in libsvn_wc core)

  * r1524145
    Make the 'm' conflict prompt option run external merge tool if configured.
    Justification:
      Usability fix. The current behaviour ('m' -> internal merge tool,
      'l' -> external merge tool) confuses users who try 'm' to run an
      external merge tool. See issue #4426.
    Votes:
      +1: stsp

  * r1534149, r1535676
    In the windows CryptoAPI ssl certificate verification, properly handle
    certificates that are only trusted via intermediate authorities.
    Justification:
       While not strictly a bug, intermediate authorities are getting more
       common since recent Certificate Authority incidents.
    Notes:
       rhuijben: This introduces a new credential type as private API.
       Not sure about our strict ABI guarantees here, but this affects
       usability of users migrating to cloud platforms, etc.
    Votes:
      +1: rhuijben

  * r1499470
    Flush stdout before exiting svn with an error.
    Justification:
       Without this patch merge_tests.py 135 may fail to output its required
       output.
    Votes:
      +1: rhuijben

  * r1535115
    Make JavaHL tests run without installing on OS X.
    Justification:
      Shouldn't need to install code to test it.
    Votes:
      +1: breser, brane

  * r1535139
    Make openssl lib and include directory available when using --with-openssl
    on Windows.
    Justification:
      Serf 1.2 simply included the openssl static library in its own static
      library, which made it very easy for Subversion to consume, but added the
      risk of linking to many different openssl versions (both static and
      shared builds) at once. This bug in serf was fixed for 1.3 and we
      updated Subversion to use 1.3 for Subversion 1.8.4, but without this
      patch we added the assumption that ssleay32.lib and libeay32.lib are
      within some existing referenced library directory.
      This patch fixes that assumption by adding the right libdir to our build,
      by borrowing some code from trunk.
    Branch: ^/subversion/branches/1.8.x-openssl-dirs
    Votes:
      +1: rhuijben, ivan

  * r1535161
    Make swig-rb tests run without installing on OS X.
    Justification:
      Shouldn't need to install code to test it.  The existing Makefile target
      is just downright wrong.
    Votes:
      +1: breser, brane

  * r1535532
    Make the certificate verification handling in ra_serf properly ask multiple
    providers for acceptance of certificate failures.
    Justification:
      Without this patch it is not possible to have one hook accept invalid
      ca roots, while another provider accepts an invalid date on a certificate.
      Probably only interesting for bindings and code that explicitly tests
      our certificate handling behavior.
    Votes:
      +1: rhuijben

  * r1531014, r1532316, r1536350, r1536383, r1536464, r1536488
    Fix working copy database inconsistency on certain move scenarios,
    including those described in issue #4437 "Deleting a moved file can maintain a
    half move"
    Justification:
      The working copy database should be consistent on any normal operation.
    Votes:
      +1: rhuijben

  * r1536537
    Fix build of ctypes-python on newer OS X versions.
    Justificaton:
      Build system should work.  We shouldn't be trying to use the CPP from the
      APR build anyway.
    Votes:
      +1: breser, brane

  * r1518942, r1536914, r1536931
    Fix commit of a specific mixed revision copy with non copy descendants that
    shadow not present nodes. This problem manifests itself as a copy of a tree
    that can't be committed because it is reported out of date.
    Justification:
      'svn cp TREE NEW-TREE; svn ci NEW-TREE' should never report out of date
      on some random descendant.
    Branch: ^/subversion/branches/1.8.x-r1536931
    Votes:
      +1: rhuijben

  * r1536854
    Make 'svnadmin verify' detect inconsistencies that will prevent loading
    dump files.
    Justification:
      Some users rely on dump files as a means of repository backup.  Without
      this patch, there is no way except of 'svnadmin load' to know that these
      dump files will load at all.  With this patch, a successful verify run
      should guarantee loadable dump files.
    Branch: ^/subversion/branches/1.8.x-r1536854
    Votes:
      +1: stefan2

  * r1537018
    Fix issue #4440, "Subversion sets permissions on files, when it shouldn't"
    Justification:
      Bug introduced by me. Unreasonable behaviour. User complained.
    Votes:
      +1: stsp
      +0.5: rhuijben (Change looks sane, but I'm unable to test this)

  * r1537193, r1537221
    Add test to verify if the used ZLib is affected by some block size bugs that
    affect our usage of serf.
    Justification:
      Most Windows binaries were affected by this bug and we should verify
      that we don't accidentally reintroduce this problem.
    Notes:
      This patch needs the source directory for obtaining its test data. The
      backport branch contains portions of r1537147 and r1537190 to enable
      this test.
    Branch: ^/subversion/branches/1.8.x-r1537193/
    Votes:
      +1: rhuijben, brane

  * r1538516, r1538517, r1538519, r1538574, r1538581
    Fix issue #4444, "diffing move inside a copy asserts"
    Justification:
      I want to get to a pub on Friday so let's get this done.
      http://svn.haxx.se/users/archive-2013-11/0037.shtml
    Votes:
      +1: stsp, rhuijben

  * r1538812, r1538925, r1538928, r1538939
    Properly remove moved_to information when the node in its original location
    is removed.
    Justification:
      The NODES table in the working copy database should always be consistent.
    Notes:
      rhuijben: The test for this problem might accidentally depend on other
                move fixes on trunk.
    Votes:
      +1: rhuijben

 Veto-blocked changes:
 =====================

 Approved changes:
 =================
	* * * * * * * * * * * * * * * * * * * * * * * * * * * *
	* *
	* THIS RELEASE STREAM IS OPEN TO BUG FIXES. *
	* *
	* * * * * * * * * * * * * * * * * * * * * * * * * * * *

	This file tracks the status of releases in the 1.8.x line.

	See http://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization
	for details on how release lines and voting work, what kinds of bugs can
	delay a release, etc.

	Status of 1.8.5:

	Candidate changes:
	==================

	* r1469519, r1469520, r1481848, r1481870
	Silence compiler warnings, tighten scopes and improve code.
	Justification:
	Clean builds make it more likely to notice real compiler warnings.
	Branch: ^/subversion/branches/1.8.x-r1469519-and-friends
	Votes:
	+1: blair, brane
	-0: philip (see dev thread: false alarm, trades one warning for another)

	* r1498000
	Improve error handling for inherited property requests by using the
	transition based xml parser in libsvn_ra_serf.
	Justification:
	This maintenance work which usually doesn't belong in a patch release
	makes it possible to properly detect if this request should be restarted
	on non-chunk-enabled proxies.
	Votes:
	+1: rhuijben, ivan

	* r1496127
	Fix occasional failure of checkout_tests.py 12.
	Justification:
	Spurious FAILs confuse people testing the release.
	Votes:
	+1: philip, rhuijben
	-0: danielsh (loses functionality; patch sent with alternative fix)
	stsp: where exactly is the alternative patch?
	danielsh: http://mid.gmane.org/20130705040533.GL3244@lp-shahaf.local

	* r1501293, r1501338, r1501360, r1501533
	Fix issue #4390, "parallel nested checkout not possible with wc-ng"
	Justification:
	Regression from 1.6. User complained. See issue for details.
	Branch:
	^/subversion/branches/1.8.x-issue4390
	Depends:
	Nomination conflicts with the branch and merges cleanly without it.
	Branch should probably be removed from the nomination (it was only needed
	before r1503552).
	Votes:
	+1: stsp, rhuijben
	-0: ivan, brane (not regression from 1.7.x,
	destructive change in libsvn_wc core)

	* r1524145
	Make the 'm' conflict prompt option run external merge tool if configured.
	Justification:
	Usability fix. The current behaviour ('m' -> internal merge tool,
	'l' -> external merge tool) confuses users who try 'm' to run an
	external merge tool. See issue #4426.
	Votes:
	+1: stsp

	* r1534149, r1535676
	In the windows CryptoAPI ssl certificate verification, properly handle
	certificates that are only trusted via intermediate authorities.
	Justification:
	While not strictly a bug, intermediate authorities are getting more
	common since recent Certificate Authority incidents.
	Notes:
	rhuijben: This introduces a new credential type as private API.
	Not sure about our strict ABI guarantees here, but this affects
	usability of users migrating to cloud platforms, etc.
	Votes:
	+1: rhuijben

	* r1499470
	Flush stdout before exiting svn with an error.
	Justification:
	Without this patch merge_tests.py 135 may fail to output its required
	output.
	Votes:
	+1: rhuijben

	* r1535115
	Make JavaHL tests run without installing on OS X.
	Justification:
	Shouldn't need to install code to test it.
	Votes:
	+1: breser, brane

	* r1535139
	Make openssl lib and include directory available when using --with-openssl
	on Windows.
	Justification:
	Serf 1.2 simply included the openssl static library in its own static
	library, which made it very easy for Subversion to consume, but added the
	risk of linking to many different openssl versions (both static and
	shared builds) at once. This bug in serf was fixed for 1.3 and we
	updated Subversion to use 1.3 for Subversion 1.8.4, but without this
	patch we added the assumption that ssleay32.lib and libeay32.lib are
	within some existing referenced library directory.
	This patch fixes that assumption by adding the right libdir to our build,
	by borrowing some code from trunk.
	Branch: ^/subversion/branches/1.8.x-openssl-dirs
	Votes:
	+1: rhuijben, ivan

	* r1535161
	Make swig-rb tests run without installing on OS X.
	Justification:
	Shouldn't need to install code to test it. The existing Makefile target
	is just downright wrong.
	Votes:
	+1: breser, brane

	* r1535532
	Make the certificate verification handling in ra_serf properly ask multiple
	providers for acceptance of certificate failures.
	Justification:
	Without this patch it is not possible to have one hook accept invalid
	ca roots, while another provider accepts an invalid date on a certificate.
	Probably only interesting for bindings and code that explicitly tests
	our certificate handling behavior.
	Votes:
	+1: rhuijben

	* r1531014, r1532316, r1536350, r1536383, r1536464, r1536488
	Fix working copy database inconsistency on certain move scenarios,
	including those described in issue #4437 "Deleting a moved file can maintain a
	half move"
	Justification:
	The working copy database should be consistent on any normal operation.
	Votes:
	+1: rhuijben

	* r1536537
	Fix build of ctypes-python on newer OS X versions.
	Justificaton:
	Build system should work. We shouldn't be trying to use the CPP from the
	APR build anyway.
	Votes:
	+1: breser, brane

	* r1518942, r1536914, r1536931
	Fix commit of a specific mixed revision copy with non copy descendants that
	shadow not present nodes. This problem manifests itself as a copy of a tree
	that can't be committed because it is reported out of date.
	Justification:
	'svn cp TREE NEW-TREE; svn ci NEW-TREE' should never report out of date
	on some random descendant.
	Branch: ^/subversion/branches/1.8.x-r1536931
	Votes:
	+1: rhuijben

	* r1536854
	Make 'svnadmin verify' detect inconsistencies that will prevent loading
	dump files.
	Justification:
	Some users rely on dump files as a means of repository backup. Without
	this patch, there is no way except of 'svnadmin load' to know that these
	dump files will load at all. With this patch, a successful verify run
	should guarantee loadable dump files.
	Branch: ^/subversion/branches/1.8.x-r1536854
	Votes:
	+1: stefan2

	* r1537018
	Fix issue #4440, "Subversion sets permissions on files, when it shouldn't"
	Justification:
	Bug introduced by me. Unreasonable behaviour. User complained.
	Votes:
	+1: stsp
	+0.5: rhuijben (Change looks sane, but I'm unable to test this)

	* r1537193, r1537221
	Add test to verify if the used ZLib is affected by some block size bugs that
	affect our usage of serf.
	Justification:
	Most Windows binaries were affected by this bug and we should verify
	that we don't accidentally reintroduce this problem.
	Notes:
	This patch needs the source directory for obtaining its test data. The
	backport branch contains portions of r1537147 and r1537190 to enable
	this test.
	Branch: ^/subversion/branches/1.8.x-r1537193/
	Votes:
	+1: rhuijben, brane

	* r1538516, r1538517, r1538519, r1538574, r1538581
	Fix issue #4444, "diffing move inside a copy asserts"
	Justification:
	I want to get to a pub on Friday so let's get this done.
	http://svn.haxx.se/users/archive-2013-11/0037.shtml
	Votes:
	+1: stsp, rhuijben

	* r1538812, r1538925, r1538928, r1538939
	Properly remove moved_to information when the node in its original location
	is removed.
	Justification:
	The NODES table in the working copy database should always be consistent.
	Notes:
	rhuijben: The test for this problem might accidentally depend on other
	move fixes on trunk.
	Votes:
	+1: rhuijben

	Veto-blocked changes:
	=====================

	Approved changes:
	=================