Re: [OPW] Introducing the 2013 Apache Subversion Intern

2013-02-14 Thread Gabriela Gibson
On 31/01/13 12:54, Nico Kadel-Garcia wrote: On Thu, Jan 31, 2013 at 7:24 AM, Gabriela Gibson wrote: Hi everyone, I am the current Gnome Outreach Program for Women intern for the Apache Subversion project, sponsored by Elego, Berlin, Germany. OPW is a GNOME Woman initiative that aims to inspir

Re: svn commit: r1445973 - in /subversion/trunk/subversion: include/svn_ra.h include/svn_repos.h libsvn_repos/rev_hunt.c tests/libsvn_repos/repos-test.c

2013-02-14 Thread Johan Corveleyn
On Thu, Feb 14, 2013 at 1:20 AM, Bert Huijben wrote: > We currently fetch all the revision numbers (inserted in an array in the > wrong order.. which we then reverse) and then start delivering changes. > > I would be surprised if the revision/path walk step would make a huge > difference compared

Re: [OPW] Introducing the 2013 Apache Subversion Intern

2013-02-14 Thread Daniel Shahaf
Gabriela Gibson wrote on Thu, Feb 14, 2013 at 08:03:41 +: > On 31/01/13 12:54, Nico Kadel-Garcia wrote: > >On Thu, Jan 31, 2013 at 7:24 AM, Gabriela Gibson > > wrote: > >>Hi everyone, > >> > >>I am the current Gnome Outreach Program for Women intern for the > >>Apache Subversion project, sponso

RE: svn commit: r1446113 - /subversion/branches/fsfs-format7/subversion/libsvn_fs_fs/index.c

2013-02-14 Thread Bert Huijben
> -Original Message- > From: stef...@apache.org [mailto:stef...@apache.org] > Sent: donderdag 14 februari 2013 11:58 > To: comm...@subversion.apache.org > Subject: svn commit: r1446113 - /subversion/branches/fsfs- > format7/subversion/libsvn_fs_fs/index.c > > Author: stefan2 > Date: Thu

Re: svn commit: r1445973 - in /subversion/trunk/subversion: include/svn_ra.h include/svn_repos.h libsvn_repos/rev_hunt.c tests/libsvn_repos/repos-test.c

2013-02-14 Thread C. Michael Pilato
On 02/14/2013 04:04 AM, Johan Corveleyn wrote: > On Thu, Feb 14, 2013 at 1:20 AM, Bert Huijben wrote: >> We currently fetch all the revision numbers (inserted in an array in the >> wrong order.. which we then reverse) and then start delivering changes. >> >> I would be surprised if the revision/pa

Re: svn r1446084 - tree-processor documentation in include/private/svn_diff_tree.h

2013-02-14 Thread Julian Foad
Hi Bert.  Thanks for this documentation.  It's really important and useful.  And it's well written. - Julian > Author: rhuijben > URL: http://svn.apache.org/r1446084 > Log: > * subversion/include/private/svn_diff_tree.h >   (*): Add global description of the tree processor. > >   (svn_diff_so

RE: svn commit: r1445973 - in /subversion/trunk/subversion: include/svn_ra.h include/svn_repos.h libsvn_repos/rev_hunt.c tests/libsvn_repos/repos-test.c

2013-02-14 Thread Bert Huijben
> -Original Message- > From: C. Michael Pilato [mailto:cmpil...@collab.net] > Sent: donderdag 14 februari 2013 15:10 > To: Johan Corveleyn > Cc: Bert Huijben; Bert Huijben; dev@subversion.apache.org > Subject: Re: svn commit: r1445973 - in /subversion/trunk/subversion: > include/svn_ra.h

Re: r1446169 - obtaining file-revs in reverse order

2013-02-14 Thread Julian Foad
> URL: http://svn.apache.org/r1446169 > Log: > Following up on r1446118, make the knowledge whether reversed obtaining of > file revisions is available visible by adding an ra capability. > > This to allow a future blame improvement to decide whether it can use an > improved algorithm instead of t

