subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Git at Apache
GitHub user techtonik opened a pull request: https://github.com/apache/subversion/pull/1 port to new style classes - http://stackoverflow.com/questions/54867/old... http://stackoverflow.com/questions/54867/old-style-and-new-style-classes-in-python You can merge this pull request in

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Greg Stein
Hey all, I moderated this through, in order to start a discussion. My feeling is: this is not how we want to accept patches. This approach is completely disconnected from our community. How do we talk to the person providing the change? How do we ask for modifications? How to interact? But even

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Greg Stein
Git people, The community is discussing what to do about this particular approach for sending a patch (we have a defined and published method for sending a patch to our community). That is a separate thread, but pending that... I have a separate meta/infra issue for you. The Subversion PMC does n

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Daniel Shahaf
On Thu, May 24, 2012 at 04:54:45AM -0400, Greg Stein wrote: > In short, this GitHub repository is representing "Apache Subversion" without > the PMC providing any actual oversight or any mechanism to manage it. Greg made a good point on IRC: PMC's should at least be _made aware_ that a github mirr

Re: [PATCH]: Speed up deletion of multiple files

2012-05-24 Thread Mat Booth
On 24 May 2012 00:47, Bert Huijben wrote: > > > -Original Message- > > From: Paul Burba [mailto:ptbu...@gmail.com] > > Sent: donderdag 24 mei 2012 0:43 > > To: Subversion Development > > Subject: [PATCH]: Speed up deletion of multiple files > > > > On one CollabNet's forums a user reported

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Julian Foad
Hi Anatoly. The Subversion team has just received this pull request from you.  The current patch submission process is here , and in particular we would require an email on this mailing list and a log message to explain th

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Julian Foad
Greg Stein wrote: > I moderated this through, in order to start a discussion. > > My feeling is: this is not how we want to accept patches. This > approach is completely disconnected from our community. How do we talk > to the person providing the change? How do we ask for modifications? > How to

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Julian Foad
What I said there is my personal opinion; not necessarily that of the Subversion community. - Julian I (Julian Foad) wrote: > > Hi Anatoly. > > The Subversion team has just received this pull request from you.  The > current > patch submission process is here >

Re: Fwd: Re: ISSUE: "To many open files" not being logged by master server when sync fails.

2012-05-24 Thread Philip Martin
Kenneth Miles writes: > Yes i have tried both scenarios you have described. Do you need the > strace output of both as it might be quite large. Somebody has to debug the problem and I can't reproduce it. This is the sort of thing I tried: svnadmin create repo svn mkdir -mm --parents file://`pw

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Filip Maj
Without a doubt getting access to the project mirrors on GitHub is a must-have. Setting up different teams on GitHub is trivial. Could have a "committers" team, and any other team deemed necessary. We can then add permissions such as ability to administer the github project to these teams. As for

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Philip Martin
Greg Stein writes: > My feeling is: this is not how we want to accept patches. This > approach is completely disconnected from our community. How do we talk > to the person providing the change? How do we ask for modifications? > How to interact? > > But even larger: our goal is to get people *in

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Stefan Sperling
On Thu, May 24, 2012 at 11:32:42AM +0100, Philip Martin wrote: > I don't like it. The email doesn't contain the patch or the log > message. The email has the address of the patch author in the body and > not in the headers. Patches should be visible and discussed on dev. > > Since you mention i

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Daniel Shahaf
(wearing both an infra hat and an svn hat...) Filip Maj wrote on Thu, May 24, 2012 at 03:22:19 -0700: > Without a doubt getting access to the project mirrors on GitHub is a > must-have. Setting up different teams on GitHub is trivial. Could have a > "committers" team, and any other team deemed nec

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Filip Maj
My answers in-line below. On 5/24/12 12:48 PM, "Daniel Shahaf" wrote: >(wearing both an infra hat and an svn hat...) > >Filip Maj wrote on Thu, May 24, 2012 at 03:22:19 -0700: >> Without a doubt getting access to the project mirrors on GitHub is a >> must-have. Setting up different teams on GitH

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Philip Martin
Filip Maj writes: > Without a doubt getting access to the project mirrors on GitHub is a > must-have. Setting up different teams on GitHub is trivial. Could have a > "committers" team, and any other team deemed necessary. We can then add > permissions such as ability to administer the github proj

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Daniel Shahaf
Filip Maj wrote on Thu, May 24, 2012 at 03:57:51 -0700: > My answers in-line below. > > On 5/24/12 12:48 PM, "Daniel Shahaf" wrote: > > >(wearing both an infra hat and an svn hat...) > > > >Filip Maj wrote on Thu, May 24, 2012 at 03:22:19 -0700: > >> Without a doubt getting access to the project

