Author: svn-role Date: Mon May 27 04:01:38 2013 New Revision: 1486506 URL: http://svn.apache.org/r1486506 Log: Merge r1483781 from trunk:
* r1483781 Fix FSFS repository corruption on power or network disk failure on Windows: http://svn.haxx.se/dev/archive-2013-05/0245.shtml Justification: Repository corruption. Regression from 1.6.x Votes: +1: ivan, blair, danielsh Modified: subversion/branches/1.8.x/ (props changed) subversion/branches/1.8.x/STATUS subversion/branches/1.8.x/subversion/libsvn_subr/io.c Propchange: subversion/branches/1.8.x/ ------------------------------------------------------------------------------ Merged /subversion/trunk:r1483781 Modified: subversion/branches/1.8.x/STATUS URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1486506&r1=1486505&r2=1486506&view=diff ============================================================================== --- subversion/branches/1.8.x/STATUS (original) +++ subversion/branches/1.8.x/STATUS Mon May 27 04:01:38 2013 @@ -105,14 +105,6 @@ Approved changes: # blocking issues. If in doubt see this link for details: # http://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization - * r1483781 - Fix FSFS repository corruption on power or network disk failure on Windows: - http://svn.haxx.se/dev/archive-2013-05/0245.shtml - Justification: - Repository corruption. Regression from 1.6.x - Votes: - +1: ivan, blair, danielsh - * r1482436 Resolve regression in svn_ra_replay_range() where it didn't apply the send_changes FALSE flag to property changes. Modified: subversion/branches/1.8.x/subversion/libsvn_subr/io.c URL: http://svn.apache.org/viewvc/subversion/branches/1.8.x/subversion/libsvn_subr/io.c?rev=1486506&r1=1486505&r2=1486506&view=diff ============================================================================== --- subversion/branches/1.8.x/subversion/libsvn_subr/io.c (original) +++ subversion/branches/1.8.x/subversion/libsvn_subr/io.c Mon May 27 04:01:38 2013 @@ -3472,17 +3472,8 @@ svn_io_write_unique(const char **tmp_pat err = svn_io_file_write_full(new_file, buf, nbytes, NULL, pool); - /* ### BH: Windows doesn't have the race condition between the write and the - ### rename that other operating systems might have. So allow windows - ### to decide when it wants to perform the disk synchronization using - ### the normal file locking and journaling filesystem rules. - - ### Note that this function doesn't handle the rename, so we aren't even - ### sure that we really have to sync. */ -#ifndef WIN32 - if (!err && nbytes > 0) + if (!err) err = svn_io_file_flush_to_disk(new_file, pool); -#endif return svn_error_trace( svn_error_compose_create(err,