Re: svn commit: r1445973 - in /subversion/trunk/subversion: include/svn_ra.h include/svn_repos.h libsvn_repos/rev_hunt.c tests/libsvn_repos/repos-test.c

2013-02-14 Thread Johan Corveleyn
On Thu, Feb 14, 2013 at 3:44 PM, Bert Huijben wrote: > > >> -Original Message- >> From: C. Michael Pilato [mailto:cmpil...@collab.net] >> Sent: donderdag 14 februari 2013 15:10 >> To: Johan Corveleyn >> Cc: Bert Huijben; Bert Huijben; dev@subversion.apache.org >> Subject: Re: svn commit: r

branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Stefan Sperling
Philip indicated yesterday on IRC that all local move features he wanted to get to implemented for 1.8 now have code written. So now we're all waiting for the "local moves" yellow light on roadmap.html to turn green. I think we're at the point where we need testing by real users. We've got some te

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread C. Michael Pilato
On 02/14/2013 12:21 PM, Stefan Sperling wrote: > Philip indicated yesterday on IRC that all local move features > he wanted to get to implemented for 1.8 now have code written. > So now we're all waiting for the "local moves" yellow light on > roadmap.html to turn green. Awesome! Congratulations,

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Stefan Sperling
On Thu, Feb 14, 2013 at 12:36:13PM -0500, C. Michael Pilato wrote: > As such -- and assuming no one else has a feature waiting in the wings and > can make a compelling case for further delaying the release to accommodate > it -- I lean towards branching ASAP. And yes, +1 on delivering alphas as >

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Mark Phippard
On Thu, Feb 14, 2013 at 12:21 PM, Stefan Sperling wrote: > Philip indicated yesterday on IRC that all local move features > he wanted to get to implemented for 1.8 now have code written. > So now we're all waiting for the "local moves" yellow light on > roadmap.html to turn green. That is good to

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Hyrum K Wright
On Thu, Feb 14, 2013 at 12:45 PM, Stefan Sperling wrote: > On Thu, Feb 14, 2013 at 12:36:13PM -0500, C. Michael Pilato wrote: > > As such -- and assuming no one else has a feature waiting in the wings > and > > can make a compelling case for further delaying the release to > accommodate > > it --

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Ben Reser
On Thu, Feb 14, 2013 at 9:50 AM, Mark Phippard wrote: > That is good to hear. I saw Ben committed some tests as well. I did > see him declare victory anywhere though, so I am assuming he still > plans to add more? Yes the tests I added were the basic framework for writing them and the simplest

[patch] Don't discard sqlite's error code