RE: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Gavin McDonald
As per suggestion, Ive created a 'Apache Subversion' "Team" which can 'push,pull,administer' the apache/subversion repo. Consider this a test at this point, also consider it committer priviliges that have same rules as for svn. I've added gstein and filmaj to this team. Thanks Gav... >

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Daniel Shahaf
Gavin McDonald wrote on Thu, May 24, 2012 at 21:08:43 +0930: > As per suggestion, > > Ive created a 'Apache Subversion' "Team" which can 'push,pull,administer' > the apache/subversion repo. > -1 on push rights (svn hat). The canonical place for new code to enter the Subversion codebase is http

RE: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Gavin McDonald
> -Original Message- > From: jukka.zitt...@gmail.com [mailto:jukka.zitt...@gmail.com] On Behalf > Of Jukka Zitting > Sent: Thursday, 24 May 2012 9:12 PM > To: ga...@16degrees.com.au > Cc: infrastructure-...@apache.org; Git at Apache; dav...@apache.org; > dev@subversion.apache.org > Subjec

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Jukka Zitting
Hi, On Thu, May 24, 2012 at 10:54 AM, Greg Stein wrote: > The Subversion PMC does not seem to have access to manage our presence > on GitHub, yet people seem to believe it is a viable approach to send > us patches. At a minimum, the PMC needs a way to manage our presence > on GitHub: the descript

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Jukka Zitting
Hi, On Thu, May 24, 2012 at 1:38 PM, Gavin McDonald wrote: > Ive created a 'Apache Subversion' "Team" which can 'push,pull,administer' > the apache/subversion repo. Please don't! The GitHub mirrors are strictly read-only for a purpose. We don't want development to diverge between the official A

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Filip Maj
It makes sense to me that Apache try to curate the strong developer communities that exist on GitHub. There is an ease of sending patches via GitHub that is at odds with the process of, at the minimum, registering for the dev list for a specific project and submitting a patch that way. My goal here

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Daniel Shahaf
Jukka Zitting wrote on Thu, May 24, 2012 at 13:39:48 +0200: > The GitHub pull request notification you saw on dev@subversion, is a > result of our work to better integrate GitHub workflows with those at > Apache. The idea here is to send the issue to dev@ from where the > project community can deal

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Filip Maj
I think all we want is the administration rights, but GitHub fails here in that you can't separate admin vs. push rights. Sigh. On 5/24/12 1:53 PM, "Daniel Shahaf" wrote: >Gavin McDonald wrote on Thu, May 24, 2012 at 21:08:43 +0930: >> As per suggestion, >> >> Ive created a 'Apache Subversion'

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Daniel Shahaf
Filip Maj wrote on Thu, May 24, 2012 at 04:58:15 -0700: > It makes sense to me that Apache try to curate the strong developer > communities that exist on GitHub. There is an ease of sending patches via > GitHub that is at odds with the process of, at the minimum, registering > for the dev list for

RE: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Greg Stein
On May 24, 2012 7:55 AM, "Gavin McDonald" wrote: > > > > > -Original Message- > > From: jukka.zitt...@gmail.com [mailto:jukka.zitt...@gmail.com] On Behalf > > Of Jukka Zitting > > Sent: Thursday, 24 May 2012 9:12 PM > > To: ga...@16degrees.com.au > > Cc: infrastructure-...@apache.org; Git

Re: [PATCH]: Speed up deletion of multiple files

