Merge ~mwhudson/ubuntu-cdimage:no-ubuntustudio-dvd into ubuntu-cdimage:main

Proposed by Michael Hudson-Doyle
Status: Merged
Merged at revision: 531f9206b6e1689dc2cd54f0c0d6f8cdc265b826
Proposed branch: ~mwhudson/ubuntu-cdimage:no-ubuntustudio-dvd
Merge into: ubuntu-cdimage:main
Diff against target: 182 lines (+15/-82)
8 files modified
dev/null (+0/-48)
etc/cdimage-to-livecd-rootfs-map (+8/-7)
etc/crontab (+1/-1)
etc/livefs-launchpad (+2/-2)
etc/qa-products (+1/-0)
lib/cdimage/germinate.py (+2/-1)
lib/cdimage/livefs.py (+1/-7)
lib/cdimage/tests/test_livefs.py (+0/-16)
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Review via email: mp+466017@code.launchpad.net

Commit message

drop the 'dvd' image type

Ubuntu Studio for noble and earlier still uses the ubuntustudio-dvd
project in livecd-rootfs but is still a 'daily-live' image which means
it will be published to slightly different paths on ubuntu-cdimage.

There are probably some bits of tree.py that can be trimmed now but one
thing at a time...

Description of the change

To post a comment you must log in.
Revision history for this message
Steve Langasek (vorlon) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/bin/cron.dvd b/bin/cron.dvd
2deleted file mode 100755
3index 5566973..0000000
4--- a/bin/cron.dvd
5+++ /dev/null
6@@ -1,48 +0,0 @@
7-#! /usr/bin/python3
8-
9-# Copyright (C) 2013 Canonical Ltd.
10-# Author: Colin Watson <cjwatson@ubuntu.com>
11-
12-# This program is free software: you can redistribute it and/or modify
13-# it under the terms of the GNU General Public License as published by
14-# the Free Software Foundation; version 3 of the License.
15-#
16-# This program is distributed in the hope that it will be useful,
17-# but WITHOUT ANY WARRANTY; without even the implied warranty of
18-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
19-# GNU General Public License for more details.
20-#
21-# You should have received a copy of the GNU General Public License
22-# along with this program. If not, see <http://www.gnu.org/licenses/>.
23-
24-"""Build a set of DVD images."""
25-
26-from optparse import OptionParser
27-import os
28-import sys
29-
30-sys.path.insert(0, os.path.join(sys.path[0], os.pardir, "lib"))
31-
32-
33-def main():
34- from cdimage.build import build_image_set
35- from cdimage.config import Config
36-
37- parser = OptionParser("%prog")
38- parser.add_option(
39- "--live", default=False, action="store_true",
40- help="build live filesystems first")
41- options, _ = parser.parse_args()
42- config = Config(IMAGE_TYPE="dvd")
43- project = config.project
44- if project not in ("ubuntu", "ubuntustudio"):
45- print("unsupported configuration!")
46- sys.exit(1)
47- config["CDIMAGE_LIVE"] = "1"
48- config["CDIMAGE_DVD"] = "1"
49- if not build_image_set(config, options):
50- sys.exit(1)
51-
52-
53-if __name__ == "__main__":
54- main()
55diff --git a/etc/cdimage-to-livecd-rootfs-map b/etc/cdimage-to-livecd-rootfs-map
56index 0ab778c..a9e60e9 100644
57--- a/etc/cdimage-to-livecd-rootfs-map
58+++ b/etc/cdimage-to-livecd-rootfs-map
59@@ -1,10 +1,11 @@
60-# PROJECT IMAGE_TYPE DIST ARCH SUBARCH LIVEFS_PROJECT LIVEFS_CPUARCH LIVEFS_SUBARCH
61+# PROJECT IMAGE_TYPE DIST ARCH SUBARCH LIVEFS_PROJECT LIVEFS_CPUARCH LIVEFS_SUBARCH
62 # Put more specific matches first.
63 # Like etc/default-arches, "PROJECT" is in fact PROJECT[-SUBPROJECT]. "-" means 'empty'.
64
65-ubuntu-server daily-preinstalled * riscv64 unleashed ubuntu-cpc * sifive_fu540
66-ubuntu-server daily-preinstalled jammy riscv64 unmatched ubuntu-cpc * sifive_hifive_unmatched_fu740
67-ubuntu-server daily-preinstalled * * - ubuntu-cpc * generic
68-ubuntu-server daily-preinstalled * * * ubuntu-cpc * *
69-ubuntu-appliance * * * * ubuntu-core * *
70-livecd-base * * * * base * *
71+ubuntu-server daily-preinstalled * riscv64 unleashed ubuntu-cpc * sifive_fu540
72+ubuntu-server daily-preinstalled jammy riscv64 unmatched ubuntu-cpc * sifive_hifive_unmatched_fu740
73+ubuntu-server daily-preinstalled * * - ubuntu-cpc * generic
74+ubuntu-server daily-preinstalled * * * ubuntu-cpc * *
75+ubuntu-appliance * * * * ubuntu-core * *
76+livecd-base * * * * base * *
77+ubuntustudio * -noble * * ubuntustudio-dvd * *
78diff --git a/etc/crontab b/etc/crontab
79index 70ccb88..058ade5 100644
80--- a/etc/crontab
81+++ b/etc/crontab
82@@ -72,7 +72,7 @@ PATH=/srv/cdimage.ubuntu.com/bin:/usr/sbin:/usr/bin:/sbin:/bin
83 40 1 * * * for-project xubuntu cron.daily-live --live; SUBPROJECT=minimal for-project xubuntu cron.daily-minimal --live
84 41 1 * * * for-project edubuntu cron.daily-live --live; for-project edubuntu cron.daily-preinstalled --live
85 10 3 * * * for-project ubuntu-server cron.daily-preinstalled --live; SUBPROJECT=live for-project ubuntu-server cron.daily-live --live
86-27 18 * * * for-project ubuntustudio cron.dvd --live
87+27 18 * * * for-project ubuntustudio cron.daily-live --live
88 39 16 * * * for-project lubuntu cron.daily-live --live
89 24 22 * * * for-project ubuntukylin cron.daily-live --live
90 10 15 * * * for-project ubuntu-budgie cron.daily-live --live
91diff --git a/etc/livefs-launchpad b/etc/livefs-launchpad
92index 2bb71b6..bfcc98b 100644
93--- a/etc/livefs-launchpad
94+++ b/etc/livefs-launchpad
95@@ -15,13 +15,13 @@ ubuntu-server daily-preinstalled bionic- * ubuntu-cdimage/cpc
96 # Temporary experimental image-set, canary preinstalled images for custom arches
97 #ubuntu-server daily-canary jammy * ubuntu-cdimage/cpc
98 ubuntu-server-live daily-live bionic- * ubuntu-cdimage/ubuntu-server-live
99-ubuntustudio dvd * * ubuntu-cdimage/ubuntustudio
100+ubuntustudio daily-live * * ubuntu-cdimage/ubuntustudio
101 lubuntu-next daily-live * * ubuntu-cdimage/lubuntu-next
102 lubuntu daily-live * * ubuntu-cdimage/lubuntu
103 livecd-base livecd-base * * ubuntu-cdimage/livecd-base
104 ubuntu-base daily * * ubuntu-cdimage/ubuntu-base
105 ubuntu-core daily-live * * ubuntu-cdimage/ubuntu-core
106-ubuntu-core-desktop daily-live * * ubuntu-cdimage/ubuntu-core-desktop
107+ubuntu-core-desktop daily-live * * ubuntu-cdimage/ubuntu-core-desktop
108 ubuntu-appliance daily-live * * ubuntu-cdimage/ubuntu-core
109 ubuntu-core-system-image daily-preinstalled * * ubuntu-cdimage/ubuntu-core-system-image
110 ubuntukylin daily-live * * ubuntu-cdimage/ubuntukylin
111diff --git a/etc/qa-products b/etc/qa-products
112index c63af06..726c148 100644
113--- a/etc/qa-products
114+++ b/etc/qa-products
115@@ -90,6 +90,7 @@ Ubuntu Server Subiquity riscv64 ubuntu-server/live daily-live live-server
116
117 # Ubuntu Studio
118 Ubuntu Studio DVD amd64 ubuntustudio dvd dvd amd64 iso
119+Ubuntu Studio amd64 ubuntustudio daily-live dvd amd64 iso
120 Ubuntu Studio DVD i386 ubuntustudio dvd dvd i386 iso
121
122 # Ubuntu Kylin
123diff --git a/lib/cdimage/germinate.py b/lib/cdimage/germinate.py
124index 004e2a3..2cc50e2 100644
125--- a/lib/cdimage/germinate.py
126+++ b/lib/cdimage/germinate.py
127@@ -207,7 +207,8 @@ class GerminateOutput:
128 project = self.config.project
129
130 if project == "ubuntustudio":
131- yield "dvd"
132+ if self.config.series <= "noble":
133+ yield "dvd"
134 yield "ship-live"
135 elif project == "ubuntu-server":
136 yield "server-ship-live"
137diff --git a/lib/cdimage/livefs.py b/lib/cdimage/livefs.py
138index f361745..2300b6f 100644
139--- a/lib/cdimage/livefs.py
140+++ b/lib/cdimage/livefs.py
141@@ -129,13 +129,7 @@ def live_build_options(config, arch):
142
143
144 def live_project(config, arch):
145- liveproject = config.livefs_project_for_arch(arch)
146-
147- if config["CDIMAGE_DVD"]:
148- if config.project in ("ubuntu", "kubuntu", "ubuntustudio"):
149- liveproject += "-dvd"
150-
151- return liveproject
152+ return config.livefs_project_for_arch(arch)
153
154
155 def live_build_command(config, arch):
156diff --git a/lib/cdimage/tests/test_livefs.py b/lib/cdimage/tests/test_livefs.py
157index 10744a8..ccedce3 100644
158--- a/lib/cdimage/tests/test_livefs.py
159+++ b/lib/cdimage/tests/test_livefs.py
160@@ -188,22 +188,6 @@ class TestLiveProject(TestCase):
161 def test_project_livecd_base(self):
162 self.assertProjectEqual("base", "livecd-base", "dapper")
163
164- def test_ubuntu_dvd(self):
165- for series in all_series[7:]:
166- self.assertProjectEqual(
167- "ubuntu-dvd", "ubuntu", series.full_name, cdimage_dvd="1")
168-
169- def test_kubuntu_dvd(self):
170- for series in all_series[7:]:
171- self.assertProjectEqual(
172- "kubuntu-dvd", "kubuntu", series.full_name, cdimage_dvd="1")
173-
174- def test_ubuntustudio_dvd(self):
175- for series in all_series[15:]:
176- self.assertProjectEqual(
177- "ubuntustudio-dvd", "ubuntustudio", series.full_name,
178- cdimage_dvd="1")
179-
180 def test_ubuntu_appliance(self):
181 # We currently only support ubuntu-appliances for bionic (UC18)
182 self.assertProjectEqual(

Subscribers

People subscribed via source and target branches