2013-02-14 Thread Daniel Shahaf
Someone on IRC got "E200030: I/O error", which prompted me to write a patch that exposes the sqlite integer error code via the error string (the err->apr_err value remains unchanged, 200030 SVN_ERR_SQLITE_ERROR). Any objections in principle, or should I go ahead and see how many tests this breaks?

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread C. Michael Pilato
On 02/14/2013 03:52 PM, Daniel Shahaf wrote: > Someone on IRC got "E200030: I/O error", which prompted me to write > a patch that exposes the sqlite integer error code via the error string > (the err->apr_err value remains unchanged, 200030 SVN_ERR_SQLITE_ERROR). > > Any objections in principle, o

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread Julian Foad
Daniel Shahaf wrote: > Someone on IRC got "E200030: I/O error", which prompted me to write > a patch that exposes the sqlite integer error code via the error string > (the err->apr_err value remains unchanged, 200030 SVN_ERR_SQLITE_ERROR). > > Any objections in principle, or should I go ahead and

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread Daniel Shahaf
Julian Foad wrote on Thu, Feb 14, 2013 at 21:17:03 +: > Daniel Shahaf wrote: > > @@ -739,8 +743,8 @@ init_sqlite(void *baton, apr_pool_t *pool) > >   { > >     int err = sqlite3_config(SQLITE_CONFIG_MULTITHREAD); > >     if (err != SQLITE_OK && err != SQLITE_MISUSE) > > -      return svn_err

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Stefan Fuhrmann
On Thu, Feb 14, 2013 at 6:36 PM, C. Michael Pilato wrote: > On 02/14/2013 12:21 PM, Stefan Sperling wrote: > > Philip indicated yesterday on IRC that all local move features > > he wanted to get to implemented for 1.8 now have code written. > > So now we're all waiting for the "local moves" yellow

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread Julian Foad
Daniel Shahaf wrote: > Julian Foad wrote on Thu, Feb 14, 2013 at 21:17:03 +: >> Daniel Shahaf wrote: >> > @@ -739,8 +743,8 @@ init_sqlite(void *baton, apr_pool_t *pool) >> >    { >> >      int err = sqlite3_config(SQLITE_CONFIG_MULTITHREAD); >> >      if (err != SQLITE_OK && err != SQLITE

Re: svn commit: r1446113 - /subversion/branches/fsfs-format7/subversion/libsvn_fs_fs/index.c

2013-02-14 Thread Stefan Fuhrmann
On Thu, Feb 14, 2013 at 1:13 PM, Bert Huijben wrote: > > > > -Original Message- > > From: stef...@apache.org [mailto:stef...@apache.org] > > Sent: donderdag 14 februari 2013 11:58 > > To: comm...@subversion.apache.org > > Subject: svn commit: r1446113 - /subversion/branches/fsfs- > > form

compiler warnings buildbot

2013-02-14 Thread Daniel Shahaf
I have a build configuration that currently does autogen+configure+'make' and has no errors or warnings at all. I suggest we make a buildbot out of it so we don't have to point new compiler warnings on IRC all the time. WDYT? That's the build env, tested on 64 bit Ubuntu 12.04 (precise). [[[ $ c

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread Daniel Shahaf
Julian Foad wrote on Thu, Feb 14, 2013 at 22:11:13 +: > Daniel Shahaf wrote: > > > Julian Foad wrote on Thu, Feb 14, 2013 at 21:17:03 +: > >> Daniel Shahaf wrote: > >> > @@ -739,8 +743,8 @@ init_sqlite(void *baton, apr_pool_t *pool) > >> >    { > >> >      int err = sqlite3_config(SQLI

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread Julian Foad
  -- Certified & Supported Apache Subversion Downloads: http://www.wandisco.com/subversion/download - Original Message - > From: Daniel Shahaf > To: Julian Foad > Cc: "dev@subversion.apache.org" > Sent: Thursday, 14 February 2013, 18:45 > Subject: Re: [patch] Don't discard sqlite's

Re: [patch] Don't discard sqlite's error code

2013-02-14 Thread Daniel Shahaf
Julian Foad wrote on Fri, Feb 15, 2013 at 00:07:21 +: > The problem is that sqlite3_errstr() seems to be rather recent -- it doesn't > exist in my installed version which is sqlite v3.7.7. > We require 3.7.12. > Feel free to commit what you have and we could re-visit this part later or > n

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Branko Čibej
On 14.02.2013 20:08, Ben Reser wrote: > On Thu, Feb 14, 2013 at 9:50 AM, Mark Phippard wrote: >> That is good to hear. I saw Ben committed some tests as well. I did >> see him declare victory anywhere though, so I am assuming he still >> plans to add more? > Yes the tests I added were the basic

Re: branch 1.8 or at least start making alpha releases?

2013-02-14 Thread Branko Čibej
On 15.02.2013 04:19, Branko Čibej wrote: > There are other new features in 1.8 that would benefit from having > potential users (and packagers) look at them sooner rather than later. > So I'm firmly in the "release alpha from trunk now" camp. And let's not forget that releasing anything but an alp