Re: [PATCH] Fix for Issue #3781 (Case Sensitive Authz)

2011-02-09 Thread Arwin Arni
On Thursday 10 February 2011 12:48 PM, Arwin Arni wrote: On Wednesday 09 February 2011 09:21 PM, Arwin Arni wrote: Hi All, Here's the patch for the implementation of the logic. Regards, Arwin Arni Sorry, Please ignore the previous patch. There was a stray caller of svn_config_create (whose s

Re: [PATCH] Fix for Issue #3781 (Case Sensitive Authz)

2011-02-09 Thread Arwin Arni
On Wednesday 09 February 2011 09:21 PM, Arwin Arni wrote: Hi All, Here's the patch for the implementation of the logic. Regards, Arwin Arni Sorry, Please ignore the previous patch. There was a stray caller of svn_config_create (whose signature I changed and didn't rev as it was new in the 1.

SlikSVN 1.6.5 Bug - Backslash in password causes auth failure on Windows 7

2011-02-09 Thread Kevin Benton
Login problems when committing changes to svn repository In order to reproduce this issue, I used the following while checking in changes to my GPL LICENSE file... > svn ci -m "Removed instructions from LICENSE file." --username myuserid --password "pphxC9k4\Q=bNL8:" --trust-server-cert --non-int

[PATCH] Clarify that the issue tracker hasn't moved yet.

2011-02-09 Thread Kevin Benton
Index: issue-tracker.html === --- issue-tracker.html (revision 1069204) +++ issue-tracker.html (working copy) @@ -53,9 +53,9 @@ -We've not migrated our issue tracker data to - Apache.org from Tigris.org. As such, the links and

[PATCH] cat tests fails for http:// proto

2011-02-09 Thread Noorul Islam K M
It looks like for file:// and http:// protocols cat prints different warning messages for non-existing target. For http:// svn: warning: W160013: '/svn-test-work/repositories/cat_tests-9/!svn/bc/1/non-existing' path not found For file:// svn: warning: W160013: File not found: revision 1, path

Re: [PATCH] svn command - add - Multiple targets

2011-02-09 Thread Noorul Islam K M
Noorul Islam K M writes: > This patch is a followup of the following thread. All tests pass with > this patch. > > http://svn.haxx.se/dev/archive-2011-01/0210.shtml > > Log > [[[ > > Make svn 'add' command to return 1 when one or more targets fail. Also > print error message at the end. > > * sub

Integrating further changes from the performance branch

2011-02-09 Thread Stefan Fuhrmann
Hi there, Those among you following the commits will have noticed that I opened 5 new branches each containing a cherry- picked patch set from the performance branch (details below). These branches can be merged independent of each other and an integration branch will be merged to /trunk on the

Re: svn commit: r1068695 - in /subversion/branches/integrate-stream-api-extensions: ./ subversion/include/svn_io.h subversion/libsvn_subr/stream.c subversion/libsvn_subr/subst.c

2011-02-09 Thread Stefan Fuhrmann
On 09.02.2011 02:07, Blair Zajac wrote: On 2/8/11 4:01 PM, stef...@apache.org wrote: Author: stefan2 Date: Wed Feb 9 00:01:04 2011 New Revision: 1068695 URL: http://svn.apache.org/viewvc?rev=1068695&view=rev Log: Introduce svn_stream_skip() and svn_stream_buffered() stream API functions and

Re: svn commit: r1069176 - in /subversion/branches/integrate-stream-api-extensions/subversion: include/svn_io.h libsvn_subr/stream.c libsvn_subr/subst.c

2011-02-09 Thread Blair Zajac
On 2/9/11 4:35 PM, stef...@apache.org wrote: Author: stefan2 Date: Thu Feb 10 00:35:35 2011 New Revision: 1069176 URL: http://svn.apache.org/viewvc?rev=1069176&view=rev Log: Incorporate review feedback from Blair (see http://svn.haxx.se/dev/archive-2011-02/0259.shtml). * subversion/include/svn_

Re: does 'svnadmin dump/verify' use the membuffer caches by default?

