Assertion in svn_uri_is_canonical

2013-08-22 Thread kmradke
Passing an invalid URL to svn co causes an abort and core dump. This fails with all protocols http, svn, file. It occurs with all versions I tested (1.7.5, 1.7.11, 1.7.12, 1.8.1, and the now defunct 1.8.2) It occurs with multiple subcommand (ls, info, etc.) It happens on both unix and windows

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-30 Thread kmradke
> On Tue, Jun 25, 2013 at 3:11 PM, wrote: > > > I agree that force-http10 is not good name and semantic. Actually > > > these proxies is not busted: it's allowed to HTTP/1.1 proxies to > > > require content-length if they want. And strictly speaking proxies may > > > have different behavior for d

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-25 Thread kmradke
> > I agree that force-http10 is not good name and semantic. Actually > > these proxies is not busted: it's allowed to HTTP/1.1 proxies to > > require content-length if they want. And strictly speaking proxies may > > have different behavior for different requests. > > From *our* standpoint, they

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-21 Thread kmradke
> > On Jun 21, 2013 10:36 AM, "Ivan Zhakov" wrote: > > > > > > >> Problem with this approach that some servers may support HTTP/1.1 > > > >> partially. I.e. declare them as HTTP/1.1 but do not support chunked > > > >> Transfer-Encoding. > > > > > > > > Then fix that problem. Add a flag saying

Re: svn commit: r1495419 - in /subversion/trunk/subversion/libsvn_ra_serf: options.c ra_serf.h serf.c util.c

2013-06-21 Thread kmradke
> On Jun 21, 2013 10:36 AM, "Ivan Zhakov" wrote: > > > > >> Problem with this approach that some servers may support HTTP/1.1 > > >> partially. I.e. declare them as HTTP/1.1 but do not support chunked > > >> Transfer-Encoding. > > > > > > Then fix that problem. Add a flag saying "busted_http11

Re: [serf-dev] [Patch] Adding NTLM Support to Serf - Work in progress / Subversion regression

2013-06-20 Thread kmradke
> Did you read my email above? Let me explain again. Server advertises > supported authentication schemes, for example Microsoft IIS: > [[[ > C: GET / HTTP/1.1 > S: HTTP/1.1 401 Authorization Required > S: WWW-Authenticate: NTLM > S: WWW-Authenticate: Negotiate > S: WWW-Authenticate: Basic > ]]] >

Re: Windows build still requires apr-util?

2013-03-13 Thread kmradke
> > Nor should you even try to, IMO. There comes a time when it ceases to > > make any sense to support an old compiler that's been EOL-d a while ago, > > especially since free versions of newer compilers are available. > > > > I would support removing the .dsp generator completely. > > +1, it's

Re: reposurgeon now writes Subversion repositories

2012-11-30 Thread kmradke
"Eric S. Raymond" wrote on 11/30/2012 03:53:45 PM: > Ben Reser : > > This is really a philosophical difference between a centralized > > version control system and DVCS. > > No, no, no. If you think that's true, I have failed to communicate. > > I'm not handwaving about philosophy. I'm pointing

Re: serf in 1.8

2012-11-13 Thread kmradke
> From: Philip Martin > kmra...@rockwellcollins.com writes: > > > *one* of my servers currently logs around 10GB per week with neon > > only access. I'd probably have to change from rotating logs > > weekly to rotating logs hourly! Disabling logging is not an option. > > It depends what sort o

Re: serf in 1.8

2012-11-13 Thread kmradke
> Greg Stein writes: > > > On Mon, Nov 12, 2012 at 6:20 PM, Philip Martin > > wrote: > >>... > >> Another concern is the increased server logging due to the large > >> increase in the number of requests. A 1.8 server does better than older > >> servers, about 50% fewer requests on checkout, bu

Re: Status of trunk

