Merge lp:~wgrant/launchpad/distroseries-source-format-selection-db into lp:launchpad/db-devel

[William Grant]

[William Grant (wgrant)]

Debian has very recently introduced new source package formats. While not yet supported by Soyuz, it is possible that we will have to CP support during between 3.1.10 and 3.1.11. To make this less impossible, this new table should be added for 3.1.10.

SourcePackageFormatSelection defines which SourcePackageFormats (one of '1.0', '3.0 (quilt)' and '3.0 (native)' -- not to be confused with the current SourcePackageFormat enum which is to be renamed) are allowed to be uploaded to or copied to each series. This is required because the versions of dpkg in releases before Karmic simply do not support the 3.0 formats properly.

The name of the table is in line with ComponentSelection and SectionSelection, and Julian appears to have no objections to it.

[Stuart Bishop (stub)]

Looks good. Patch number patch-2207-08-0.sql.

Can land with jml's approval.

review: Approve (db)

[Jonathan Lange (jml)]

Thanks William.

If I understand correctly, this patch says "different releases of a distro (Debian or Ubuntu) support different source package formats", and "source package format support is determined primarily by distro series".

If these two statements are true, I'm happy for this patch to land. I'd really like some sort of external reference to back it up though. (How many times can I use "Citation needed" outside of a wikipedia article before it becomes trite?)

review: Approve (db)

[William Grant (wgrant)]

> Thanks William.
>
> If I understand correctly, this patch says "different releases of a distro
> (Debian or Ubuntu) support different source package formats", and "source
> package format support is determined primarily by distro series".

Those two statements are correct.

> If these two statements are true, I'm happy for this patch to land. I'd really
> like some sort of external reference to back it up though. (How many times can
> I use "Citation needed" outside of a wikipedia article before it becomes
> trite?)

The supported source formats are technically defined by the version of dpkg in
the series, however on top of that it's also a matter of distro policy.

I'm not sure of a better way to do it. This is also how dak does it, but if you
have alternative ideas I would love to know them.

[Julian Edwards (julian-edwards)]

Looks fine. On overnight reflection I might have preferred DistroseriesSourcePackageFormat but that's gotta be bad for my wrists :)

review: Approve

[Michael Nelson (michael.nelson)]

review: Approve (release-critical)

=== modified file 'database/schema/comments.sql'
--- database/schema/comments.sql	2009-10-26 13:48:04 +0000
+++ database/schema/comments.sql	2009-11-03 05:39:14 +0000
@@ -2269,3 +2269,9 @@
 COMMENT ON TABLE FlatPackagesetInclusion IS 'In order to facilitate the querying of set-subset relationships an expanded or flattened representation of the set-subset hierarchy is provided by this table.';
 COMMENT ON COLUMN FlatPackagesetInclusion.parent IS 'The package set that is (directly or indirectly) including a subset.';
 COMMENT ON COLUMN FlatPackagesetInclusion.child IS 'The package set that is being included as a subset.';
+
+-- SourcePackageFormatSelection
+COMMENT ON TABLE SourcePackageFormatSelection IS 'Allowed source package formats for a given distroseries.';
+COMMENT ON COLUMN SourcePackageFormatSelection.distroseries IS 'Refers to the distroseries in question.';
+COMMENT ON COLUMN SourcePackageFormatSelection.format IS 'The SourcePackageFormat to allow.';
+

=== added file 'database/schema/patch-2207-08-0.sql'
--- database/schema/patch-2207-08-0.sql	1970-01-01 00:00:00 +0000
+++ database/schema/patch-2207-08-0.sql	2009-11-03 05:39:14 +0000
@@ -0,0 +1,20 @@
+-- Copyright 2009 Canonical Ltd.  This software is licensed under the
+-- GNU Affero General Public License version 3 (see the file LICENSE).
+
+SET client_min_messages=ERROR;
+
+CREATE TABLE sourcepackageformatselection (
+  id serial PRIMARY KEY,
+  distroseries integer NOT NULL
+    CONSTRAINT sourceformatselection__distroseries__fk
+    REFERENCES distroseries,
+  format integer NOT NULL,
+  CONSTRAINT sourceformatselection__distroseries__format__key
+    UNIQUE (distroseries, format)
+);
+
+-- Allow all series to accept format 1.0 by default.
+INSERT INTO sourcepackageformatselection (distroseries, format)
+  SELECT id, 0 AS format FROM distroseries;
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2207, 08, 0);