RE: [WIP] Enable passing copyfrom information for the diff code when dealing with repository diffs.

2010-08-16 Thread Bert Huijben
-Original Message- From: Daniel Näslund [mailto:dan...@longitudo.com] Sent: zondag 15 augustus 2010 12:27 To: dev@subversion.apache.org Subject: [WIP] Enable passing copyfrom information for the diff code when dealing with repository diffs. Hi! Why doesn't svn_ra_do_diff3()

RE: svn diff - Is this behaviour expected?

2010-08-16 Thread Bert Huijben
-Original Message- From: Vincent Lefevre [mailto:vincent-...@vinc17.net] Sent: zondag 15 augustus 2010 16:50 To: dev@subversion.apache.org Subject: Re: svn diff - Is this behaviour expected? On 2010-08-15 05:34:08 -0700, Bert Huijben wrote: Conversely, when blah has been

Re: svn commit: r985514 - stream_move_mark()

2010-08-16 Thread Julian Foad
On Sat, 2010-08-14, stef...@apache.org wrote: Log: Extend the stream API by three functions: svn_stream_move_mark() to move an existing mark by some delta Hi Stefan. Unfortunately this is not a logical extension to the stream API. Subversion's svn_stream_t streams support arbitrary filtering

Re: svn commit: r985514 - stream_move_mark()

2010-08-16 Thread Julian Foad
On Mon, 2010-08-16 at 09:55 +0100, Julian Foad wrote: On Sat, 2010-08-14, stef...@apache.org wrote: Log: Extend the stream API by three functions: svn_stream_move_mark() to move an existing mark by some delta Hi Stefan. Unfortunately this is not a logical extension to the stream API.

Re: svn commit: r985514 - stream_move_mark()

2010-08-16 Thread Stefan Sperling
On Mon, Aug 16, 2010 at 10:04:31AM +0100, Julian Foad wrote: On Mon, 2010-08-16 at 09:55 +0100, Julian Foad wrote: On Sat, 2010-08-14, stef...@apache.org wrote: Log: Extend the stream API by three functions: svn_stream_move_mark() to move an existing mark by some delta Hi Stefan.

Re: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-08-16 Thread Stephen Butler
On Aug 16, 2010, at 10:40 , Julian Foad wrote: On Thu, 2010-08-12, Paul Burba wrote: [...] ### Check diff --summarize for r2 svn diff -r1:2 --summarize AM nu AM J Should we really be describing the props for this added (not copied!) file as modified? It seems incorrect

svn api doc error

2010-08-16 Thread Shrinivasan T
Hi, I am trying to get the doc for the function svn_io_copy_perm I searched in the following URL. http://subversion.apache.org/docs/api/latest/search.php?query=svn_io_copy_perm It is not searching properly. Please fix this issue. If need any help, I will gladly do that on the guidance.

