diff4-optimization-bytes

2011-01-28 Thread Daniel Shahaf
Johan, I'm concerned about this change: on the one hand, it's untested and no one claims to be understanding the code; on the other hand, it doesn't exactly parallel the diff3 change: specifically, the last hunk of the diff3 patch (which is also included below) has no equivalent in the diff4 patc

Re: svn commit: r1064261 - /subversion/trunk/COMMITTERS

2011-01-28 Thread Julian Foad
Daniel Shahaf wrote: > I'd rather not remove their entries entirely from COMMITTERS. +1. Being unable to contact them (currently) is a completely different thing from deciding to remove them from the list of committers. And failure to keep one's email address up to date is not a serious breach o

Re: svn commit: r1064200 - in /subversion/branches/uris-as-urls: ./ subversion/include/ subversion/libsvn_client/ subversion/libsvn_ra/ subversion/libsvn_ra_local/ subversion/libsvn_ra_neon/ subversio

2011-01-28 Thread Julian Foad
On Thu, 2011-01-27, cmpil...@apache.org wrote: > URL: http://svn.apache.org/viewvc?rev=1064200&view=rev > Log: > On the 'uris-as-urls' branch: Begin ensuring that the relpaths > returned from svn_url_* functions are URI-decoded. > > * BRANCH-README > Update status. > > * subversion/include/svn_

Does uris-as-urls break ra_svn on Windows?

2011-01-28 Thread Philip Martin
cmpil...@apache.org writes: > Author: cmpilato > Date: Fri Jan 28 02:34:40 2011 > New Revision: 1064436 > > URL: http://svn.apache.org/viewvc?rev=1064436&view=rev > Log: > Reintegration the 'uris-as-urls' branch into trunk. > (Merged /subversion/branches/uris-as-urls:r1060426-1064427.) > > This sh

Re: Does uris-as-urls break ra_svn on Windows?

2011-01-28 Thread Philip Martin
Philip Martin writes: > cmpil...@apache.org writes: > >> Author: cmpilato >> Date: Fri Jan 28 02:34:40 2011 >> New Revision: 1064436 >> >> URL: http://svn.apache.org/viewvc?rev=1064436&view=rev >> Log: >> Reintegration the 'uris-as-urls' branch into trunk. >> (Merged /subversion/branches/uris-as-

RE: [Feature] New feature about handling 'HTTP Error 302 Moved temporarily'?

2011-01-28 Thread Bolstridge, Andrew
> > On 01/27/2011 08:27 PM, Wu Di wrote: > > I found path in subversion sometimes too long, it's hard to be sent by > > Email and something else. > > And 'Tiny URL' is more popular now. it redirect a short URL to a long one. > > > > I thing is a good idea if TortoiseSvn could handle 'HTTP Error 30

Re: Status of the branch diff-optimizations-bytes branch

2011-01-28 Thread Johan Corveleyn
On Fri, Jan 28, 2011 at 5:51 AM, Hyrum K Wright wrote: > On Tue, Jan 25, 2011 at 8:31 PM, Johan Corveleyn wrote: >> On Tue, Jan 25, 2011 at 4:58 PM, Hyrum K Wright >> wrote: >>> Johan (and other interested parties), >>> I've been following some of the commits to the >>> diff-optimizations-branc

Re: Document return codes of 'svn ls'

2011-01-28 Thread Julian Foad
On Thu, 2011-01-27, C. Michael Pilato wrote: > Pssst... file the issue. I chose to file an "enhancement" issue about including the error number in the messages: . - Julian > On 01/26/2011 01:24 PM, Daniel Shahaf wrote: > > Julian Foad wr

Re: diff4-optimization-bytes

2011-01-28 Thread Johan Corveleyn
On Fri, Jan 28, 2011 at 9:29 AM, Daniel Shahaf wrote: > Johan, > > I'm concerned about this change: on the one hand, it's untested and > no one claims to be understanding the code; on the other hand, it > doesn't exactly parallel the diff3 change: > > specifically, the last hunk of the diff3 patch

Re: [PATCH] fix for issue #3789 log -g regression in r1028108

2011-01-28 Thread Kevin Radke
On 1/28/2011 12:59 AM, Daniel Shahaf wrote: Is this a fix or a work-around? i.e., if you set MAX_OPEN_HISTORIES to, say, 2, can you then reproduce the failure you saw before? It's just not clear to me whether MAX_OPEN_HISTORIES is in fact an assumption that "no file will be 'svn mv'd more than

Re: Document return codes of 'svn ls'

