[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

RE: svn commit: r1102843 - /subversion/trunk/subversion/libsvn_wc/wc-metadata.sql

From: Bert Huijben <bert_at_qqmail.nl>
Date: Sat, 14 May 2011 08:28:00 +0200

> -----Original Message-----
> From: Bert Huijben [mailto:bert_at_qqmail.nl]
> Sent: zaterdag 14 mei 2011 8:06
> To: Greg Stein; dev_at_subversion.apache.org
> Subject: RE: svn commit: r1102843 -
> /subversion/trunk/subversion/libsvn_wc/wc-metadata.sql
>
> The format 20 bump introduces a foreign key (now) which breaks upgrades
> if you have md5 checksums.
>
> When we originally bumped to 20 this constraint wasn't there yet (added
> in January as 'small' change without impact.
>
> danielsh reported a broken update caused by this change on IRC.

------------------------------------------------------------------------
r1051475 | julianfoad | 2010-12-21 13:45:31 +0100 (di, 21 dec 2010) | 34
lines

Enforce in the WC DB schema that a pristine text checksum always references
a pristine text that is in the store. To accord with this, change the
1.6-to-1.7 upgrade code so that it populates the pristine store before the
NODES table, and so that it always writes the SHA-1 checksum (not the MD-5)
into the NODES table.

The assertions in write_entry are a bit ugly and in one case incomplete.

* subversion/libsvn_wc/wc-metadata.sql
  (ACTUAL_NODE): Declare 'older_checksum', 'left_checksum', 'right_checksum'
    as referencing a pristine text.
  (NODES): Declare 'checksum' as referencing a pristine text.
  (format: 99): Note these changes.

=============================================

This adds a validation that wasn't there before. And we can't fix this in
the format 28 bump, because the format 20 bump fails before we reach that.

        Bert
Received on 2011-05-14 08:29:25 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.