G vijay{_AT_}collab.net
Index: CHANGES
===
--- CHANGES (revision 1455002)
+++ CHANGES (working copy)
@@ -12,6 +12,7 @@
- Minor new features and improvements:
* new 'svnadmin load --revision' load filtering support
.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Index: subversion/svndumpfilter/svndumpfilter.c
===
--- subversion/svndumpfilter/svndumpfilter.c(revision 1451099)
+++ subversion/svndumpfilter/svndumpfilter.c(working copy
/exclude' (issue #3681).
Patch by: Vijayaguru G vijay{_AT_}collab.net
Index: CHANGES
===
--- CHANGES (revision 1451469)
+++ CHANGES (working copy)
@@ -33,6 +33,7 @@
* new 'svn --version --verbose' option shows runtime
/svndumpfilter_tests.py
(drop_all_empty_revisions): New test.
(test_list): Add reference to the new test.
* subversion/tests/cmdline/svndumpfilter_tests_data/empty_revisions.dump:
New dump file for the test.
Patch by: Vijayaguru G vijay{_AT_}collab.net
used together.
* subversion/tests/cmdline/svndumpfilter_tests.py
(drop_all_empty_revisions): New test.
(test_list): Add reference to the new test.
* subversion/tests/cmdline/svndumpfilter_tests_data/empty_revisions.dump:
New dump file for the test.
Patch by: Vijayaguru G vijay{_AT_}collab.net
On Tuesday 26 February 2013 07:47 PM, Philip Martin wrote:
vijay vi...@collab.net writes:
/* write out the revision */
/* Revision is written out in the following cases:
- 1. No --drop-empty-revs has been supplied.
- 2. --drop-empty-revs has been supplied,
- but revision
On Thursday 20 December 2012 07:52 PM, vijay wrote:
I note that 'svnadmin create r; svnrdump dump file://$PWD/r/foo' does
not error --- maybe your patch fixes that too?
The attached patch fixes this issue. This patch is different from the
previous versions.
We have to error out immediately
is not present in HEAD.
* subversion/svnrdump/svnrdump.c
(main): Check that the dump target actually exists. If not, error out
immediately.
* subversion/tests/cmdline/svnrdump_tests.py
(dump_non_existent_url): New test.
(test_list): Add reference to new test.
Patch by: Vijayaguru G vijay{_AT_
On Monday 17 December 2012 09:34 PM, Philip Martin wrote:
vijay vi...@collab.net writes:
The function svn_ra_serf__replay_range() doesn't return any error
while running 'svnrdump dump -rN:M URL-deleted-in-HEAD'[Issue #4100].
It behaves as if the dump completed successfully.
* subversion
found while running the serf loop.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Index: subversion/libsvn_ra_serf/replay.c
===
--- subversion/libsvn_ra_serf/replay.c (revision 1422748)
+++ subversion/libsvn_ra_serf/replay.c (working
On Saturday 15 December 2012 02:33 PM, cmpil...@apache.org wrote:
Author: cmpilato
Date: Sat Dec 15 09:03:23 2012
New Revision: 1422206
URL: http://svn.apache.org/viewvc?rev=1422206view=rev
Log:
Fix issue #4104 ('svnrdump dump -rN:M URL/subpath' strips subpath
in first rev only)
Issue #4101,
.
* subversion/libsvn_ra_serf/replay.c
(svn_ra_serf__replay_range): As above.
Patch by: Vijayaguru G vijay{_AT_}collab.net
= replay_ctx;
handler-conn = session-conns[0];
* subversion/libsvn_ra_serf/replay.c
(svn_ra_serf__replay, svn_ra_serf__replay_range): Replace the session url
string with the request path portion of the url.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Thanks, Philip and Julian.
I will explain things clearly in future patches.
Thanks Regards,
Vijayaguru
On Friday 07 December 2012 10:02 PM, Julian Foad wrote:
Philip Martin wrote:
Philip Martin philip.mar...@wandisco.com writes:
The logging is really just a side-effect. We setup this
'.
* subversion/include/svn_client.h:
(svn_client_list_func2_t): Rename the argument to scratch_pool.
* subversion/svn/list-cmd.c
(print_dirent, print_dirent_xml): Rename the argument to scratch_pool.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Thanks Julian.
This updated patch has all the changes.
I have removed some redundant portion of code in print_dirent() and
print_dirent_xml() in svn/list-cmd.c.
Thanks Regards,
Vijayaguru
On Tuesday 27 November 2012 03:19 AM, Julian Foad wrote:
vijay vi...@collab.net wrote:
On Tuesday
On Tuesday 27 November 2012 11:52 PM, Julian Foad wrote:
vijay vi...@collab.net wrote:
This updated patch has all the changes.
I have removed some redundant portion of code in print_dirent() and
print_dirent_xml() in svn/list-cmd.c.
Thanks Vijay. That looks great. I've committed
/externals_tests.py
(include_externals): Rename it to 'commit_include_externals'.
(list_include_externals): New test.
(test_list): Add a reference to the new test. Rename 'include_externals' to
'commit_include_externals'.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Review by: stsp
(print_dirent_xml): Remove the unused code.
Patch by: Vijayaguru G vijay{_AT_}collab.net
On Thursday 15 November 2012 08:10 PM, C. Michael Pilato wrote:
On 11/15/2012 03:47 AM, Bert Huijben wrote:
Can't you just add the new argument to every call to list_func() that
applies to an external?
There is no separate list_func() call to list external items. We are
just calling
On Friday 16 November 2012 04:41 PM, Neels J Hofmeyr wrote:
Hi Vijay,
I gave that patch a spin (after all I was the one who brought
--include-externals upon us), and found stuff. Using current trunk + your
patch v3.
I was looking at your code 'commit --include-externals' before working
On Tuesday 13 November 2012 06:45 PM, Bert Huijben wrote:
-Original Message-
From: vijay [mailto:vi...@collab.net]
Sent: dinsdag 13 november 2012 14:06
To: Subversion Development
Subject: Re: [PATCH] Implement '--include-externals' option to 'svn list'
On Tuesday 13 November 2012 03
On Tuesday 13 November 2012 03:02 AM, Stefan Sperling wrote:
Attached the updated patch and log message.
+ /* Notify that we're about to handle an external. */
+ SVN_ERR(list_func(baton, NULL, NULL, NULL, NULL,
+externals_parent_url,
+
On Thursday 08 November 2012 04:10 PM, vijay wrote:
On Wednesday 07 November 2012 08:19 PM, Stefan Sperling wrote:
On Wed, Nov 07, 2012 at 07:54:33PM +0530, vijay wrote:
Thanks stefan for your detailed review.
I will keep in mind all your review comments. I will correct my
mistakes in future
On Wednesday 07 November 2012 08:19 PM, Stefan Sperling wrote:
On Wed, Nov 07, 2012 at 07:54:33PM +0530, vijay wrote:
Thanks stefan for your detailed review.
I will keep in mind all your review comments. I will correct my
mistakes in future patches.
Attached the updated patch and log message
On Wednesday 07 November 2012 08:57 PM, Stefan Fuhrmann wrote:
On Wed, Nov 7, 2012 at 3:02 PM, vijay vi...@collab.net
mailto:vi...@collab.net wrote:
On Tuesday 06 November 2012 08:09 PM, Stefan Fuhrmann wrote:
On Mon, Nov 5, 2012 at 5:24 PM, vijay vi...@collab.net
mailto:vi
On Tuesday 06 November 2012 08:09 PM, Stefan Fuhrmann wrote:
On Mon, Nov 5, 2012 at 5:24 PM, vijay vi...@collab.net
mailto:vi...@collab.net wrote:
Hi,
This patch implements '--include-externals' option to 'svn list'
[Issue #4225] [1].
All tests pass with 'make check' 'make
at 09:54:56PM +0530, vijay wrote:
Hi,
This patch implements '--include-externals' option to 'svn list'
[Issue #4225] [1].
All tests pass with 'make check' 'make davautocheck'.
Attached the patch and log message.
Please review this patch and share your thoughts.
My review comments are below
for 'list'.
* subversion/tests/cmdline/externals_tests.py
(include_externals): Rename it to 'commit_include_externals'.
(list_include_externals): New test.
(test_list): Add a reference to the new test. Rename 'include_externals' to
'commit_include_externals'.
Patch by: Vijayaguru G vijay
On Wednesday 24 October 2012 07:16 PM, C. Michael Pilato wrote:
On 10/24/2012 09:25 AM, vijay wrote:
On Wednesday 24 October 2012 02:33 AM, Ben Reser wrote:
Does it make more sense to make --include-externals behave something
more like --depth rather than a simple boolean?
E.G. it'd behave
On Wednesday 24 October 2012 07:16 PM, C. Michael Pilato wrote:
On 10/24/2012 09:25 AM, vijay wrote:
On Wednesday 24 October 2012 02:33 AM, Ben Reser wrote:
Does it make more sense to make --include-externals behave something
more like --depth rather than a simple boolean?
E.G. it'd behave
On Tuesday 23 October 2012 11:42 PM, C. Michael Pilato wrote:
On 10/23/2012 01:46 PM, Bert Huijben wrote:
Before looking at the details some questions about the issue space:
How do you handle externals that are stored in svn:externals many directory
levels up? (Or even defined from above the
On Wednesday 24 October 2012 02:33 AM, Ben Reser wrote:
On Tue, Oct 23, 2012 at 9:58 AM, vijay vi...@collab.net wrote:
Hi,
I am working on patch to add '--include-externals' option to 'svn list'
[Issue #4225].
Let me try to answer few questions in the issue comment [1].
Please correct me if I
Hi,
I am working on patch to add '--include-externals' option to 'svn list'
[Issue #4225].
Let me try to answer few questions in the issue comment [1].
Please correct me if I am wrong.
1. What happens with externals inside of externals? Are they also
listed? Does
recursion stop at some
On Monday 22 October 2012 06:53 PM, Hyrum K Wright wrote:
On Mon, Oct 22, 2012 at 7:55 AM, Philip Martin
philip.mar...@wandisco.com wrote:
hwri...@apache.org writes:
Author: hwright
Date: Sun Oct 21 01:18:26 2012
New Revision: 1400545
URL: http://svn.apache.org/viewvc?rev=1400545view=rev
On Wednesday 26 September 2012 03:19 PM, Stefan Sperling wrote:
On Wed, Sep 26, 2012 at 03:52:57AM +0530, vijay wrote:
From the issue[1] description,
snip
When svndumpfilter exclude command accepts path prefixes starting with either
/pathname
or just pathname, I expect it to work the same
jeyanthan{_AT_}collab.net
Patch by: Vijayaguru G vijay{_AT_}collab.net
%02u%02u%02u $(echo $SQLITE_VERSION |
sed -e s/\./ /g))
HTTPD=httpd-2.2.22
* get-deps.sh: Update SQLite version.
Patch by: Vijayaguru G vijay{_AT_}collab.net
On 01/26/2012 06:44 PM, C. Michael Pilato wrote:
Vijay, I think you have misunderstood what these docstrings are claiming.
They are not saying, When Subversion calls the FS-loading API, it tends to
pass FALSE for validate_props by default.
They are describing the quite literal implementation
(change_rev_prop): Do the property validation indeed if 'validate_props' is
set.
Patch by: Vijayaguru G vijay{_AT_}collab.net
On Friday 14 October 2011 07:56 PM, Philip Martin wrote:
vijayvi...@collab.net writes:
$ svnmucc put -URL
svnmucc: invalid option:
../tools/client-side/svnmucc/svnmucc.c:875: (apr_err=70012)
svnmucc: E070012: getopt failure: Bad character specified on command line
Like other Unix utilities
On Monday 22 August 2011 09:37 AM, Prabhu Gnana Sundar wrote:
On Thursday 18 August 2011 06:46 PM, Daniel Shahaf wrote:
I tried your patch against
https://svn.eu.apache.org/repos/asf/subversion/README
(which uses a non-self-signed cert, but rather one for which the cert's
hostname differs from
svn_fs_get_mergeinfo2().
Patch by: Vijayaguru G vijay{_AT_}collab.net
Inspired by: hwright
On Thursday 30 June 2011 07:04 AM, Daniel Shahaf wrote:
The problem is that the prototype of ap_log_rerror() in
httpd-2.3.12-beta,
[[[
AP_DECLARE(void) ap_log_rerror(const char *file, int line, int module_index,
int level, apr_status_t status,
On Thursday 30 June 2011 05:38 PM, Daniel Shahaf wrote:
Thanks for the pointer. Do we support httpd 2.3+ in 1.6.x?
This is from our INSTALL file in our source tree.
snip
Subversion tries to compile against the latest released version
of Apache httpd 2.X.
/snip
I think we need to
On Saturday 04 June 2011 05:27 AM, Peter Samuelson wrote:
Concept patch - not tested. Does this seem reasonable? I'd really
prefer to do all our prompting on the terminal. The fact that this
frees up stdin for things like 'svn patch' and 'svnrdump load' is only
one reason.
I tested your
/cmdline/lock_tests.py
(block_unlock_if_pre_unlock_hook_fails): New test.
(test_list): Add block_unlock_if_pre_unlock_hook_fails to the list.
Patch by: Vijayaguru G vijay{_AT_}collab.net
On Thursday 02 June 2011 08:36 PM, Daniel Shahaf wrote:
+#--
+def block_unlock_if_pre_unlock_hook_fails(sbox):
+ block unlock operation if pre-unlock hook fails
+
+ sbox.build()
+ wc_dir = sbox.wc_dir
+ repo_dir =
Typo?
s/expect/except
Thanks Regards,
Vijayaguru
On Tuesday 24 May 2011 04:46 AM, s...@apache.org wrote:
Author: stsp
Date: Mon May 23 23:16:36 2011
New Revision: 1126808
URL: http://svn.apache.org/viewvc?rev=1126808view=rev
Log:
* subversion/svn/main.c
(svn_cl__options): Remove all
number of RA requests during 'svnsync info'.
* subversion/svnsync/main.c
(info_cmd): Retrieve three properties using one
svn_ra_rev_proplist() call instead of requesting them one by one.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Inspired by: Ivan
On Tuesday 05 April 2011 03:12 AM, Daniel Shahaf wrote:
kame...@apache.org wrote on Mon, Apr 04, 2011 at 12:55:38 -:
@@ -438,5 +434,29 @@ fi
say PASS: master, slave are both at r4, as expected
-exit 0
+# The following test case is for the regression issue triggered by r917523.
+# The
On Thursday 24 March 2011 07:41 PM, Kamesh Jayachandran wrote:
On 03/24/2011 05:54 PM, vijay wrote:
Just now I came to know that it fails in neon only.
I configured neon as a default ra_layer in my runtime configuration
area.
When I use serf as ra_layer, the commit succeeds.
Yes
On Sunday 27 March 2011 12:21 AM, Mark Phippard wrote:
I would love to see someone do some tests with the WC on local disk vs
network mount (1.6 and 1.7). I tried to do it using some virtual
machines I have access to at CollabNet. The problem is that the
connection of these boxes to the NetApp
On Thursday 24 March 2011 10:09 PM, Peter Samuelson wrote:
Location /svn 1/
DAV svn
SVNParentPath /repositories
/Location
Location /svn 2/
DAV svn
SVNParentPath /repositories-slave
SVNMasterURI http://localhost/svn 1
/Location
That seems like an incorrect configuration. The
by r917523.
* subversion/mod_dav_svn/mirror.c
(dav_svn__location_header_filter): Remove redundant url-encoding of new_uri.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Index: subversion/mod_dav_svn/mirror.c
Vijay.
In my FC14 your testcase passes both with and without patch.
I am investigating
Will get back.
With regards
Kamesh Jayachandran
On 03/24/2011 04:11 PM, vijay wrote:
Hi,
This patch adds a testcase for the regression triggered by r917523
and fixes it.
The revision r917523 do some url
as actual parameter.
[1]
http://httpd.apache.org/docs/trunk/developer/new_api_2_4.html#upgrading_logging
Patch by: Vijayaguru G vijay{_AT_}collab.net
Suggested by: kameshj
]]]
Index: subversion/mod_authz_svn/mod_authz_svn.c
arguments till httpd-2.2.x but 3
arguments in httpd-2.3-dev, which causes failure while compiling with
httpd-2.3-dev. So we need to handle both the cases.
* subversion/mod_authz_svn/mod_authz_svn.c
(log_access_verdict): Modify the signature.
Patch by: Vijayaguru G vijay{_AT_}collab.net
Suggested
On Saturday 01 January 2011 03:10 AM, Gavin Beau Baumanis wrote:
Ping. This patch has received no comments.
Gavin,
Julian modified this patch and committed in r1050449.
Thanks Regards,
Vijayaguru
On 18/12/2010, at 2:45 AM, vijay wrote:
Hi,
In r877617, Arfrever improved some messages
:
(organize_lock_targets): Update the comment.
Patch by: Vijayaguru G vijay{_AT_}collab.net
]]]
On Monday 27 December 2010 03:32 PM, Noorul Islam K M wrote:
vijayvi...@collab.net writes:
Hi,
This patch replaces few stale comments about svn_path_condense_targets
with svn_uri_condense_targets/svn_dirent_condense_targets in
libsvn_client/locking_commands.c:organize_lock_targets().
Thanks
On Wednesday 22 December 2010 06:42 PM, Daniel Shahaf wrote:
Kamesh Jayachandran wrote on Wed, Dec 22, 2010 at 15:10:46 +0530:
On 12/22/2010 02:02 AM, vijay wrote:
Hi,
I have attached a patch that fixes few deprecation warnings while
compiling libsvn_client/log.c.
Attached log message also
On Friday 24 December 2010 03:46 PM, vijay wrote:
On Wednesday 22 December 2010 06:42 PM, Daniel Shahaf wrote:
Kamesh Jayachandran wrote on Wed, Dec 22, 2010 at 15:10:46 +0530:
On 12/22/2010 02:02 AM, vijay wrote:
Hi,
I have attached a patch that fixes few deprecation warnings while
Hi,
I have attached a patch that fixes few deprecation warnings while
compiling libsvn_client/log.c.
Attached log message also.
Thanks Regards,
Vijayaguru
[[[
Reslove some deprecation warnings using uri/dirent functions in its respective
places.
* subversion/libsvn_client/log.c
On Wednesday 22 December 2010 11:32 AM, Noorul Islam K M wrote:
vijayvi...@collab.net writes:
Hi,
I have attached a patch that fixes few deprecation warnings while
compiling libsvn_client/log.c.
Attached log message also.
I could see this usage at one more place,
65 matches
Mail list logo