2011-01-28 Thread Julian Foad
I (Julian Foad) wrote: > On Thu, 2011-01-27, C. Michael Pilato wrote: > > Pssst... file the issue. > > I chose to file an "enhancement" issue about including the error number > in the messages: > . (To clarify "I chose": I mean there seem

Re: svn commit: r1064168 - in /subversion/trunk/subversion/include/private: svn_eol_private.h svn_fs_util.h svn_mergeinfo_private.h svn_opt_private.h svn_sqlite.h svn_wc_private.h

2011-01-28 Thread Travis
On Jan 27, 2011, at 10:50 PM, Greg Hudson wrote: > "NUL" (all caps) is sometimes used as an abbreviation for the null > character. And is (I think, though I've not read the ASCII std directly), the formal ASCII abbreviation for the non-printable '\0' character. I've certainly seen it on ASCI

Re: svn commit: r1064200 - in /subversion/branches/uris-as-urls: ./ subversion/include/ subversion/libsvn_client/ subversion/libsvn_ra/ subversion/libsvn_ra_local/ subversion/libsvn_ra_neon/ subversio

2011-01-28 Thread C. Michael Pilato
On 01/28/2011 06:42 AM, Julian Foad wrote: > On Thu, 2011-01-27, cmpil...@apache.org wrote: >> URL: http://svn.apache.org/viewvc?rev=1064200&view=rev >> Log: >> On the 'uris-as-urls' branch: Begin ensuring that the relpaths >> returned from svn_url_* functions are URI-decoded. >> >> * BRANCH-README

Re: Does uris-as-urls break ra_svn on Windows?

2011-01-28 Thread C. Michael Pilato
"We'll put our best man on it." /me makes sad puppy dog eyes at pburba. On 01/28/2011 07:33 AM, Philip Martin wrote: > Philip Martin writes: > >> cmpil...@apache.org writes: >> >>> Author: cmpilato >>> Date: Fri Jan 28 02:34:40 2011 >>> New Revision: 1064436 >>> >>> URL: http://svn.apache.org/v

Re: Status of the branch diff-optimizations-bytes branch

2011-01-28 Thread Hyrum K Wright
On Fri, Jan 28, 2011 at 6:49 AM, Johan Corveleyn wrote: > On Fri, Jan 28, 2011 at 5:51 AM, Hyrum K Wright wrote: >> On Tue, Jan 25, 2011 at 8:31 PM, Johan Corveleyn wrote: >>> On Tue, Jan 25, 2011 at 4:58 PM, Hyrum K Wright >>> wrote: Johan (and other interested parties), I've been f

Re: Does uris-as-urls break ra_svn on Windows?

2011-01-28 Thread C. Michael Pilato
Bert and I independently arrived at this patch, which he and Paul are testing: Index: subversion/svnserve/serve.c === --- subversion/svnserve/serve.c (revision 1064439) +++ subversion/svnserve/serve.c (working copy) @@ -2998,9 +2998,9

Re: Does uris-as-urls break ra_svn on Windows?

2011-01-28 Thread C. Michael Pilato
Committed this fix in r1064731. On 01/28/2011 10:26 AM, C. Michael Pilato wrote: > Bert and I independently arrived at this patch, which he and Paul are testing: > > Index: subversion/svnserve/serve.c > === > --- subversion/svnserve/

Re: svn commit: r1064261 - /subversion/trunk/COMMITTERS

2011-01-28 Thread Hyrum K Wright
On Fri, Jan 28, 2011 at 5:04 AM, Julian Foad wrote: > Daniel Shahaf wrote: >> I'd rather not remove their entries entirely from COMMITTERS. > > +1.  Being unable to contact them (currently) is a completely different > thing from deciding to remove them from the list of committers.  And > failure t

Re: Code doesn't seem ... right

2011-01-28 Thread C. Michael Pilato
On 01/27/2011 10:05 AM, Justin Erenkrantz wrote: > On Wed, Jan 26, 2011 at 10:28 AM, C. Michael Pilato > wrote: >> I'm not sure how to move forward. Should we require that SVNMasterURI >> values point to something other than the server root? > > It was never intended to be at the server root -

Re: Merging /branches/integrate-cache-membuffer to /trunk

2011-01-28 Thread Branko Čibej
On 28.01.2011 05:56, Daniel Shahaf wrote: > We could ask ASF infra to enable the "optional" feature on svn.eu > (without enabling it on svn.us). > > The experimental code gets tested, and if it cooks the disk alive and > burns the data then we (pay for a new disk and) svnsync svn.us to svn.eu :-)

apache + svn proxy does not work when repository name is used in the path inside repository.

2011-01-28 Thread Iliyan Osenski
ah_create, for WWW-Authenticate Body block (104 bytes): [] Read block (30 bytes): [ http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";> http://www.w3.org/1999/xhtml"; lang="] Read block (14 bytes): [en" xml:lang="] Read block (19 bytes): [en"> ] Read block (63 bytes): [Authentication required! <

Re: Status of the branch diff-optimizations-bytes branch

2011-01-28 Thread Julian Foad
Hyrum K Wright wrote: > On Fri, Jan 28, 2011 at 6:49 AM, Johan Corveleyn wrote: > > And one more thing came to mind with regards to the new API > > (datasources_open function): currently it only supports up to 4 > > datasources, so not an arbitrary number of datasources (the > > implementation in

Re: [PATCH] Re: Incorporate svn 1.7 changes to vc-svn.el

2011-01-28 Thread C. Michael Pilato
On 01/26/2011 12:05 AM, Noorul Islam K M wrote: > Noorul Islam K M writes: > >> After migrating working copy to 1.7, vc-svn.el is not functioning >> properly. This is because .svn/entries file does not exist. I think we >> no longer need to check existence of this file. Since .svn exists at the >>

Re: diff4-optimization-bytes

2011-01-28 Thread Daniel Shahaf
Johan Corveleyn wrote on Fri, Jan 28, 2011 at 14:04:07 +0100: > On Fri, Jan 28, 2011 at 9:29 AM, Daniel Shahaf > wrote: > > May I suggest that, if this code is to be released, then you validate > > its correctnss?  For example, a minimal regression test that is written > > to record trunk's pre-b

Re: [Feature] New feature about handling 'HTTP Error 302 Moved temporarily'?

2011-01-28 Thread Daniel Shahaf
Bolstridge, Andrew wrote on Fri, Jan 28, 2011 at 12:46:40 +: > > > > On 01/27/2011 08:27 PM, Wu Di wrote: > > > I found path in subversion sometimes too long, it's hard to be sent by > > > Email and something else. > > > And 'Tiny URL' is more popular now. it redirect a short URL to a long one

Re: [PATCH] fix for issue #3789 log -g regression in r1028108

2011-01-28 Thread Daniel Shahaf
Just a quick note that log_tests.py 27 fails when I set MAX_OPEN_HISTORIES=0 (without applying the patch). Kevin Radke wrote on Fri, Jan 28, 2011 at 07:45:40 -0600: > On 1/28/2011 12:59 AM, Daniel Shahaf wrote: >> Is this a fix or a work-around? i.e., if you set MAX_OPEN_HISTORIES to, >> say, 2,

Re: [PATCH] fix for issue #3789 log -g regression in r1028108

2011-01-28 Thread C. Michael Pilato
On 01/26/2011 05:59 PM, Kevin Radke wrote: > This also needs a back-port to the 1.6.x branch. > > [[[ >Fix issue #3789: Correctly ignore missing locations when a renamed file >has more than MAX_OPEN_HISTORIES. > >* subversion/libsvn_repos/log.c > (get_path_histories): Ignore more

Re: svn commit: r1064261 - /subversion/trunk/COMMITTERS

2011-01-28 Thread Daniel Shahaf
Hyrum K Wright wrote on Fri, Jan 28, 2011 at 09:29:47 -0600: > On Fri, Jan 28, 2011 at 5:04 AM, Julian Foad wrote: > > Daniel Shahaf wrote: > >> I'd rather not remove their entries entirely from COMMITTERS. > > > > +1.  Being unable to contact them (currently) is a completely different > > thing f

Re: svn commit: r1064261 - /subversion/trunk/COMMITTERS

2011-01-28 Thread Konstantin Kolinko
2011/1/28 Daniel Shahaf : >> The fact is that they just don't currently *have* commit >> access.  They couldn't even commit if they wanted to, due to the lack >> of an account at the ASF. >> >> I don't think we have to forget these folks (who have obviously given >> valuable contributions in the pa

Re: svn commit: r1064905 - in /subversion/branches/ignore-mergeinfo-log/subversion: libsvn_ra_svn/client.c svnserve/serve.c

2011-01-28 Thread Daniel Shahaf
What about libsvn_ra_svn/protocol ? hwri...@apache.org wrote on Fri, Jan 28, 2011 at 21:59:14 -: > Author: hwright > Date: Fri Jan 28 21:59:14 2011 > New Revision: 1064905 > > URL: http://svn.apache.org/viewvc?rev=1064905&view=rev > Log: > On the ignore-mergeinfo-log branch: > Send the list o

Re: svn commit: r1064847 - /subversion/trunk/subversion/svnserve/serve.c

2011-01-28 Thread Daniel Shahaf
hwri...@apache.org wrote on Fri, Jan 28, 2011 at 20:01:35 -: > Author: hwright > Date: Fri Jan 28 20:01:35 2011 > New Revision: 1064847 > > URL: http://svn.apache.org/viewvc?rev=1064847&view=rev > Log: > * subversion/svnserve/serve.c > (log_cmd): Remove a useless check, and replace it with a

Re: svn commit: r1064847 - /subversion/trunk/subversion/svnserve/serve.c

2011-01-28 Thread Daniel Shahaf
hwri...@apache.org wrote on Fri, Jan 28, 2011 at 20:01:35 -: > Author: hwright > Date: Fri Jan 28 20:01:35 2011 > New Revision: 1064847 > > URL: http://svn.apache.org/viewvc?rev=1064847&view=rev > Log: > * subversion/svnserve/serve.c > (log_cmd): Remove a useless check, and replace it with a

Re: [PATCH] Re: Incorporate svn 1.7 changes to vc-svn.el

2011-01-28 Thread Noorul Islam K M
"C. Michael Pilato" writes: > On 01/26/2011 12:05 AM, Noorul Islam K M wrote: > >> Noorul Islam K M writes: >> >>> After migrating working copy to 1.7, vc-svn.el is not functioning >>> properly. This is because .svn/entries file does not exist. I think we >>> no longer need to check existence of