Re: [PATCH] code file names linkified in general.html of the Hacking Gttuide

2012-12-20 Thread Daniel Shahaf
Gabriela Gibson wrote on Thu, Dec 20, 2012 at 21:05:39 +: > (Optional: Please use wget to download an uncorrupted original to modify > instead of a copy saved by your browser.) Just note that we use SSI (server-side includes) to include the top and left navigation bars on each page, so the s

Re: In-Repo-Authz

2012-12-20 Thread Ben Reser
On Wed, Dec 19, 2012 at 8:21 AM, Ben Reser wrote: > My intention was to wait 72 hours and merge giving people a chance to > object to the feature or implementation. However, I said next week > because 72 hours would have had me merging this weekend, which I > didn't figure was realistic. But you

Re: Work in progress: WC "add node" including the props

2012-12-20 Thread Julian Foad
I (Julian Foad) wrote: > "Teach some WC and client 'add' functions to take the properties as > a parameter so that the whole addition including any properties can > be done all at once.  ..." I finished this and committed it in . I did this because of the update-mo

Re: Relocate and inherited props

2012-12-20 Thread Paul Burba
On Wed, Dec 19, 2012 at 1:31 PM, Philip Martin wrote: > Paul Burba writes: > >> On Wed, Dec 19, 2012 at 11:32 AM, Philip Martin >> wrote: >>> $ svnadmin create repo >>> $ svn mkdir -mm --parents file://`pwd`/repo/A/B/C >>> $ svnmucc -mm -U file://`pwd`/repo propset svn:inheritable:ppp vv1 A >>>

Re: svn commit: r1424708 - in /subversion/trunk/subversion: include/ libsvn_client/ libsvn_ra/ libsvn_wc/

2012-12-20 Thread Julian Foad
> Author: pburba > Date: Thu Dec 20 21:19:08 2012 > New Revision: 1424708 > > URL: http://svn.apache.org/viewvc?rev=1424708&view=rev > Log: > Store repos root relative paths in NODES.INHERITED_PROPS rather than full > URLs. Glad to see this, overall.  One concern: > See http://svn.haxx.se/dev/a

Re: [PATCH] code file names linkified in general.html of the Hacking Gttuide

2012-12-20 Thread Stefan Sperling
On Thu, Dec 20, 2012 at 09:05:39PM +, Gabriela Gibson wrote: > Most of this would not have happened if I had svn protection here I > guess -- would it be worthwhile to add the html part of the project > to the revision tree? It is in the repository at https://svn.apache.org/repos/asf/subversio

Re: Relocate and inherited props

2012-12-20 Thread Paul Burba
On Wed, Dec 19, 2012 at 12:14 PM, Bert Huijben wrote: > We should store relative paths. Done r1424708 -- Paul T. Burba CollabNet, Inc. -- www.collab.net -- Enterprise Cloud Development Skype: ptburba > Bert Huijben (Cell phone) > From: Philip Martin > Sent: 19-12-2012 17:33 > To: dev@subversi

Re: svn commit: r1413451 - /subversion/branches/cache-server/BRANCH-README

2012-12-20 Thread Stefan Fuhrmann
On Thu, Dec 20, 2012 at 5:08 AM, Branko Čibej wrote: > On 20.12.2012 02:08, Stefan Fuhrmann wrote: > > The ineffectiveness of our use of memcached in 1.6 had > > prompted the development of membuffer in the first place. > > > > Despite the relevant APR bug that got fixed only recently, > > there

Re: [PATCH] code file names linkified in general.html of the Hacking Gttuide

2012-12-20 Thread Gabriela Gibson
On 19/12/12 12:59, Stefan Sperling wrote: Here it looks like we'll end up closing twice. Was that on purpose? Oops :( -- Firefox saved a mangled original I worked on and when I looked at the messy patch this produced, I decided to get a fresh copy and to 'buff it out' by copying the changes

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread Daniel Shahaf
Julian Foad wrote on Thu, Dec 20, 2012 at 17:55:19 +: > Please write responses to the points I mentioned in this email.  You > don't have to agree with all of them, but I want to know whether you > agree or disagree or don't understand or aren't able to do what > I suggest, or if I misunderstoo

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread Julian Foad
Hi Prabhu. I have now looked in detail at your patch and tried using it.  I think I have found an inconsistency and a serious problem. The output for a failed revision depends on whether --keep-going was passed.  With --keep-going you print a "* Error verifying revision 2." line; without it yo