2012-06-21 Thread kmradke
Daniel Shahaf wrote on 06/21/2012 12:57:46 PM: > > > > Also, specifying --without-apxs does not correctly disable building > > > > mod_dav_svn and mod_dontdothat. The build fails with a missing > > httpd.h, > > > > error, so I pointed it to my local build of httpd as a workaround. > > > > > >

Re: Status of trunk

2012-06-21 Thread kmradke
> From: Daniel Shahaf > > Also, specifying --without-apxs does not correctly disable building > > mod_dav_svn and mod_dontdothat. The build fails with a missing httpd.h, > > error, so I pointed it to my local build of httpd as a workaround. > > > > httpd 2.2 or 2.4? I pointed it to 2.2.22 as

Re: Status of trunk

2012-06-21 Thread kmradke
> Philip and I were just chatting about the 1.8 release and where we are > on that. For a thought experiment, we asked, "can we ship trunk > today?" The answer was "no" for three specific items: > > * local moves are kinda broken (thus: fix or roll back) > * svnrdump does not work over ra_serf (mo

Re: [PATCH]: credentials are not saved if username differs from cached version but password does not

2012-06-21 Thread kmradke
> On 06/20/2012 10:22 PM, kmra...@rockwellcollins.com wrote: > > Dmitry Pavlenko wrote on 06/20/2012 11:14:58 AM: > >> simple_providers.c (svn_auth__simple_creds_cache_get): I propose to > >> drop all assignments > >> "need_to_save = FALSE" except the initial one; otherwise assignment > >> to FAL

Re: [PATCH]: credentials are not saved if username differs from cached version but password does not

2012-06-20 Thread kmradke
Dmitry Pavlenko wrote on 06/20/2012 11:14:58 AM: > simple_providers.c (svn_auth__simple_creds_cache_get): I propose to > drop all assignments > "need_to_save = FALSE" except the initial one; otherwise assignment > to FALSE may override existing > TRUE value. This may happen if default_username

Re: [Sidebar] - Interesting discussion on impact of "simple" HTTP requests

2012-06-06 Thread kmradke
Mark Phippard wrote on 06/06/2012 10:18:52 AM: > > On Tue, Jun 5, 2012 at 8:28 AM, Mark Phippard wrote: > >> Keep in mind that this is not about server load, they are looking at > >> this from bandwidth. So a cache in front of the server would not help > >> them at all. Eclipse.org has a 10MB

Re: format of svn:author

2012-01-05 Thread kmradke
Mark Mielke wrote on 01/05/2012 12:36:10 PM: > On 01/05/2012 12:34 PM, Branko Čibej wrote: > > On 05.01.2012 18:25, Mark Mielke wrote: > >> On 01/05/2012 12:04 PM, Branko Čibej wrote: > >>> Ha, but svn:author currently fills that role. So why add another > >>> property? > >> If svn:author is defin

Re: server impact (was: 1.7.0-beta2 this afternoon)

2011-07-20 Thread kmradke
Greg Stein wrote on 07/20/2011 12:32:42 PM: > > Somewhat off-topic, but there was also previous concern that > > the multiple connections that serf uses might overly stress some > > larger servers. Do we have any idea how many additional connections > > a typical server would see? For example, i

Re: 1.7.0-beta2 this afternoon

2011-07-20 Thread kmradke
> From: Greg Stein > On Wed, Jul 20, 2011 at 09:23, wrote: > > Greg Stein wrote on 07/19/2011 06:53:12 PM: > >> On Tue, Jul 19, 2011 at 10:41, C. Michael Pilato > >> wrote: > >> >... > >> > release-readiness? What about ra_serf's "default-ness" -- is it in the > >> > appropriate state base

Re: 1.7.0-beta2 this afternoon

2011-07-20 Thread kmradke
Greg Stein wrote on 07/19/2011 06:53:12 PM: > On Tue, Jul 19, 2011 at 10:41, C. Michael Pilato wrote: > >... > > release-readiness? What about ra_serf's "default-ness" -- is it in the > > appropriate state based on that library's readiness at this moment? > > The only pending serf issue is m

Re: [PATCH] Improve reporting WC is locked/busy ('run cleanup')

2011-07-14 Thread kmradke
Stefan Sperling wrote on 07/14/2011 08:40:29 AM: > On Thu, Jul 14, 2011 at 08:17:07AM -0500, kmra...@rockwellcollins.com wrote: > > Julian Foad wrote on 07/14/2011 06:58:24 AM: > > > The attached patch makes some improvements to how we report that the WC > > > is locked or busy (the 'run clean

Re: [PATCH] Improve reporting WC is locked/busy ('run cleanup')

2011-07-14 Thread kmradke
Julian Foad wrote on 07/14/2011 06:58:24 AM: > The attached patch makes some improvements to how we report that the WC > is locked or busy (the 'run cleanup' messages). I need a sanity check > to make sure I've understood the relationship between how we want to > handle 'the work queue is busy' a

Re: Does fsfs revprop packing no longer allow usage of traditional backup software?

2011-06-30 Thread kmradke
Philip Martin wrote on 06/30/2011 01:54:15 PM: > Daniel Shahaf writes: > > >> Or asked a different way, does a new commit update the revprop.db or > >> does it just create files like 1.6 did and things are only added > >> to the db during svnadmin pack... > > > > The latter. > > "Only" is wron

Re: Does fsfs revprop packing no longer allow usage of traditional backup software?

2011-06-30 Thread kmradke
Daniel Shahaf wrote on 06/30/2011 01:41:51 PM: > > >> The reason SQLite was used is that rev props can be edited via svn > > >> prop*--revprop > > >> > > >> So you could also disable the pre-revprop-change hook during a backup > > >> and that should make you safe. And if you have not enabled it

Re: Does fsfs revprop packing no longer allow usage of traditional backup software?

2011-06-30 Thread kmradke
Mark Phippard wrote on 06/30/2011 01:08:52 PM: > On Thu, Jun 30, 2011 at 2:06 PM, Daniel Shahaf wrote: > > You can't currently pack revision shards unless you also pack revprop > > shards. (but that's easy to change) > > FWIW, that was my assumption. I do not personally think that is a big >

Re: Does fsfs revprop packing no longer allow usage of traditional backup software?

2011-06-30 Thread kmradke
Philip Martin wrote on 06/30/2011 12:43:54 PM: > >> - since 1.6, if FSFS rep-sharing is not disabled there is the problem > >>of copying the rep-sharing SQLite database. > > > > rep-cache.db? I thought that could be easily regenerated. Is there > > another .db file I missed? > > Yes, rep

Re: Does fsfs revprop packing no longer allow usage of traditional backup software?

2011-06-30 Thread kmradke
Philip Martin wrote on 06/30/2011 11:53:54 AM: > > I know using traditional server backup software has never been > > recommended on fsfs repositories. However, since the server > > never rewrote any files after the transaction was finalized, the > > only previous issues with using a snapshot ba

Does fsfs revprop packing no longer allow usage of traditional backup software?

2011-06-30 Thread kmradke
[This thread was originally started on the users list. A snippet is attached below for reference. Daniel encouraged me to bring it to the dev list.] I know using traditional server backup software has never been recommended on fsfs repositories. However, since the server never rewrote any files

Re: should we really ship serf as default for 1.7? (was: Re: serf and sourceforge.net don't get along)

2011-06-24 Thread kmradke
Ivan Zhakov wrote on 06/24/2011 10:46:37 AM: > On Fri, Jun 24, 2011 at 17:48, wrote: > > > > Is anybody using serf with negotiate authentication? I can't get it to > > work with a 1.6.5 server using mod_auth_kerb and negotiate. > > > > The server sends the 401: > > > > WWW-Authenticate: Negoti

Re: should we really ship serf as default for 1.7?

2011-06-24 Thread kmradke
Stefan Küng wrote on 06/24/2011 09:04:41 AM: > On 24.06.2011 15:50, Mark Phippard wrote: > > On Fri, Jun 24, 2011 at 9:48 AM, wrote: > > > >> Is anybody using serf with negotiate authentication? I can't get it to > >> work with a 1.6.5 server using mod_auth_kerb and negotiate. > >> > >> The ser

Re: should we really ship serf as default for 1.7? (was: Re: serf and sourceforge.net don't get along)

2011-06-24 Thread kmradke
Stefan Sperling wrote on 06/24/2011 09:02:15 AM: > On Fri, Jun 24, 2011 at 08:55:02AM -0500, kmra...@rockwellcollins.com wrote: > > Not sure, but I know serf shouldn't keep sending the same endless > > unauthenticated request until I control-C the command. No indication > > it was even doing any

Re: should we really ship serf as default for 1.7? (was: Re: serf and sourceforge.net don't get along)

2011-06-24 Thread kmradke
Mark Phippard wrote on 06/24/2011 08:50:22 AM: > On Fri, Jun 24, 2011 at 9:48 AM, wrote: > > > Is anybody using serf with negotiate authentication? I can't get it to > > work with a 1.6.5 server using mod_auth_kerb and negotiate. > > > > The server sends the 401: > > > > WWW-Authenticate: Neg

Re: should we really ship serf as default for 1.7? (was: Re: serf and sourceforge.net don't get along)

2011-06-24 Thread kmradke
Stefan Sperling wrote on 06/24/2011 08:06:28 AM: > Subject: should we really ship serf as default for 1.7? (was: Re: > serf and sourceforge.net don't get along) > > On Wed, Jun 22, 2011 at 09:17:37PM -0400, C. Michael Pilato wrote: > > On 06/22/2011 07:30 PM, Daniel Shahaf wrote: > > > Bolstridg

Re: Running benchmarks/suite1 on Windows

2011-06-21 Thread kmradke
Johan Corveleyn wrote on 06/21/2011 03:01:31 PM: [snip] > > Max_Path is 256 chars excluding drive part. Max path for unicode > > version of API is 32,767 when using path with "\\?\" style. I think > > cmd should be able to work with \\?\ . > > > > Ref: http://msdn.microsoft.com/en-us/library/aa3

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

2011-03-02 Thread kmradke
Paul Burba wrote on 03/02/2011 10:10:40 AM: > On Wed, Mar 2, 2011 at 10:44 AM, C. Michael Pilato > wrote: > > On 03/02/2011 10:40 AM, kmra...@rockwellcollins.com wrote: > >> "C. Michael Pilato" wrote on 01/28/2011 01:44:26 PM: > >>> Subject: Re: [PATCH] fix for issue #3789 log -g regression in

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

2011-03-02 Thread kmradke
"C. Michael Pilato" wrote on 01/28/2011 01:44:26 PM: > Subject: Re: [PATCH] fix for issue #3789 log -g regression in r1028108 > > 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

Re: 1.6.16 next week (now 23 Feb)

2011-02-28 Thread kmradke
> On Wed, Feb 9, 2011 at 10:00 PM, Hyrum K Wright wrote: > > Hello folks: I'd like to cut 1.6.16 next Wednesday. Please nominate / > > review / vote / etc in the mean time. > > Hopefully this isn't dashing too many hopes, but I'm running out of > tuits this week, and it appears folks still have

Re: State of Python Bindings for Windows

2010-09-21 Thread kmradke
anatoly techtonik wrote on 09/20/2010 05:16:18 AM: > I can't find any binaries for Python bindings for Windows. Previously > they were available from tigris.org site, but now there aren't any > updates. It will greatly reduce frustration for users of hgsubversion > and other Windows tools if up t

lock report inefficiencies

2010-04-14 Thread kmradke
When debugging some repository performance problems, I narrowed it down to repositories with large numbers of locked files. For example, if I do an "svn ls -v http://server/repo/path"; on a repository with 26k locked files, the server generates an 11MB xml response that included lock information a

Re: Is sqlite fast enough?

2010-02-19 Thread kmradke
Mark Phippard wrote on 02/19/2010 09:32:23 AM: > On Fri, Feb 19, 2010 at 6:43 AM, wrote: > > > Good to hear. I know the TortoiseSVN devs are not using the dev svn > > branch > > currently because of the previous huge performance issues they saw in > > the past. Sounds like it might be time t

RE: Is sqlite fast enough?

2010-02-19 Thread kmradke
> > > What platform were these test executed on? We need to make sure > > > windows platforms are just as zippy. > > > > I tested on Debian stable. I don't currently build on Windows, but > > all the code I used is in my first email so you should be able to > > reproduce it if you want. I do ha

Re: Is sqlite fast enough?

2010-02-18 Thread kmradke
Greg Stein wrote on 02/18/2010 03:31:19 PM: > Awesome work! We've just been assuming/hoping it would be fast enough, > and would resolve any problems "later". It is good to see we're in the > right ballpark. > > On Thu, Feb 18, 2010 at 13:21, Philip Martin mar...@wandisco.com> wrote: > > How fas

Re: Re-use connection for svn:externals

2010-02-10 Thread kmradke
Phillip Hellewell wrote on 02/10/2010 09:11:40 AM: > > The '3 seconds delay' is most likely caused by the slow opening of HTTP > > sessions which walk up the repository to find the repository root.This case > > is already fixed in HTTPv2. (svn:// is much faster than this. And svn+ssh:// > > dep

Re: website getting subversion windows binaries

2010-02-09 Thread kmradke
justin.erenkra...@gmail.com wrote on 02/09/2010 12:11:04 PM: > On Tue, Feb 9, 2010 at 8:05 AM, wrote: > > news wrote on 02/09/2010 01:02:52 AM: > >> > >> 'getting subversion' still lists the 'tigris.org' windows binaries > >> that were provided by D.J. Heap. > >> But they are stuck at 1.6.6 and

Re: website getting subversion windows binaries

2010-02-09 Thread kmradke
news wrote on 02/09/2010 01:02:52 AM: > > 'getting subversion' still lists the 'tigris.org' windows binaries > that were provided by D.J. Heap. > But they are stuck at 1.6.6 and as D.J. stated in > > http://subversion.tigris.org/ds/viewMessage.do? > dsForumId=462&dsMessageId=2423180 > > he won'

Re: 1.6.8 up for signing / testing

2010-01-19 Thread kmradke
"Hyrum K. Wright" wrote on 01/12/2010 04:52:33 PM: > After pulling 1.6.7 due to a segfault regression, we've rerolled the > branch as Subversion 1.6.8. The magic revision is r898464, and you > can find the tarballs here: > > http://orac.ece.utexas.edu/pub/svn/1.6.8/ > > Please be sure to tes

Re: svn+ssh issues with svn 1.6.6 on windows (especially for svn-external repo)

2010-01-11 Thread kmradke
Magnus Torfason wrote on 01/11/2010 01:39:18 PM: > After some further exploration, I have an update on my issue with svn up > failing on my svn-externals-infested repository. Below, I report an > alternative workaround, and the output of ps while doing the update. > > On 1/11/2010 11:53 AM, M

Re: Subversion in 2010

2010-01-06 Thread kmradke
Julian Foad wrote on 01/06/2010 10:16:55 AM: > Greg Hudson wrote: > > On Mon, 2010-01-04 at 11:31 -0500, C. Michael Pilato wrote: > > > "To be a compelling replacement for git/Mercurial", perhaps? > > > > That seems tough. > > Heh. A vision that's simple to attain is hardly a vision. I person

Does get lock report support depth?

2009-11-30 Thread kmradke
Is there a reason the get lock report does not support a "depth" concept and just reports all locks "at or below" the specified path? I noticed on a large repository with a large number of locks, this report can be many megabytes worth of (worthless) network transmitted data when doing a simple no