Re: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-08-16 Thread Julian Foad
Stephen Butler wrote: On Aug 16, 2010, at 10:40 , Julian Foad wrote: I would suggest that the output of svn diff --summarize should be consistent with the likes of svn add (no 'M'), svn status (no 'M') and svn update (I haven't checked this one). Fixed in r985735, while I was working on

Re: Returning revprops through commit info

2010-08-16 Thread Julian Foad
On Thu, 2010-08-12, Hyrum K. Wright wrote: Recently, we've updated the various client APIs which do commits to return commit info back through a callback[1], since they may be extended to perform multiple commits (see issue #1199 for instance). In doing so, I've had the opportunity to take a

Re: [PATCH] Tweak description of BASE_NODE table

2010-08-16 Thread Julian Foad
On Thu, 2010-08-12, Greg Stein wrote: Looks good! Committed. Now I'm trying to understand and document BASE_NODE (first) in more detail. Then I/we can tackle the other tables, and especially it should give me (and anyone else who may be struggling to follow) a starting point for understanding

[PATCH] was: [WIP] Enable passing copyfrom information for the diff code when dealing with repository diffs.

2010-08-16 Thread Daniel Näslund
I'm touching a lot of code here. Reviews would be much apprecieated. [[[ Make the diff editor able to receive copyfrom information. Involves passing down a 'send_copyfrom_args' to all RA implemtations. Note that this commit merely allows the copyfrom args to be passed to the client. They

Re: [PATCH] was: [WIP] Enable passing copyfrom information for the diff code when dealing with repository diffs.

2010-08-16 Thread Daniel Näslund
And this time with a patch attached. On Mon, Aug 16, 2010 at 03:09:49PM +0200, Daniel Näslund wrote: I'm touching a lot of code here. Reviews would be much apprecieated. [[[ Make the diff editor able to receive copyfrom information. Involves passing down a 'send_copyfrom_args' to all RA

Re: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-08-16 Thread Stephen Butler
On Aug 16, 2010, at 12:11 , Julian Foad wrote: Stephen Butler wrote: On Aug 16, 2010, at 10:40 , Julian Foad wrote: I would suggest that the output of svn diff --summarize should be consistent with the likes of svn add (no 'M'), svn status (no 'M') and svn update (I haven't checked this

Re: svn diff - Is this behaviour expected?

2010-08-16 Thread Vincent Lefevre
On 2010-08-15 23:30:09 -0700, Bert Huijben wrote: BTW, the local pristine version does correspond to some revision in the repository. It shouldn't be mixed up with the working copy version. What if the file is a copy? In my example, it wasn't a copy. But... In that case BASE refers to

RE: svn commit: r985735 - in /subversion/trunk/subversion: libsvn_client/repos_diff_summarize.c tests/cmdline/diff_tests.py

2010-08-16 Thread Bert Huijben
-Original Message- From: sbut...@apache.org [mailto:sbut...@apache.org] Sent: zondag 15 augustus 2010 12:05 To: comm...@subversion.apache.org Subject: svn commit: r985735 - in /subversion/trunk/subversion: libsvn_client/repos_diff_summarize.c tests/cmdline/diff_tests.py Author:

Re: svn commit: r985735 - in /subversion/trunk/subversion: libsvn_client/repos_diff_summarize.c tests/cmdline/diff_tests.py

2010-08-16 Thread Hyrum K. Wright
On Sun, Aug 15, 2010 at 2:04 PM, sbut...@apache.org wrote: Author: sbutler Date: Sun Aug 15 19:04:54 2010 New Revision: 985735 URL: http://svn.apache.org/viewvc?rev=985735view=rev Log: Eliminate 'AM' in 'svn diff --summarize' output, for items added with properties.  Print 'A ' instead,

Re: svn api doc error

2010-08-16 Thread Hyrum K. Wright
On Mon, Aug 16, 2010 at 5:07 AM, Shrinivasan T tshriniva...@gmail.com wrote: Hi, I am trying to get the doc for the function svn_io_copy_perm I searched in the following URL. http://subversion.apache.org/docs/api/latest/search.php?query=svn_io_copy_perm It is not searching properly.

Re: svn commit: r966822 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_tests.py

2010-08-16 Thread Hyrum K. Wright
On Mon, Aug 16, 2010 at 3:40 AM, Julian Foad julian.f...@wandisco.com wrote: On Thu, 2010-08-12, Paul Burba wrote: I'm working on that approach, but in the meantime I have a question about the behavior of svn diff --summarize when describing the addition of paths with properties. ### Let's

Re: [PATCH] was: [WIP] Enable passing copyfrom information for the diff code when dealing with repository diffs.

2010-08-16 Thread Daniel Shahaf
Daniel Näslund wrote on Mon, Aug 16, 2010 at 15:16:33 +0200: And this time with a patch attached. On Mon, Aug 16, 2010 at 03:09:49PM +0200, Daniel Näslund wrote: I'm touching a lot of code here. Reviews would be much apprecieated. [[[ Make the diff editor able to receive copyfrom

doxy...@subversion.a.o should be updated to 1.6.12 (was: svn api doc error)

2010-08-16 Thread Daniel Shahaf
Shrinivasan T wrote on Mon, Aug 16, 2010 at 15:37:22 +0530: http://subversion.apache.org/docs/api/latest/search.php?query=svn_io_copy_perm This latest is currently 1.6.6. Shouldn't it be 1.6.12?