* * * * * * * * * * * * * * * * * * * * * * * * * * * *
      *                                                     *
      *  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.1:

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)

 * r1496007, r1496111
   Fix issue #4383, "Subversion 1.8.0 crash when WC path is symlink"
   Justification:
     Crash, regression from 1.7.
   Votes:
     +1: stsp, breser
     +0: rhuijben (looks good)

 * 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

 * r1495597
   Fix a build warning, followup to already-merged r1493424.
   Justification:
     Makes the build warning-free (which in turn makes the bot useful).
   Votes:
     +1: danielsh, philip

 * r1501199, r1501207
   Verify the result of the xml parser in ra serf, instead of assuming the
   parsing always succeeds.
   Justification:
     While in general we can just rely on the parsed result being incomplete,
     this makes many network (and server implementation) problems easier to
     diagnose.
   Votes:
     +1: rhuijben

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

 * r1489117, r1496470, r1497975, r1497980, r1498012, r1499423, r1499595
   Support for HTTP/1.1 reverse proxies that require Content-Length headers
   and deny chunked requests.
   Branch: ^/subversion/branches/1.8.x-busted-proxy
   Justification:
     Older nginx servers cannot handle chunked requests. This patch will
     keep HTTP/1.1 features in the connection, but will always use C-L
     headers for the length (at a variant performance cost). This increases
     svn's compatibility in various environments. Most installations will
     never see this; the runtime test is only enabled with a config settting.
   Notes:
     r1489117: this is actually unrelated to the proxy work, but it removes
               some redundant minimum-serf-version checking and allows the
               actual work to cleanly merge.
     r1496470: add/respect using_chunked_requests flag to session state
     r1497975: rename config variable, tweak session variables
     r1497980: add runtime/dynamic checking of proxy support
     r1498012: remove improper check of client-side proxy; the real problem
               is within the server-side reverse proxy
     r1499423: fix 411 handling
     r1499595: don't probe when a redirect occurs
     
     Local to branch:
     r1499225: remove SVN_CONFIG_OPTION_BUSTED_PROXY from the public API,
               since we cannot add a symbol in a patch release.
   Votes:
     +1: gstein, philip, lgo
     -1: breser (given danielsh's veto, the discussion on IRC and my email to
                 dev I think we need to discuss this more before approving)
     -1: danielsh (option name is wrong. Those proxies follow the spec, so they
                   are not busted, period.)

 * r1500837
   Follow up to the r1489117 group, modifies the 411 Content length required
   error message to point the user to the busted-proxy option.
   Justification:
     The original error message was meaningless, this patch at least guides
     the user in the right direction.
   Notes:
     Depends on the r1489117 group (logically, not merge-wise).
   Votes:
     +1: lgo, rhuijben, danielsh, breser
     -1: (not vetoed, but depends on the r1489117 group which is)

 * r1500857
   A further clarification after r1500837
   Justification:
     Save users a tour to $SEARCH_ENGINE.
   Depends: r1500837
   Votes:
     +1: danielsh, lgo, breser
     +1: rhuijben (but also +1 on not applying it if that would be the
                   consensus)
     -1: (not vetoed, but depends on the r1489117 group which is)

 * r1501049
   ra_serf: Return Subversion error codes in svn_error_t.
   Justification:
     Other error codes are useless to API users.
   Votes:
     +1: danielsh
     -1: rhuijben (We also report apr and OS statuses, which are certainly
                   not useless, as GUI clients depend on this extensive error
                   codes to provide proper workarounds.
                   This use-case should be fixed by directly setting
                   the right message in the error chain when leaving ra_serf.
                   This adds up to a -0 for me.
                   
                   The reason for the -1 is the re-use of a specific error code
                   that is automatically unwrapped in some ra_serf code, to avoid
                   handling codes like APR_EOF as non fatal)

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