On Wed, Nov 9, 2011 at 10:46 AM, Philip Martin
philip.mar...@wandisco.com wrote:
Daniel Näslund dan...@dannas.name writes:
Index: subversion/libsvn_client/patch.c
===
--- subversion/libsvn_client/patch.c (revision 1199347
On Wed, Nov 9, 2011 at 12:41 PM, Daniel Näslund dan...@dannas.name wrote:
On Wed, Nov 9, 2011 at 10:46 AM, Philip Martin
philip.mar...@wandisco.com wrote:
Daniel Näslund dan...@dannas.name writes:
What is not clear to me is why skipped targets get added to targets_info
in the first place
Hi,
It's been a year since my last commit and I've forgotten about the
internals of the patch code. Sending this patch for review so that I
don't mess something up.
[[[
Fix issue #4049 - SEGV on patch that deletes and skips.
When we do the sanity check of the supplied patch file we set
On Sun, Mar 26, 2000 at 10:27:38AM +0200, Daniel Shahaf wrote:
You can declare VALID_HEADER_FILE in the do{} loop's scope. (I spotted
this because the double initialization stood out.)
dan...@apache.org wrote on Tue, Mar 22, 2011 at 11:19:44 -:
Author: dannas
Date: Tue Mar 22
On Fri, Nov 12, 2010 at 03:14:13AM +0200, Daniel Shahaf wrote:
Stefan Sperling wrote on Fri, Nov 12, 2010 at 01:07:53 +0100:
On Thu, Nov 11, 2010 at 09:21:26PM -, dan...@apache.org wrote:
Add support for handling symlinks in 'svn patch'.
If you haven't done so, can you close the
On Tue, Sep 28, 2010 at 03:45:33PM +0200, Daniel Shahaf wrote:
Daniel Näslund wrote on Mon, Sep 27, 2010 at 22:09:26 +0200:
* What is wrong with the way I handle the error? I hit err_abort() when
the program terminates. (I'm expecting the answer to hurt a bit - this
is surely something
On Tue, Sep 28, 2010 at 07:18:39PM +0100, Philip Martin wrote:
Daniel Näslund dan...@longitudo.com writes:
On Tue, Sep 28, 2010 at 03:45:33PM +0200, Daniel Shahaf wrote:
Daniel Näslund wrote on Mon, Sep 27, 2010 at 22:09:26 +0200:
err_abort() is called when an error object hadn't been
Hi!
Questions
-
* Is it sane to add a svn_wc_notify_failed_prop_patch action for this
special case? We're starting to have a lot of actions.
* What is wrong with the way I handle the error? I hit err_abort() when
the program terminates. (I'm expecting the answer to hurt a bit -
On Wed, Sep 01, 2010 at 06:37:08PM +0100, Julian Foad wrote:
Daniel Näslund wrote:
The question is: Do we want 'svn patch to be able to add empty files.
If add an empty file is a well defined operation in Git-diff syntax,
then yes, it would be good to support it. As well as delete
On Fri, Sep 03, 2010 at 12:18:37PM +0200, Branko Čibej wrote:
On 02.09.2010 10:50, Branko Čibej wrote:
Hmm, this is interesting. :) Git faithfully (blindly?) interprets Unix
permission bits, whiles SVN faithfully (blindly?) interprets the
contents of special files ... I wonder if svn patch
On Thu, Sep 02, 2010 at 11:27:07AM +0300, Daniel Shahaf wrote:
Daniel Näslund wrote on Thu, Sep 02, 2010 at 07:13:00 +0200:
On Wed, Sep 01, 2010 at 06:37:08PM +0100, Julian Foad wrote:
This may be off topic, but I'm wondering whether Git has defined such
operations on directories fully
On Thu, Sep 02, 2010 at 11:14:27AM +0300, Daniel Shahaf wrote:
(Thanks for the examples. I suppose next time I should try to run
'touch foo; $svn add foo; $svn diff --git' myself...)
Daniel Näslund wrote on Thu, Sep 02, 2010 at 07:05:41 +0200:
On Wed, Sep 01, 2010 at 10:54:08PM +0300
Hi!
(This started out as me trying to apply added paths using the information
from a patch file in the git diff format. The only that I could come up
with where an add could not be detected by just looking at the regular
unidiff headers was adding an empty file (it has no unidiff headers).
If
On Wed, Sep 01, 2010 at 10:54:08PM +0300, Daniel Shahaf wrote:
Daniel Näslund wrote on Wed, Sep 01, 2010 at 11:28:51 +0200:
(This started out as me trying to apply added paths using the information
from a patch file in the git diff format. The only that I could come up
with where an add
Daniel,
Sorry for the delayed response. I've committed the patch with your
suggestions in r990790. Below is just ACK's for those changes.
On Mon, Aug 16, 2010 at 10:59:03PM +0300, Daniel Shahaf wrote:
Daniel Näslund wrote on Mon, Aug 16, 2010 at 15:16:33 +0200:
And this time with a patch
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
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
Hi!
Why doesn't svn_ra_do_diff3() request that the repository sends copyfrom
args? The best explanation I've found is this commit message:
r866577 | sussman | 2007-09-10 06:56:55 +0200 (mån, 10 sep 2007) | 87 lines
Send
On Sat, Aug 14, 2010 at 03:46:13PM -, stef...@apache.org wrote:
Author: stefan2
Date: Sat Aug 14 15:46:13 2010
New Revision: 985514
URL: http://svn.apache.org/viewvc?rev=985514view=rev
Log:
Extend the stream API by three functions:
svn_stream_move_mark() to move an existing mark by
Hi!
The soft deadline for GSoC 2010 is today. Not much time left then...
What has been accomplished:
* A parseable diff format for properties.
* svn patch can apply properties
* We have Python tests for applying properties.
* svn diff can create 'git diffs when passed a special flag.
* C-unit
On Thu, Aug 05, 2010 at 11:52:14AM -0400, Paul Burba wrote:
On Thu, Aug 5, 2010 at 8:41 AM, dan...@apache.org wrote:
Author: dannas
Date: Thu Aug 5 12:41:51 2010
New Revision: 982582
Modified:
subversion/trunk/subversion/tests/libsvn_diff/parse-diff-test.c
@@ -834,8 +834,9 @@
Hi!
Why doesn't svn_ra_do_diff3() provide a reporter that supplies copyfrom
info? svn_ra_do_update2() does, so why doesn't the diff func?
Is it based on a notion of diffs as beeing text compares and thus not
interrested in tree changes such as copies/moves?
Is there something in the FS layer
On Fri, Jul 30, 2010 at 01:28:39PM -, phi...@apache.org wrote:
Author: philip
Date: Fri Jul 30 13:28:39 2010
New Revision: 980784
Modified: subversion/trunk/subversion/libsvn_wc/adm_files.c
URL:
Hi!
resolve_target_path() makes a few assumptions about the target for a
patch file beeing a wc_dir. Currently we can only invoke svn patch like
this:
svn patch PATCH_FILE WC_DIR_PATH
Is this the intended behavoiour?
[[[
Make it clearer that the patch code can only use base paths that are
On Thu, Jul 29, 2010 at 02:54:29PM +0200, Stefan Sperling wrote:
On Thu, Jul 29, 2010 at 12:43:56PM -, dan...@apache.org wrote:
Author: dannas
Date: Thu Jul 29 12:43:56 2010
New Revision: 980427
URL: http://svn.apache.org/viewvc?rev=980427view=rev
Log:
Make 'svn patch' do
Hi!
Property diffs
===
* We can create parseable prop patches
* We can parse property patches
* We can match property patches
* We can write patched properties to tmp files
* We can install those prop changes to files and dirs (not the wc-root dir yet)
Todo
--
* Decide on how we
Hi Stefan!
A few questions and a WIP patch for making the patch code deal with
property targets being dirs.
* How should we deal with SVN_ERR_ILLEGAL_TARGET? svn_wc_prop_set()
throws one of those if we for instance try to set svn:executable on a
dir or svn:ignore on a file. Should we do some
Hi!
Posting this one here so I don't forget about it.
The patch code should not change the wc when run with the --dry-run
option but it will if we're adding a file with a parent scheduled for
deletion.
The patch code will add a file with a parent scheduled for deletion to
the filesystem but not
On Mon, Jul 26, 2010 at 01:04:26PM +0200, Daniel Näslund wrote:
Hi!
Posting this one here so I don't forget about it.
The patch code should not change the wc when run with the --dry-run
option but it will if we're adding a file with a parent scheduled for
deletion.
The patch code
On Mon, Jul 26, 2010 at 01:07:14PM +0200, Stefan Sperling wrote:
On Mon, Jul 26, 2010 at 09:59:01AM +0200, Daniel Näslund wrote:
Index: subversion/libsvn_client/patch.c
===
--- subversion/libsvn_client/patch.c(revision
Hi Stefan!
Do you think the attached patch looks reasonable, e.g. it does not
interfer with the overall design of 'svn patch'? It's not fun to review
a patch that touches on so many parts. Sorry 'bout that. Find below some
specific questions.
The patch code consist of four steps:
1) match text
On Fri, Jul 09, 2010 at 12:36:35PM +0100, Julian Foad wrote:
On Fri, 2010-07-09 at 09:52 +0200, Daniel Näslund wrote:
Hi!
The diff callbacks for handling text have revision arguments. The prop
callbacks don't.
I'm creating diff headers for the cases when we only have property
Hi!
The diff callbacks for handling text have revision arguments. The prop
callbacks don't.
I'm creating diff headers for the cases when we only have property
changes. When creating those diff headers I need revisions. Since the
prop diff callback doesn't have revision arguments I'm using the
Hi!
Just trying to sort out what work lays ahead.
Things todo for property diffs
---
1) Make svn diff print diff headers even if we have just property
changes.
2) Make svn patch able to match (compare streams) property hunks
3) Make svn patch able to apply (write
On Sat, Jun 26, 2010 at 04:47:46PM -0400, Greg Stein wrote:
On Sat, Jun 26, 2010 at 13:56, dan...@apache.org wrote:
Author: dannas
Date: Sat Jun 26 17:56:06 2010
New Revision: 958260
URL: http://svn.apache.org/viewvc?rev=958260view=rev
Log:
With SVN_EXPERIMENTAL_PATCH defined, only
On Mon, Jun 28, 2010 at 02:38:26PM +0200, Daniel Näslund wrote:
Since r958537, there is a diff extension switch to enable git diffs:
svn diff -x --git-diff [-g]
In hindsight, I'm not sure that was the best thing to do. I read this
part from svn help diff:
[[[
-x --extensions ARG
Hi!
Some questions about how the workqueues... work.
Have I got it right?
How are files installed during an update?
We acquire a write lock
We crawl the wc
We tell the repos about our state
The repo runs an editor
open_root(): We set status to
Hi!
Is 'svn switch does not update keywords' a problem that needs solving?
Is the performance of this solution acceptable (traverse the entire wc
one more time after the editor drive)?
Alternatives?
* Custom db-query that only returns the paths with matching property set
on them. Could be
On Thu, Jun 24, 2010 at 09:20:22AM +0100, Philip Martin wrote:
Daniel Näslund dan...@longitudo.com writes:
How are files installed during an update?
We acquire a write lock
We crawl the wc
We tell the repos about our state
The repo runs
On Fri, Jun 25, 2010 at 02:03:17AM +, Paul Smedley wrote:
[[[
Fix paths command prompts for OS/2 by re-using WIN32 #ifdef's
* subversion/libsvn_subr/dirent_uri.c
* subversion/libsvn_subr/io.c
* subversion/libsvn_subr/prompt.c
* subversion/libsvn_ra_local/split_url.c
*
On Mon, Jun 21, 2010 at 11:12:43AM +0200, Stefan Sperling wrote:
On Sun, Jun 20, 2010 at 09:53:29PM +0200, Daniel Näslund wrote:
Hi stsp!
Attaching a patch showing a suggestion on how to split up
patch_target_t. Maybe you have some insights on this, before I rush out
and start grooking
Hi stsp!
Attaching a patch showing a suggestion on how to split up
patch_target_t. Maybe you have some insights on this, before I rush out
and start grooking around and making changes to your design.
What 'svn patch' does
--
1) Match hunks
2) Apply hunks to a stream backed up
Hi!
The goal is to be able to create, parse and apply diffs in the 'git
unidiff' format. The original thought was to add support for property
diffs when the basic parts of the 'git unidiff' part was in place, but
Stefan suggested that we could easily add property diffs even without
the git parts.
Hi!
I need to have a diff header even if there's only property changes for
the sake of parsing.
And I want only one diff header per file. Makes it easier to apply the
patch. For each found diff header we create one svn_patch_t object.
Simple and clean.
What to do? I could add a flag to the
On Mon, Jun 07, 2010 at 03:33:16PM +0200, Stefan Sperling wrote:
On Mon, Jun 07, 2010 at 12:43:19PM -, dan...@apache.org wrote:
+static svn_error_t *
+print_git_diff_header_copied(svn_stream_t *os, const char
*header_encoding,
+ const char *path, const
On Mon, Jun 07, 2010 at 03:41:27PM +0200, Stefan Sperling wrote:
On Mon, Jun 07, 2010 at 12:43:19PM -, dan...@apache.org wrote:
Author: dannas
Date: Mon Jun 7 12:43:19 2010
New Revision: 952205
URL: http://svn.apache.org/viewvc?rev=952205view=rev
Log:
Use several smaller
On Sun, Jun 06, 2010 at 02:49:16PM -, dan...@apache.org wrote:
Author: dannas
Date: Sun Jun 6 14:49:15 2010
New Revision: 951871
URL: http://svn.apache.org/viewvc?rev=951871view=rev
Log:
First small step towards using the 'git unidiff' format for 'svn diff'.
The parts that writes
On Sun, May 30, 2010 at 11:16:55AM +0200, Stefan Sperling wrote:
On Sat, May 29, 2010 at 02:34:06PM +0200, Daniel Näslund wrote:
Problem 1
==
We're processing the delete-half instead of the add-half for a
copy/move. At present, copy/move add-half is not shown by 'svn diff
Hi!
The Google Summer of Code period has started. Posting about what I'm
currently working on as part of implementing the 'git unidiff' format
for wc-wc diffs. Perhaps someone sees an obvious solution to any of the
three problems I present below.
The callchain:
===
do_diff()
On Wed, May 26, 2010 at 03:57:52AM -, hwri...@apache.org wrote:
Author: hwright
Date: Wed May 26 03:57:52 2010
New Revision: 948304
URL: http://svn.apache.org/viewvc?rev=948304view=rev
Log:
Remove a couple of goto statements in favor of some refactoring and a helper
function.
Hi Stefan Sperling
Just posting to tell I'm looking into this.
On Sat, May 22, 2010 at 12:58:42AM +0200, Stefan Sperling wrote:
On Fri, May 21, 2010 at 10:27:42PM +0200, Stefan Küng wrote:
* ShellCache.h
- this file results in three applied and one rejected hunk
the result as it
Hi!
There's a lot of parameteter tracking in this patch. Basically it's all
about passing down the url arguments to assemble_status().
The goal is that we should be able to remove the entry and parent_entry
fields in a follow-up and be able to use the parent_relpath when we want
to detect
On Thu, May 13, 2010 at 10:35:36PM +0200, Bert Huijben wrote:
Log:
Don't use svn_wc_entry for fetching an url.
* subversion/libsvn_wc/status.c
(assemble_status): Fetch the url with a node func. Use
svn_wc__check_wc_root() for determining if a path is switched.
[...]
+
Hi!
Greg [1] identified the following items in the beginning of February.
* remove entry_t usage within libsvn_wc/client
* remove access_t usage withing libsvn_wc/client
* move props into wc.db
* move to single/root wc.db
* switch to new pristine file mgmt
I have the feeling, we've made some
On Thu, May 06, 2010 at 04:27:27PM -, hwri...@apache.org wrote:
Author: hwright
Date: Thu May 6 16:27:27 2010
New Revision: 941793
URL: http://svn.apache.org/viewvc?rev=941793view=rev
Log:
* subversion/libsvn_wc/util.c
(svn_wc__status2_from_3): Fix a pointer error, and one of the
On Mon, May 03, 2010 at 08:41:14PM -0400, Greg Stein wrote:
On Mon, May 3, 2010 at 18:01, dan...@apache.org wrote:
...
+++ subversion/trunk/subversion/libsvn_wc/util.c Mon May 3 22:01:53 2010
...
@@ -565,6 +567,12 @@ svn_wc__status2_from_3(svn_wc_status2_t
Hi!
First, I've been accepted as a GSoC student for the summer of 2010. I'm
really excited and look forward to a summer of coding.
I'm supposed to implement the git unidiff format for 'svn diff' and
'svn patch' and I'll start with the diff side. The git unidiff format
can represent tree changes
Hi!
Since I can't test it, and know too little of libsvn_wc, I ask if
any of you guys think it looks ok?
[[[
Implmenent converter function for converting svn_wc_status3_t to
svn_wc_status2_t.
* subversion/include/private/svn_wc_private.h
(svn_wc__status2_from_3): Return svn_error_t * instead
On Thu, Apr 22, 2010 at 05:07:56PM -0400, Greg Stein wrote:
On Thu, Apr 22, 2010 at 15:48, Daniel Näslund dan...@longitudo.com wrote:
read_info() returns changed_* information for copied nodes. A copied
node has a corresponding node in the repository, and the changed_*
information
On Thu, Apr 22, 2010 at 02:45:08PM -0400, Greg Stein wrote:
On Thu, Apr 22, 2010 at 10:05, Daniel Näslund dan...@longitudo.com wrote:
[[[
As part of WC-NG, replace entry calls for revisions from
svn_wc_status2_t related code.
The entries code set the revision of newly added nodes to 0
On Mon, Apr 05, 2010 at 01:41:03PM +0200, Stefan Sperling wrote:
On Mon, Apr 05, 2010 at 10:02:56AM +0200, Daniel Näslund wrote:
Git unidiff format extension to 'svn patch' and 'svn diff'
===
[..]
2) Add the ability to track renames
On Thu, Apr 22, 2010 at 01:28:11AM +0200, Neels J Hofmeyr wrote:
Daniel Näslund wrote:
On Fri, Apr 16, 2010 at 11:09:06PM -, ne...@apache.org wrote:
==
--- subversion/trunk/subversion/libsvn_wc/node.c (original
On Fri, Apr 16, 2010 at 11:09:06PM -, ne...@apache.org wrote:
[...]
==
--- subversion/trunk/subversion/libsvn_wc/node.c (original)
+++ subversion/trunk/subversion/libsvn_wc/node.c Fri Apr 16 23:09:06 2010
@@
Hi!
#3610 [1] is about ignoring white space changes to be able to apply
patches that have been mangled, i.e. a mailing software converting tabs
to spaces, removing trailing white spaces or leading white spaces.
It means that if a certain option is given, we will match and apply
hunks if the only
On Wed, Apr 14, 2010 at 05:35:36PM +0200, Stefan Sperling wrote:
On Wed, Apr 14, 2010 at 04:21:56PM +0200, Daniel Näslund wrote:
Hi!
#3610 [1] is about ignoring white space changes to be able to apply
patches that have been mangled, i.e. a mailing software converting tabs
to spaces
On Wed, Apr 14, 2010 at 08:18:06PM +0200, Alan Barrett wrote:
On Wed, 14 Apr 2010, Stefan Sperling wrote:
Yes. Just use whatever comes from the patch, including context lines.
This is consistent with the current behaviour. I think we should avoid
special cases where this rule is currently
On Mon, Apr 12, 2010 at 08:58:49AM -0400, Greg Stein wrote:
On Fri, Apr 9, 2010 at 16:16, Greg Stein gst...@gmail.com wrote:
On Fri, Apr 9, 2010 at 16:07, dan...@apache.org wrote:
...
...
+++ subversion/trunk/subversion/libsvn_wc/deprecated.c Fri Apr 9 20:07:51
2010
...
@@ -2515,13
Hi!
This is a _big_ patch but it does very little. I've just replaced every
occurence of svn_wc_status2_t with the new svn_wc_status3_t. It's
needed, in the cause of removing the entry field which I intend to do in
a follow-up. There's a WIP on the list, that handles most of the chores of the
Hi!
I'm supposed to send this proposal to the Google Summer of Code
machinery and let it be forwarded to the interrested mentor of the
Subversion community, in this case Stefan. In the interrest of openess
I'm posting it here before sending it off to Google later today. Maybe
someone has
Committed in r930662 on my branch svn-patch-improvements.
On Wed, Mar 03, 2010 at 04:25:07PM +0100, Daniel Näslund wrote:
Hi!
Here's my revised patch. I've commented on the changes inline.
One strange thing: If you compile this you'll get a compiler warning
about this variable not beeing
Hi Julian!
On Wed, Mar 31, 2010 at 10:19:10AM +0100, Julian Foad wrote:
On Tue, 2010-03-30, Daniel Näslund wrote:
Ping! This patch has not been revieved
[...]
[[[
Follow-up to r922176. Fix that tree changes were not considered when
determining if the wc has modifications
On Tue, Mar 30, 2010 at 11:04:13PM +0200, Stefan Sperling wrote:
On Tue, Mar 30, 2010 at 10:14:42PM +0200, Daniel Näslund wrote:
Ping! This patch has not been reviewed!
On Sun, Mar 14, 2010 at 09:38:15PM +0100, Daniel Näslund wrote:
Hi!
The 1.6 svnversion message was 'path
Mispelled Uncommitted with one 't'. This patch corrects that.
On Fri, Apr 02, 2010 at 10:36:53AM +0200, Daniel Näslund wrote:
On Tue, Mar 30, 2010 at 11:04:13PM +0200, Stefan Sperling wrote:
On Tue, Mar 30, 2010 at 10:14:42PM +0200, Daniel Näslund wrote:
Ping! This patch has not been
We're running out of patch versions here ...
gstein pointed out that I should use SVN_IS_VALID_REVNUM(). Fixed!
On Fri, Apr 02, 2010 at 10:59:15AM +0200, Daniel Näslund wrote:
Mispelled Uncommitted with one 't'. This patch corrects that.
On Fri, Apr 02, 2010 at 10:36:53AM +0200, Daniel
On Tue, Mar 30, 2010 at 02:15:32PM +0200, neels wrote:
Hi all,
every now and then I would find it useful if there was a public log of
#svn-dev (on FreeNode IRC).
I believe it was previously decided not to log #svn-dev because
discussions about potential new committers took place there.
Hi!
This patch passes make check with modifications to some tests to take
into account some changed behaviors. I've tried to justificate those
changes in the log message below.
Is this really right?
==
When scanning for a deletion I never use the base_del_abspath parameter
of
Ping! This patch has not been revieved
On Fri, Mar 12, 2010 at 10:52:55PM +0100, Daniel Näslund wrote:
On Fri, Mar 12, 2010 at 05:43:18AM -0500, Greg Stein wrote:
On Fri, Mar 12, 2010 at 03:21, dan...@apache.org wrote:
...
+++ subversion/trunk/subversion/libsvn_wc/revision_status.c Fri
Ping! This patch has not been reviewed!
On Sun, Mar 14, 2010 at 09:38:15PM +0100, Daniel Näslund wrote:
Hi!
The 1.6 svnversion message was 'path' not versioned, and not exported.
But on trunk more than one message has been changed. My first thought
was that we should be backward compat
On Fri, Mar 19, 2010 at 02:33:33AM +0100, Neels J Hofmeyr wrote:
C. Michael Pilato wrote:
Stefan Sperling wrote:
On Tue, Mar 16, 2010 at 09:01:28PM +0100, Daniel Näslund wrote:
Hi!
When trying to replace entries in the status code I got a couple of test
failures saying
On Fri, Mar 12, 2010 at 10:00:48AM +0100, Stefan Sperling wrote:
On Tue, Mar 09, 2010 at 03:01:36PM +, Justin Erenkrantz wrote:
On Tue, Mar 9, 2010 at 9:59 AM, Stefan Sperling s...@elego.de wrote:
Does this mean that we don't have to do anything special to apply
for Gsoc, but just
On Fri, Mar 12, 2010 at 07:42:53PM -0600, Hyrum K. Wright wrote:
On Mar 12, 2010, at 3:52 PM, Daniel Näslund wrote:
- if (status-entry-depth != svn_depth_exclude)
+ /* Added files have a revision of no interest */
+ if (revision != SVN_INVALID_REVNUM)
{
- sb-result-switched
On Thu, Mar 11, 2010 at 05:44:18PM -0500, Greg Stein wrote:
[[[
As part of WC-NG, remove some uses of svn_wc_entry_t.
* subversion/include/svn_wc.h
(svn_wc_revision_status2): Add note about us returning
SVN_ERR_WC_PATH_NOT_FOUND.
* subversion/libsvn_wc/revision_status.c
On Tue, Mar 09, 2010 at 02:19:52PM +, Julian Foad wrote:
On Sun, 2010-03-07, dan...@apache.org wrote:
Author: dannas
Date: Sun Mar 7 21:21:58 2010
New Revision: 920118
URL: http://svn.apache.org/viewvc?rev=920118view=rev
Log:
Fix issue #3483 - extend svn_client_upgrade() to
On Fri, Mar 12, 2010 at 06:15:27PM +, Philip Martin wrote:
Daniel Näslund dan...@longitudo.com writes:
Can this be committed?
[[[
Follow up to r920118.
* subversion/libsvn_client/cleanup.c
(svn_client_upgrade): Use svn__apr_hash_index_{key,val} to avoid
casts. Use
On Fri, Mar 12, 2010 at 05:43:18AM -0500, Greg Stein wrote:
On Fri, Mar 12, 2010 at 03:21, dan...@apache.org wrote:
...
+++ subversion/trunk/subversion/libsvn_wc/revision_status.c Fri Mar 12
08:21:45 2010
...
{
- struct status_baton *sb = baton;
+ struct walk_baton *wb = baton;
On Thu, Mar 11, 2010 at 03:05:23AM -0500, Greg Stein wrote:
On Wed, Mar 10, 2010 at 05:51, Daniel Näslund dan...@longitudo.com wrote:
[...]
I've removed the comments in the end about sparse_checkout not detecting
all cases. The walker reaches all nodes beneath local_abspath and if we
Hi!
Here's my new patch.
On Tue, Mar 09, 2010 at 12:52:23AM +0100, Bert Huijben wrote:
Several points:
* use svn_wc__internal_text_modified_p(); then you won't need wc_ctx
(just the db)
Done!
* read_info can return NULL for the repos_* values
I'm checking that the values are null
Hi!
I have run the merge_tests on this patch. When grepping I only found
this code to be used there. If this looks ok, I'll run a make check on
it.
[[[
As part of WC-NG, remove some uses of svn_wc_entry_t.
* subversion/libsvn_wc/revision_status.c
(status_baton): Rename this...
(walk_baton):
used in
externals_tests.py.
Patch by: Daniel Näslund daniel{_AT_}longitudo.com
]]]
Index: subversion/tests/cmdline/upgrade_tests.py
===
--- subversion/tests/cmdline/upgrade_tests.py (revision 919571)
+++ subversion/tests/cmdline
Hi!
I have my own branch, and I was told that I could just commit there. But
it's _scary_. I'll ask for permission first.
[[[
Fix bug with 'svn patch' not recognizing diff headers when parsing
patches without the 'Index' line and the '==' line. The old code
just assumed that the first line
someone explain why a variable that isn't used must be declared in order
for my code to work. What is the simple logical explanation?
On Mon, Mar 01, 2010 at 04:10:09PM +0100, Stefan Sperling wrote:
On Sat, Feb 27, 2010 at 09:09:52PM +0100, Daniel Näslund wrote:
Index: subversion/libsvn_client
On Tue, Feb 23, 2010 at 08:07:05AM -0600, Hyrum K. Wright wrote:
On Sun, Feb 21, 2010 at 5:11 AM, Daniel Näslunddan...@longitudo.com wrote:
Google Summer of Code is on for 2010 [1]. Is there any interest in
participating this year? If so, I've found that the mentoring
application deadline is
no associated patch.
Patch by: Daniel Näslund daniel{_AT_}longitudo.com
Review by: julianf, stsp, neels
]]]
What target fields must be set for the parent dir target? I've just set
deleted, abs_path and deleted. May be a bit hard to maintain. Would be
better with a central initialize_basic_target() func
On Tue, Feb 02, 2010 at 09:11:52PM +0100, Stefan Sperling wrote:
On Tue, Feb 02, 2010 at 08:48:43PM +0100, Daniel Näslund wrote:
Hi Stefan!
In match_hunk() we try to match lines from the context of the patch with
lines in the target. Earlier, in init_patch_target() we detect the eol
On Tue, Feb 23, 2010 at 09:27:34PM +0100, Stefan Sperling wrote:
On Tue, Feb 23, 2010 at 08:57:26PM +0100, Daniel Näslund wrote:
On Tue, Feb 02, 2010 at 09:11:52PM +0100, Stefan Sperling wrote:
On Tue, Feb 02, 2010 at 08:48:43PM +0100, Daniel Näslund wrote:
Hi Stefan
On Sun, Feb 21, 2010 at 07:28:13AM -0800, Justin Erenkrantz wrote:
On Sun, Feb 21, 2010 at 5:11 AM, Daniel Näslund dan...@longitudo.com wrote:
Hi!
Google Summer of Code is on for 2010 [1]. Is there any interest in
participating this year? If so, I've found that the mentoring
application
Hi!
Google Summer of Code is on for 2010 [1]. Is there any interest in
participating this year? If so, I've found that the mentoring
application deadline is 12 march. [2]
Since we're part of ASF, perhaps an application should go
through them? [3]
I've tried locating the GSoC ideas pages for
Ping!
This patch has not been reviewed!
On Tue, Dec 29, 2009 at 07:44:49PM +0100, Daniel Näslund wrote:
Hi!
[[[
Fix issue #3483 - extend svn_client_upgrade() to include externals. I've
done the externals upgrading after wc upgrade is finished. In that way
no errors in the externals
Hi Stefan!
This patch is growing and growing and I'm worried that I'm heading off
in the wrong direction. Perhaps you can skim through the patch and tell
me if this is the right approach. You've said so earlier but all these
lines of code makes me nervous. I like small fixes, and this is starting
, Neels J Hofmeyr wrote:
Daniel Näslund wrote:
Design spec for tree conflict resolution in the commandline client
~~~
The hard part is figuring out what state the wc is in during the
different user cases.
Actually, wc-ng
1 - 100 of 143 matches
Mail list logo