Subversion 1.7.8 released

2012-12-20 Thread Ben Reser
I'm happy to announce the release of Apache Subversion 1.7.8. Please choose the mirror closest to you by visiting: http://subversion.apache.org/download/#recommended-release The SHA1 checksums are: 12c7d8d5414bba74c9777c4d1dae74f152df63c2 subversion-1.7.8.tar.bz2 1e298368cc2a73337eaa

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread Daniel Shahaf
Julian Foad wrote on Thu, Dec 20, 2012 at 13:24:46 +: > That sounds good.  I was surprised to learn that the progress > notifications ("* Verified...") were already going to stderr. Probably because "* Dumped revision %ld." goes to stderr, and verify shares code with dump.

Re: svn commit: r1424469 - in /subversion/trunk/subversion: libsvn_client/repos_diff.c tests/cmdline/merge_reintegrate_tests.py tests/cmdline/merge_tests.py tests/cmdline/merge_tree_conflict_tests.py

2012-12-20 Thread Julian Foad
+1 on this patch.  All looks good, just one minor point below. Bert Huijben wrote: > URL: http://svn.apache.org/viewvc?rev=1424469&view=rev > Log: > In the repository-repository diff handler: Don't retrieve pristine > properties > when we already know that there are no differences to report on t

Re: [PATCH] Improve error handling in svn_ra_serf__replay_range()

2012-12-20 Thread vijay
On Wednesday 19 December 2012 07:27 PM, Daniel Shahaf wrote: +@SkipUnless(svntest.main.is_ra_type_dav) +def dump_url_not_in_head(sbox) : Spurious whitespace. + "dump: URL deleted in HEAD should return error" + sbox.build(create_wc = False) + + E_url = sbox.repo_url + '/A/B/E' + # Delete

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread C. Michael Pilato
On 12/20/2012 08:24 AM, Julian Foad wrote: > The way I understood C-Mike's suggestion was that all the "* Verified > revision R" and "* Error verifying revision R" messages (and probably the > final summary line too) would go to stdout, whereas all the "svnadmin: > Exx: ..." messages would go t

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread Julian Foad
Prabhu Gnana Sundar wrote on 14 December 2012: > On 12/10/2012 08:15 PM, Julian Foad wrote: >> Prabhu Gnana Sundar >> >>> This patch is a follow up of the long discussion we had in thread [1]. >> >> Please will you summarize the issues that were raised in the previous >> discussion and how y

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread Daniel Shahaf
> Index: subversion/libsvn_repos/dump.c > === > --- subversion/libsvn_repos/dump.c(revision 1420101) > +++ subversion/libsvn_repos/dump.c(working copy) > @@ -1413,19 +1452,31 @@ >void *cancel_edit_baton; >svn_fs

Re: [PATCH] Implement svnadmin verify --keep-going

2012-12-20 Thread Prabhu Gnana Sundar
Thanks Daniel, I have worked on the suggestions that you gave and am attaching the new patch and log message with this mail. Please share your thoughts Thanks and regards Prabhu On 12/10/2012 08:24 PM, Daniel Shahaf wrote: Index: subversion/tests/cmdline/svnadmin_tests.py

Re: [Issue 4278] 'svn log' can't be backgrounded over serf

2012-12-20 Thread Daniel Shahaf
l...@tigris.org wrote on Wed, Dec 19, 2012 at 23:12:37 -0800: > --- Additional comments from l...@tigris.org Wed Dec 19 23:12:37 -0800 > 2012 --- > Fix has been committed to serf trunk in r1708. I'll backport it to the 1.2.x > branch before its release. Thanks Lieven.