2011-02-09 Thread Stefan Fuhrmann
On 09.02.2011 03:43, Daniel Shahaf wrote: Stefan, < wayita> stefan2: danielsh said: does 'svnadmin dump/verify' use the caches by default? <@stefan2> danielsh: yes. However, the biggest performance improvement for dump / verify will come from delta-caching. This is like 2 mer

Re: Remove svn_wc__db_temp_reset_format()

2011-02-09 Thread Hyrum K Wright
On Wed, Feb 9, 2011 at 10:52 PM, Philip Martin wrote: > Hyrum K Wright writes: > >> The svn_wc__db_temp_reset_format() function is only used in the >> upgrade process, but I'm not really sure its intent.  Does anybody >> familiar with the process know?  (the upgrade tests all pass (or >> appropri

Re: Remove svn_wc__db_temp_reset_format()

2011-02-09 Thread Philip Martin
Hyrum K Wright writes: > The svn_wc__db_temp_reset_format() function is only used in the > upgrade process, but I'm not really sure its intent. Does anybody > familiar with the process know? (the upgrade tests all pass (or > appropriately xfail) with this patch.) I think it's a relic of when w

Re: svn commit: r1068863 - /subversion/trunk/build/generator/gen_msvc_dsp.py

2011-02-09 Thread Joe Swatosh
On Wed, Feb 9, 2011 at 4:32 AM, wrote: > Author: rhuijben > Date: Wed Feb  9 12:32:57 2011 > New Revision: 1068863 > > URL: http://svn.apache.org/viewvc?rev=1068863&view=rev > Log: > Following up on r1034362, fix generating the Visual C++ 6.0 style project > and workspace files. Visual C++ 6.0 is

Remove svn_wc__db_temp_reset_format()

2011-02-09 Thread Hyrum K Wright
I'm looking at some of the temp functions introduced early in the development of wc-ng. I suspect some of them can be removed, but just want to check and see if I'm missing something. The svn_wc__db_temp_reset_format() function is only used in the upgrade process, but I'm not really sure its inte

1.6.16 next week (16 Feb)

2011-02-09 Thread Hyrum K Wright
Hello folks: I'd like to cut 1.6.16 next Wednesday. Please nominate / review / vote / etc in the mean time. -Hyrum

Re: [PATCH] Fix issue #3686 - executable bit not set during merge

2011-02-09 Thread Daniel Becroft
> > > Index: subversion/libsvn_wc/merge.c >> > === >> > --- subversion/libsvn_wc/merge.c (revision 1068136) >> > +++ subversion/libsvn_wc/merge.c (working copy) >> > @@ -1094,6 +1094,7 @@ >> >const char *l

Re: svn commit: r1068814 - /subversion/trunk/subversion/tests/libsvn_diff/diff-diff3-test.c

2011-02-09 Thread Lieven Govaerts
Daniel, one comment on a comment inline. On Wed, Feb 9, 2011 at 10:41 AM, wrote: > Author: danielsh > Date: Wed Feb 9 09:41:38 2011 > New Revision: 1068814 > > URL: http://svn.apache.org/viewvc?rev=1068814&view=rev > Log: > Silence the random XPASSes in the bots. > > * subversion/tests/libsvn_

svn info and svnversion show wrong last committed revision when switching branches.

2011-02-09 Thread Bert Wesarg
Hi, I have svn version 1.6.12 (r955767) here. Creating a new file in a subdirectory in ^/trunk, than merging this ^/trunk into a branch and switching back to ^/trunk, still shows me as the last changed revision of this file in ^/trunk the merge revision of the branch. The last changed revision fo

Re: SQLite and callbacks