2012-05-24 Thread Paul Burba
On Wed, May 23, 2012 at 7:47 PM, Bert Huijben wrote: > >> -Original Message- >> From: Paul Burba [mailto:ptbu...@gmail.com] >> Sent: donderdag 24 mei 2012 0:43 >> To: Subversion Development >> Subject: [PATCH]: Speed up deletion of multiple files >> >> On one CollabNet's forums a user repo

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread anatoly techtonik
Hi Julian, Thanks for the notice. The pull request was premature (I actually made by editing the file from GitHub web interface) - there are more patches to come, but as soon as they are ready I'll send an update to this thread. Feel free to comment separate revisions in the meanwhile. https://gi

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Stefan Sperling
On Thu, May 24, 2012 at 05:37:17PM +0300, anatoly techtonik wrote: > Hi Julian, > > Thanks for the notice. The pull request was premature (I actually made > by editing the file from GitHub web interface) - there are more > patches to come, but as soon as they are ready I'll send an update to > thi

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread anatoly techtonik
On Thu, May 24, 2012 at 6:03 PM, Stefan Sperling wrote: > On Thu, May 24, 2012 at 05:37:17PM +0300, anatoly techtonik wrote: >> Hi Julian, >> >> Thanks for the notice. The pull request was premature (I actually made >> by editing the file from GitHub web interface) - there are more >> patches to c

Test for reintegrating a renamed branch [svn commit: r1342283]

2012-05-24 Thread Julian Foad
Hi Cristian. I committed that merge test you are helping me write, in r1342283.  It is merge_reintegrate_tests.py 20, and it performs a reintegrate merge after renaming the source branch:   # A      -1-3-4-5-6--9   #            \              \                /

Re: [PATCH]: Speed up deletion of multiple files

2012-05-24 Thread Hyrum K Wright
On Thu, May 24, 2012 at 4:38 AM, Mat Booth wrote: > On 24 May 2012 00:47, Bert Huijben wrote: >> >> > -Original Message- >> > From: Paul Burba [mailto:ptbu...@gmail.com] >> > Sent: donderdag 24 mei 2012 0:43 >> > To: Subversion Development >> > Subject: [PATCH]: Speed up deletion of multi

Compiling SVN on Windows using Windows SDK

2012-05-24 Thread Mark Phippard
I know we have done work in recent versions so that you can compile SVN on Windows using just the freely available Windows SDK. Note that Microsoft is changing things so that this will no longer be possible. This is just a heads-up for anyone who is currently relying on that option. http://arstec

Re: Compiling SVN on Windows using Windows SDK

2012-05-24 Thread Stefan Küng
On 24.05.2012 21:55, Mark Phippard wrote: I know we have done work in recent versions so that you can compile SVN on Windows using just the freely available Windows SDK. Note that Microsoft is changing things so that this will no longer be possible. This is just a heads-up for anyone who is curr

Re: Compiling SVN on Windows using Windows SDK

2012-05-24 Thread Cristian Amarie
If I remember well, Platform SDK contains the same (AFAIK) toolchain (C, C++ compilers, linker, rc etc.). That until PSDK for Windows 8; the bad news indeed is that command line tools seems to be removed from Windows SDK 8: http://msdn.microsoft.com/en-us/windows/hardware/hh852363.aspx "The Windo

Re: subversion pull request: port to new style classes - http://stackoverflow.c...

2012-05-24 Thread Paul Davis
On Thu, May 24, 2012 at 3:54 AM, Greg Stein wrote: > Git people, > > The community is discussing what to do about this particular approach > for sending a patch (we have a defined and published method for > sending a patch to our community). That is a separate thread, but > pending that... I have

Re: Issue(?) with mod_dav_svn and requiring access to $reporoot

2012-05-24 Thread Branko Čibej
On 21.05.2012 23:00, Tony Stevenson wrote: > So, > > Earlier today I was asked to open a few children from a repo that is > essentially close by default. > Some background information: > > This setup allows commits only via an https vhost, ad this is working > perfectly. The http vhost is con