On Wed, Jul 13, 2011 at 03:29:15PM -0500, Hyrum K Wright wrote:
> All,
>
> Our first prerelease from the 1.7.x branch is now up for testing and
> signing: 1.7.0-beta1. The magic revision is r1146221 (but a known bug
> in the release scripts doesn't include that rev in the header file).
> You can
I was chatting with Chris Mattmann earlier today, and was doing some
archeology to determine when the serf project was really started. The answer
to that is September, 2001.
But here is where it gets more interesting...
Serf was actually started as "apr-serf". A project within the APR household
t
rhuij...@apache.org writes:
> Author: rhuijben
> Date: Wed Jul 13 12:47:26 2011
> New Revision: 1145982
>
> URL: http://svn.apache.org/viewvc?rev=1145982&view=rev
> Log:
> Resolve another review todo from issue #3779 "how to handle actual only nodes"
> by making 'svn proplist' return a non zero ex
rhuij...@apache.org writes:
> Author: rhuijben
> Date: Wed Jul 13 12:28:17 2011
> New Revision: 1145972
>
> URL: http://svn.apache.org/viewvc?rev=1145972&view=rev
> Log:
> Make the 'populate target tree' code in wc_db.c return an error when it
> receives an nonexisting node as (root-)target.
> (Th
>> +++ subversion/trunk/subversion/include/svn_client.h Wed Jul 13 21:43:48 2011
>> @@ -5388,10 +5388,10 @@ typedef struct svn_client_info2_t
>> const char *last_changed_author;
>>
>> /** An exclusive lock, if present. Could be either local or remote. */
>> - svn_lock_t *lock;
>> + const s
On Wed, Jul 13, 2011 at 22:39, wrote:
>...
> +++ subversion/branches/fs-progress/subversion/include/svn_fs.h Thu Jul 14
> 02:39:52 2011
> @@ -246,6 +246,24 @@ svn_fs_upgrade(const char *path,
> apr_pool_t *pool);
>
> /**
> + * Callback function type for progress notification.
> +
On Wed, Jul 13, 2011 at 21:29, wrote:
> Author: danielsh
> Date: Thu Jul 14 01:29:58 2011
> New Revision: 1146537
>
> URL: http://svn.apache.org/viewvc?rev=1146537&view=rev
> Log:
> Create fs-progress branch, for issue #3958.
Why a branch rather thank trunk? How would this interfere with trunk w
Nothing to see here. (The syntax I used doesn't have the semantics
I expected it to have.)
Daniel Shahaf wrote on Thu, Jul 14, 2011 at 05:53:52 +0300:
> From my shell session:
>
> [[[
> % time sqlite3 foo.db "SELECT hash, revision, offset, size, expanded_size
> FROM rep_cache;" > /dev/null
> sq
>From my shell session:
[[[
% time sqlite3 foo.db "SELECT hash, revision, offset, size, expanded_size FROM
rep_cache;" > /dev/null
sqlite3 foo.db > /dev/null 12.72s user 0.69s system 98% cpu 13.575 total
% time sqlite3 foo.db "SELECT hash, revision, offset, size, expanded_size FROM
rep_cache;
gst...@apache.org wrote on Wed, Jul 13, 2011 at 21:43:48 -:
> Author: gstein
> Date: Wed Jul 13 21:43:48 2011
> New Revision: 1146492
>
> URL: http://svn.apache.org/viewvc?rev=1146492&view=rev
> Log:
> The public API should keep the underlying structures const to disallow
> changing "via" the
On Wed, Jul 13, 2011 at 21:10, Daniel Shahaf wrote:
> Greg Stein wrote on Wed, Jul 13, 2011 at 20:54:56 -0400:
>> On Wed, Jul 13, 2011 at 20:32, wrote:
>> > Author: danielsh
>> > Date: Thu Jul 14 00:32:05 2011
>> > New Revision: 1146528
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1146528&vie
Greg Stein wrote on Wed, Jul 13, 2011 at 20:54:56 -0400:
> On Wed, Jul 13, 2011 at 20:32, wrote:
> > Author: danielsh
> > Date: Thu Jul 14 00:32:05 2011
> > New Revision: 1146528
> >
> > URL: http://svn.apache.org/viewvc?rev=1146528&view=rev
> > Log:
> > * subversion/libsvn_fs_fs/fs_fs.c
> > (wr
On Wed, Jul 13, 2011 at 20:32, wrote:
> Author: danielsh
> Date: Thu Jul 14 00:32:05 2011
> New Revision: 1146528
>
> URL: http://svn.apache.org/viewvc?rev=1146528&view=rev
> Log:
> * subversion/libsvn_fs_fs/fs_fs.c
> (write_config): Document that 'svnadmin verify' will access (and thus create)
Blair Zajac wrote on Wed, Jul 13, 2011 at 16:59:51 -0700:
>
> On Jul 13, 2011, at 4:22 PM, danie...@apache.org wrote:
>
> > Author: danielsh
> > Date: Wed Jul 13 23:22:45 2011
> > New Revision: 1146513
> >
> > URL: http://svn.apache.org/viewvc?rev=1146513&view=rev
> > Log:
> > Withdraw this befo
On Wed, Jul 13, 2011 at 4:29 PM, Hyrum K Wright wrote:
> All,
>
> Our first prerelease from the 1.7.x branch is now up for testing and
> signing: 1.7.0-beta1. The magic revision is r1146221 (but a known bug
> in the release scripts doesn't include that rev in the header file).
> You can find the
On Jul 13, 2011, at 4:22 PM, danie...@apache.org wrote:
> Author: danielsh
> Date: Wed Jul 13 23:22:45 2011
> New Revision: 1146513
>
> URL: http://svn.apache.org/viewvc?rev=1146513&view=rev
> Log:
> Withdraw this before I set a record on the number of vetoes.
>
> [ If anyone would be +1 on a b
See:
http://subversion.apache.org/docs/release-notes/1.7.html#subtree-mergeinfo-recording
Sent from my iPhone
On Jul 13, 2011, at 6:29 PM, Peter Samuelson wrote:
>
> [Greg Stein]
>> Is there any way that we can remove the svn:mergeinfo property from
>> those files? I'm not sure if that would
[Greg Stein]
> Is there any way that we can remove the svn:mergeinfo property from
> those files? I'm not sure if that would be Proper, or if that would
> bust something up.
They probably aren't needed, but we really should eat our own dogfood.
If subtree mergeinfo causes problems (even if only t
Oh. And I realized that svn_checksum_to_cstring() only started to
accept NULL in 1.7. Which means that, currently, in 1.7.x,
* svn_checksum_to_cstring(NULL) returns NULL
* svn_checksum_to_cstring_display(NULL) segfaults
Generally I prefer consistency before I prefer that My Way gets
chosen... w
If somebody is editing STATUS he/you can add my +1 to this patch.
Bert Huijben (Cell phone) From: gst...@apache.org
Sent: woensdag 13 juli 2011 23:46
To: comm...@subversion.apache.org
Subject: svn commit: r1146494 - /subversion/branches/1.7.x/STATUS
Author: gstein
Date: Wed Jul 13 21:45:58 2011
Ne
On Wed, Jul 13, 2011 at 13:34, wrote:
> Author: danielsh
> Date: Wed Jul 13 17:34:25 2011
> New Revision: 1146149
>
> URL: http://svn.apache.org/viewvc?rev=1146149&view=rev
> Log:
> Merge r1146145 from the revprop-caching branch. (This just
> adds a docstring.)
>
> [ Somehow 'svn info' reported
Just adding two facts to the mix:
The use case I had was representation_string() trying to stringify
md5_checksum, which is NULL for reps retrieved from the cache.
Looking at the other callers, most of them seem to pass the checksum to
'info' or to an editor function documented as ignoring the ch
On Wed, Jul 13, 2011 at 4:21 PM, Greg Stein wrote:
> On Wed, Jul 13, 2011 at 13:11, wrote:
>> Author: hwright
>> Date: Wed Jul 13 17:11:42 2011
>> New Revision: 1146143
>>
>> URL: http://svn.apache.org/viewvc?rev=1146143&view=rev
>> Log:
>> * tools/dist/release.py
>> (roll_tarballs): One more s
On Fri, Jul 8, 2011 at 16:12, wrote:
>...
> +++ subversion/branches/revprop-packing/subversion/include/svn_io.h Fri Jul
> 8 20:12:48 2011
> @@ -1183,6 +1183,12 @@ svn_stream_readline(svn_stream_t *stream
> * Read the contents of the readable stream @a from and write them to the
> * writable s
On Wed, Jul 13, 2011 at 13:11, wrote:
> Author: hwright
> Date: Wed Jul 13 17:11:42 2011
> New Revision: 1146143
>
> URL: http://svn.apache.org/viewvc?rev=1146143&view=rev
> Log:
> * tools/dist/release.py
> (roll_tarballs): One more special case for the 'nightly' Version.
All of these special c
On Wed, Jul 13, 2011 at 17:08, Daniel Shahaf wrote:
> gst...@apache.org wrote on Wed, Jul 13, 2011 at 20:51:17 -:
>> Author: gstein
>> Date: Wed Jul 13 20:51:17 2011
>> New Revision: 1146452
>>
>> URL: http://svn.apache.org/viewvc?rev=1146452&view=rev
>> Log:
>> votes!
>>
>> Modified:
>> s
On Wed, Jul 13, 2011 at 15:00, wrote:
> Author: hwright
> Date: Wed Jul 13 19:00:15 2011
> New Revision: 1146228
>
> URL: http://svn.apache.org/viewvc?rev=1146228&view=rev
> Log:
> release.py: String-ify the Version objects for use by ezt. (I would have
> thought that ezt would have done this fo
gst...@apache.org wrote on Wed, Jul 13, 2011 at 20:51:17 -:
> Author: gstein
> Date: Wed Jul 13 20:51:17 2011
> New Revision: 1146452
>
> URL: http://svn.apache.org/viewvc?rev=1146452&view=rev
> Log:
> votes!
>
> Modified:
> subversion/branches/1.7.x/STATUS
>
> Modified: subversion/branc
On Wed, Jul 13, 2011 at 15:44, wrote:
> Author: danielsh
> Date: Wed Jul 13 19:44:57 2011
> New Revision: 1146306
>
> URL: http://svn.apache.org/viewvc?rev=1146306&view=rev
> Log:
> Nominate r1146222.
>
> I've checked the rest of fs_fs.c, and there are no other similar problems,
> assuming that a
All,
Our first prerelease from the 1.7.x branch is now up for testing and
signing: 1.7.0-beta1. The magic revision is r1146221 (but a known bug
in the release scripts doesn't include that rev in the header file).
You can find the tarballs here:
http://people.apache.org/~hwright/svn/1.7.0-beta1/
svn_checksum_to_cstring() was added in 1.6 and learnt to do this NULL check in
1.7.
I've documented the change for svn_checksum_to_cstring_display() and
added that to the STATUS entry.
Bert, Mike: if you're standing by your -0's, please let me know so I can
prepare a 1.7.0 backport patch fixing
On 07/13/2011 04:00 PM, Daniel Shahaf wrote:
> C. Michael Pilato wrote on Wed, Jul 13, 2011 at 15:54:31 -0400:
>> On 07/13/2011 03:46 PM, Daniel Shahaf wrote:
>>> cmpil...@apache.org wrote on Wed, Jul 13, 2011 at 19:28:18 -:
* r1146214
Handle NULL inputs when stringifying svn_ch
C. Michael Pilato wrote on Wed, Jul 13, 2011 at 15:54:31 -0400:
> On 07/13/2011 03:46 PM, Daniel Shahaf wrote:
> > cmpil...@apache.org wrote on Wed, Jul 13, 2011 at 19:28:18 -:
> >> * r1146214
> >> Handle NULL inputs when stringifying svn_checksum_t.
> >> @@ -38,6 +39,7 @@ Candidate chang
On 07/13/2011 03:47 PM, danie...@apache.org wrote:
> Author: danielsh
> Date: Wed Jul 13 19:47:55 2011
> New Revision: 1146313
>
> URL: http://svn.apache.org/viewvc?rev=1146313&view=rev
> Log:
> * STATUS: Clarify a section header.
Ah! That helps. Thanks!
--
C. Michael Pilato
CollabNet <>
On 07/13/2011 03:46 PM, Daniel Shahaf wrote:
> cmpil...@apache.org wrote on Wed, Jul 13, 2011 at 19:28:18 -:
>> * r1146214
>> Handle NULL inputs when stringifying svn_checksum_t.
>> @@ -38,6 +39,7 @@ Candidate changes:
>> Avoids segfaults.
>> Votes:
>> +1: danielsh
>> +
On 07/13/2011 03:40 PM, Daniel Shahaf wrote:
> As I see it, the options are:
>
> * Pull this backport nomination. No svn_fs_verify() in 1.7.x
>
> * Backport it, and backport the implementation in 1.7.x (x >= 1)
>
> * Backport svn_fs_verify() and its implementation in 1.7.0
I see what you see,
cmpil...@apache.org wrote on Wed, Jul 13, 2011 at 19:28:18 -:
> * r1146214
> Handle NULL inputs when stringifying svn_checksum_t.
> @@ -38,6 +39,7 @@ Candidate changes:
> Avoids segfaults.
> Votes:
> +1: danielsh
> + -0: cmpilato (problem is with callers, not implement
cmpil...@apache.org wrote on Wed, Jul 13, 2011 at 19:28:18 -:
> @@ -31,6 +31,7 @@ Candidate changes:
> Votes:
> +1: danielsh, rhuijben
> -1: stsp (no-op API change)
> + -1: cmpilato (no-op API change -- will reconsider if real utility is
> added)
Well, I've since implement
Stefan Sperling wrote on Wed, Jul 13, 2011 at 15:59:36 +0200:
> Hi all,
>
> while repairing some bad repository breakage we noticed that
> svnadmin verify does not check whether offsets mentioned in
> rep-cache.db point to valid reps.
>
> In some cases I had to append repaired reps after the chan
Daniel Shahaf writes:
> Or did you mean --record-only?
yes.
--
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com
[Stefan Sperling]
> So now 'svnadmin verify' runs fine. But when the rep-cache is
> referred to during commit we can still run into reps that are broken.
>
> Obviously we'll either have to fix up the rep-cache as well,
> or simply delete it. But the point is that 'svnadmin verify'
> does not comp
phi...@tigris.org wrote on Wed, Jul 13, 2011 at 08:49:18 -0700:
> http://subversion.tigris.org/issues/show_bug.cgi?id=3957
> Issue #|3957
> Summary|reintegrate fails with no subtree mergeinfo on source
>Component|subversion
> Versio
>From svn_fs_fs__initialize_txn_caches()
[[[
/* We don't support caching for concurrent transactions in the SAME
* FSFS session. Maybe, you forgot to clean POOL. */
if (ffd->txn_dir_cache != NULL || ffd->concurrent_transactions)
{
ffd->txn_dir_cache = NULL;
ffd->concurrent_t
Daniel Shahaf wrote on Wed, Jul 13, 2011 at 17:11:34 +0300:
> Stefan Sperling wrote on Wed, Jul 13, 2011 at 15:59:36 +0200:
> > while repairing some bad repository breakage we noticed that
> > svnadmin verify does not check whether offsets mentioned in
> > rep-cache.db point to valid reps.
>
> Doe
Does implementing this require API changes? (extending the fs-loader.h
vtables)
There is currently no svn_fs_verify() API.
Stefan Sperling wrote on Wed, Jul 13, 2011 at 15:59:36 +0200:
> Hi all,
>
> while repairing some bad repository breakage we noticed that
> svnadmin verify does not check w
+1; we already validate reps given in node-revision headers.
(could you CC me on the issue?)
Stefan Sperling wrote on Wed, Jul 13, 2011 at 15:59:36 +0200:
> Hi all,
>
> while repairing some bad repository breakage we noticed that
> svnadmin verify does not check whether offsets mentioned in
> re
On Wed, Jul 13, 2011 at 08:21:28AM -0500, Hyrum K Wright wrote:
> Thanks for the hard work, and there will be a release process quiz next week.
Oh, now you're telling me! Can you please reiterate what questions
will be on the quiz? Cause I wasn't listening at all.
Hi all,
while repairing some bad repository breakage we noticed that
svnadmin verify does not check whether offsets mentioned in
rep-cache.db point to valid reps.
In some cases I had to append repaired reps after the changed-paths
section in revision files and leave broken reps in place to avoid
As promised, I created the 1.7.x branch in r1145993. I hope to have a
beta from the branch posted in a couple of hours. A couple of notes:
This probably goes without saying, but the branch is a release
stabilization branch, so changed which are merged to the branch need
to be reviewed an voted o
On Wed, Jul 13, 2011 at 14:40, Philip Martin wrote:
> i...@apache.org writes:
>
>> Author: ivan
>> Date: Wed Jul 13 08:07:53 2011
>> New Revision: 1145901
>>
>> URL: http://svn.apache.org/viewvc?rev=1145901&view=rev
>> Log:
>> * subversion/libsvn_ra_serf/update.c
>> (end_report): Simplify condit
i...@apache.org writes:
> Author: ivan
> Date: Wed Jul 13 08:07:53 2011
> New Revision: 1145901
>
> URL: http://svn.apache.org/viewvc?rev=1145901&view=rev
> Log:
> * subversion/libsvn_ra_serf/update.c
> (end_report): Simplify condition a bit.
>
> Modified:
> subversion/trunk/subversion/libsv
"Bert Huijben" writes:
>> -Original Message-
>> From: Noorul Islam K M [mailto:noo...@collab.net]
>> Sent: woensdag 13 juli 2011 9:38
>> To: Subversion
>> Subject: Recursive deletion of property throws unnecessary warning
>>
>>
>> Initial mail from Geoff Rowell is here.
>> http://svn.ha
> -Original Message-
> From: Noorul Islam K M [mailto:noo...@collab.net]
> Sent: woensdag 13 juli 2011 9:38
> To: Subversion
> Subject: Recursive deletion of property throws unnecessary warning
>
>
> Initial mail from Geoff Rowell is here.
> http://svn.haxx.se/dev/archive-2009-07/0191.s
Greg Stein writes:
> Heh. Call it "done" and changes come out of the woodwork. (gee, where
> have we seen that before?) ... there has been a good dozen changes
> today for minor cleanups, packaging, compiler fixes, etc. Nothing
> functional, thankfully (for a couple/three weeks, actually).
This
Initial mail from Geoff Rowell is here.
http://svn.haxx.se/dev/archive-2009-07/0191.shtml
He said:
- Subversion 1.6.3 on Windows
When recursively deleting a property from a working copy, an unnecessary
warning is thrown because the property isn't defined on the root working
copy folder.
55 matches
Mail list logo
