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

Re: Patches pending review

From: Noorul Islam K M <noorul_at_collab.net>
Date: Mon, 14 Feb 2011 17:03:46 +0530

See below in line more information about the patches.

Noorul Islam K M <noorul_at_collab.net> writes:

> I have been working on some issues lately but I have not got many review
> comments for the same. I have listed them below so that it gets some
> attention and I can move forward.
>
> 1. I started sending patches for
> http://svn.haxx.se/dev/archive-2011-01/0210.shtml long back and one
> of them got committed in r1068977. Apart from that there are two
> more patches as a follow-up to r1068977
>
> http://svn.haxx.se/dev/archive-2011-02/0315.shtml

Subject of this thread is "[PATCH] svn command - ls - Multiple
targets". The patch does the following.

Make 'svn ls' continue processing targets after printing warning if one
or more of the targets is a non-existent URL or wc-entry. Also return a
non-zero error code and print an error message at the end in those
situations.

> http://svn.haxx.se/dev/archive-2011-01/0298.shtml
>

Subject of the above patch is "[PATCH] svn command - add - Multiple
targets". The patch does the following.

Make 'svn add' return a non-zero errorcode when
one or more targets fails. Also print an error message in those
situations.

> Similarly I will be submitting more once these patches are reviewed
> and committed.
>
> 2. Issue 3792
>
> http://svn.haxx.se/dev/archive-2011-02/0061.shtml
>
> I got some review comments from Daniel and incorporated them into the
> latest version of the patch. There is another issue 3787 which is
> kind of related to this one. I thought of working on that also once
> this one gets a +1.
>

The above patch fixes issue #3792. Make svn info to display information
for excluded items.

> 3. Issue 3690
>
> http://svn.haxx.se/dev/archive-2011-01/0414.shtml
>
> This is actually an enhancement and there were two approaches. One
> that I initially submitted and another one suggested by Hyrum. This
> thread has three to four patches which are pending review. May be it
> is not getting reviewed because it is not yet finalized how are are
> we going to proceed.
>

Above thread contains patches for issue 3690 which adds an option
"--ignore-properties" to log command so that user can ignore revisions
which has only property changes.

Thanks and Regards
Noorul
Received on 2011-02-14 12:36:00 CET

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