2011-02-09 Thread Ivan Zhakov
On Wed, Feb 9, 2011 at 23:06, Stefan Sperling wrote: > On Wed, Feb 09, 2011 at 10:56:35PM +0300, Ivan Zhakov wrote: >> On Wed, Feb 9, 2011 at 17:40, Ivan Zhakov wrote: >> > On Windows there is FILE_ATTRIBUTE_TEMPORARY [1] attribute, which >> > prevents file to be written to disk: >> > [[[ >> > Sp

Re: SQLite and callbacks

2011-02-09 Thread Stefan Sperling
On Wed, Feb 09, 2011 at 10:56:35PM +0300, Ivan Zhakov wrote: > On Wed, Feb 9, 2011 at 17:40, Ivan Zhakov wrote: > > On Windows there is FILE_ATTRIBUTE_TEMPORARY [1] attribute, which > > prevents file to be written to disk: > > [[[ > > Specifying the FILE_ATTRIBUTE_TEMPORARY attribute causes file s

Re: SQLite and callbacks

2011-02-09 Thread Ivan Zhakov
On Wed, Feb 9, 2011 at 17:40, Ivan Zhakov wrote: > On Tue, Feb 8, 2011 at 12:50, Bert Huijben wrote: >>> -Original Message- >>> From: Branko Čibej [mailto:br...@xbc.nu] On Behalf Of Branko Cibej >>> Sent: dinsdag 8 februari 2011 4:39 >>> To: dev@subversion.apache.org >>> Subject: Re: SQLi

Re: svn commit: r1069001 - in /subversion/trunk/subversion: libsvn_wc/wc_db.c libsvn_wc/wc_db.h tests/libsvn_wc/op-depth-test.c

2011-02-09 Thread Philip Martin
Branko Čibej writes: > On 09.02.2011 18:54, phi...@apache.org wrote: >> Author: philip >> Date: Wed Feb 9 17:54:02 2011 >> New Revision: 1069001 >> >> URL: http://svn.apache.org/viewvc?rev=1069001&view=rev >> Log: >> The new revert is non-recursive in the database so remove the depth >> paramete

Re: apr_file_flush and short writes

2011-02-09 Thread Blair Zajac
On 2/9/11 1:38 AM, John Szakmeister wrote: On Mon, Feb 7, 2011 at 4:26 PM, Blair Zajac wrote: [I sent this to d...@apr.apache.org but haven't received a response. Thread here: http://mail-archives.apache.org/mod_mbox/apr-dev/201102.mbox/%3cf7b1928d-d32f-48dd-b8d9-80b26906a...@orcaware.com%3E .

Re: svn commit: r1069001 - in /subversion/trunk/subversion: libsvn_wc/wc_db.c libsvn_wc/wc_db.h tests/libsvn_wc/op-depth-test.c

2011-02-09 Thread Branko Čibej
On 09.02.2011 18:54, phi...@apache.org wrote: > Author: philip > Date: Wed Feb 9 17:54:02 2011 > New Revision: 1069001 > > URL: http://svn.apache.org/viewvc?rev=1069001&view=rev > Log: > The new revert is non-recursive in the database so remove the depth > parameter. Allow actual-only revert. > >

Re: [PATCH] svn command - cat - Multiple targets

2011-02-09 Thread C. Michael Pilato
On 02/09/2011 02:24 AM, Noorul Islam K M wrote: > "C. Michael Pilato" writes: > >> The patch no longer applies cleanly to trunk. Can you regenerate it? > > Please find attached updated patch. > > Thanks and Regards > Noorul > Committed in r1068977, with a minor spelling fix. Thanks. -- C.

RE: svn_diff.h truncated

2011-02-09 Thread Stimely, Noelle
Thanks! Noelle -Original Message- From: C. Michael Pilato [mailto:cmpil...@collab.net] Sent: Wednesday, February 09, 2011 7:33 AM To: Stimely, Noelle Cc: Subversion Development Subject: Re: svn_diff.h truncated On 02/08/2011 05:28 PM, Stimely, Noelle wrote: > How do I unsubscribe to t

Re: [PATCH] Fix for Issue #3781 (Case Sensitive Authz)

2011-02-09 Thread Arwin Arni
Hi All, Here's the patch for the implementation of the logic. Regards, Arwin Arni Index: subversion/libsvn_subr/config.c === --- subversion/libsvn_subr/config.c (revision 1068869) +++ subversion/libsvn_subr/config.c (workin

Re: svn_diff.h truncated

2011-02-09 Thread C. Michael Pilato
On 02/08/2011 05:28 PM, Stimely, Noelle wrote: > How do I unsubscribe to this group? Send mail from the subscribed account to dev-unsubscr...@subversion.apache.org . -- C. Michael Pilato CollabNet <> www.collab.net <> Distributed Development On Demand signature.asc Description: Ope

Re: [PATCH] Fix for Issue #3781 (Case Sensitive Authz)

2011-02-09 Thread Arwin Arni
On Wednesday 09 February 2011 07:02 PM, Daniel Shahaf wrote: Arwin Arni wrote on Wed, Feb 09, 2011 at 17:39:55 +0530: * subversion/libsvn_repos/authz.c, subversion/tests/libsvn_subr/cache-test.c, subversion/tests/libsvn_subr/config-test.c, subversion/tests/cmdline/atomic-ra-revprop-chan

Re: SQLite and callbacks

2011-02-09 Thread Ivan Zhakov
On Tue, Feb 8, 2011 at 12:50, Bert Huijben wrote: > > >> -Original Message- >> From: Branko Čibej [mailto:br...@xbc.nu] On Behalf Of Branko Cibej >> Sent: dinsdag 8 februari 2011 4:39 >> To: dev@subversion.apache.org >> Subject: Re: SQLite and callbacks >> >> On 07.02.2011 21:51, Stefan Sp

Re: [PATCH] Fix for Issue #3781 (Case Sensitive Authz)

2011-02-09 Thread Daniel Shahaf
Arwin Arni wrote on Wed, Feb 09, 2011 at 17:39:55 +0530: > * subversion/libsvn_repos/authz.c, > subversion/tests/libsvn_subr/cache-test.c, > subversion/tests/libsvn_subr/config-test.c, > subversion/tests/cmdline/atomic-ra-revprop-change.c, > subversion/svnserve/serve.c, > subversion/libsv

Re: diff4-optimization-bytes

2011-02-09 Thread Johan Corveleyn
On Wed, Feb 9, 2011 at 10:20 AM, Daniel Shahaf wrote: > In other news, I looked into the cause --- tried to make > datasource_get_next_token() do one more loop in the place where > currently it does 'return if at_start_of_suffix()' --- but that didn't > fix the truncation... Indeed, that won't fi

[PATCH] Fix for Issue #3781 (Case Sensitive Authz)

2011-02-09 Thread Arwin Arni
Hi All, This patch fixes Issue #3781 -> http://subversion.tigris.org/issues/show_bug.cgi?id=3781 and the corresponding XFail case in authz_tests.py (case_sensitive_authz). This makes the section names of the authz files case sensitive. Attached is the patch and log message. Index: subversi

Re: [PATCH] - Fix for issue #3792

2011-02-09 Thread Noorul Islam K M
Daniel Shahaf writes: > Noorul Islam K M wrote on Wed, Feb 09, 2011 at 14:28:49 +0530: > >> Daniel Shahaf writes: >> > Noorul Islam K M wrote on Sat, Feb 05, 2011 at 12:55:55 +0530: >> >> Daniel Shahaf writes: >> >> > Noorul Islam K M wrote on Thu, Feb 03, 2011 at 14:15:48 +0530: >> >> >>

Re: apr_file_flush and short writes

2011-02-09 Thread John Szakmeister
On Mon, Feb 7, 2011 at 4:26 PM, Blair Zajac wrote: > [I sent this to d...@apr.apache.org but haven't received a response. Thread > here: > http://mail-archives.apache.org/mod_mbox/apr-dev/201102.mbox/%3cf7b1928d-d32f-48dd-b8d9-80b26906a...@orcaware.com%3E > .  Given the importance of writing compl

Re: [PATCH] - Fix for issue #3792

2011-02-09 Thread Daniel Shahaf
Noorul Islam K M wrote on Wed, Feb 09, 2011 at 14:28:49 +0530: > Daniel Shahaf writes: > > Noorul Islam K M wrote on Sat, Feb 05, 2011 at 12:55:55 +0530: > >> Daniel Shahaf writes: > >> > Noorul Islam K M wrote on Thu, Feb 03, 2011 at 14:15:48 +0530: > >> >>SVN_ERR(svn_wc__node_get_re

Re: diff4-optimization-bytes

2011-02-09 Thread Daniel Shahaf
In other news, I looked into the cause --- tried to make datasource_get_next_token() do one more loop in the place where currently it does 'return if at_start_of_suffix()' --- but that didn't fix the truncation... In the meantime, I tweaked a test to make it XFail (r1068798). From a quick glance

RE: svn_diff.h truncated

2011-02-09 Thread Stimely, Noelle
How do I unsubscribe to this group? Thanks... Noelle -Original Message- From: Johan Corveleyn [mailto:jcor...@gmail.com] Sent: Tuesday, February 08, 2011 2:19 PM To: Daniel Shahaf; dev@subversion.apache.org Subject: Re: svn_diff.h truncated On Tue, Feb 8, 2011 at 10:50 PM, Stefan S

Re: [PATCH] - Fix for issue #3792

2011-02-09 Thread Noorul Islam K M
Noorul Islam K M writes: > Daniel Shahaf writes: > >> Noorul Islam K M wrote on Sat, Feb 05, 2011 at 12:55:55 +0530: >> >>> Daniel Shahaf writes: >>> > Noorul Islam K M wrote on Thu, Feb 03, 2011 at 14:15:48 +0530: >>> >> + if (info->depth == svn_depth_exclude) >>> >> +SVN_ERR(svn_cmdline_

RE: [PATCH] New XFail test for issue 3781

2011-02-09 Thread Kamesh Jayachandran
Thanks Prabhu committed your patch in r1068802. With regards Kamesh Jayachandran -Original Message- From: Prabhu Gnana Sundar Ponnarasu Sent: Wed 2/9/2011 2:30 PM To: Bert Huijben Cc: Kamesh Jayachandran; dev@subversion.apache.org Subject: Re: [PATCH] New XFail test for issue 3781 Hi Be

Re: diff4-optimization-bytes

2011-02-09 Thread Daniel Shahaf
Johan Corveleyn wrote on Wed, Feb 09, 2011 at 08:42:20 +0100: > On Wed, Feb 9, 2011 at 4:54 AM, Daniel Shahaf wrote: > > The experimental code is svn_diff_diff4_2(); AFAIK svn_diff_diff4() is > > as stable as ever. > > I have no objections on adding such an annotation. However, from where > I'm s

Re: [PATCH] Re: Regarding issue 3690 - Work in progress

2011-02-09 Thread Noorul Islam K M
Hyrum K Wright writes: > On Tue, Feb 8, 2011 at 1:41 AM, Daniel Shahaf wrote: > >> Hyrum K Wright wrote on Mon, Feb 07, 2011 at 23:14:10 +: >>> On Fri, Feb 4, 2011 at 4:36 PM, Noorul Islam K M wrote: >>> > hwright noorul: ignored_props_mod takes a list of properties, yes (as >>> >        op

Re: [PATCH] New XFail test for issue 3781

2011-02-09 Thread Prabhu Gnana Sundar
Hi Bert, The problem was that my patch handled the expected error more specifically and also failed to handle the svnserve case. That was wrong on my part. I have tweaked the test case and attached a patch and log message with this mail. Please share your views on the same. On Tuesday 08 Fe

Re: [PATCH] - Fix for issue #3792

2011-02-09 Thread Noorul Islam K M
Daniel Shahaf writes: > Noorul Islam K M wrote on Sat, Feb 05, 2011 at 12:55:55 +0530: > >> Daniel Shahaf writes: >> > Noorul Islam K M wrote on Thu, Feb 03, 2011 at 14:15:48 +0530: >> >> + if (info->depth == svn_depth_exclude) >> >> +SVN_ERR(svn_cmdline_printf(pool, _("Depth: exclude\n")))