[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] Proof-of-concept patch for APR 1.2 support on Windows

From: Sergey A. Lipnevich <sergey_at_optimaltec.com>
Date: 2006-06-28 07:42:08 CEST

Hi All,

I was trying to run Subversion 1.3.x (.1 and .2) on Windows with Apache
2.2, and came across a discussion related to handling of APR_INCOMPLETE
error (http://svn.haxx.se/dev/archive-2005-12/0522.shtml) and a related
question of what to do with svn_io_set_file_read_write_carefully
(http://svn.haxx.se/dev/archive-2006-06/0093.shtml). Since my setup
wasn't working, I tried to fix Subversion 1.3.2 according to the ideas
mentioned in these threads, and came up with the following patch
(borrowing an idea from D.J. Heap's patch above for
svn_io_set_file_read_write_carefully). It's not intended for commit as
it is. Please let me know if this is the right approach to take, and
what needs to be done to finish this patch.

I ran the test suite with the following two consistent failures that I
didn't yet investigate:

FAIL: stat_tests.py 8: status for unignored file and directory
Log:
  svn: warning: 'newfile' is not under version control
  FAIL: stat_tests.py 8: status for unignored file and directory

FAIL: lock_tests.py 19: update handles svn:needs-lock correctly
Log:
  svn: '.' is not a working copy
  FAIL: lock_tests.py 19: update handles svn:needs-lock correctly

The compiler I use is from VS 2005.
Thanks!

Sergey.

[
  Initial support for APR 1.2, boils down to avoiding APR_FINFO_PROT as much
  as possible.

  * subversion/libsvn_subr/io.c:

    use APR_FREADONLY to detect read-only file attribute on Win32;

    replace contents of svn_io_set_file_read_write_carefully with
    a call to either svn_io_set_file_read_write or
    svn_io_set_file_read_only;

    don't do anything about "executable" attribute on Win32;

    if svn_io_stat gets APR_INCOMPLETE status, disregard it if all but
    APR_FINFO_PROT bits are valid;

    don't try to handle SGID bit on Win32;

    remove pre-APR 0.9.5 code related to SGID.
]

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org

Received on Wed Jun 28 07:50:38 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.