Merge ~mwhudson/ubuntu-cdimage:CDIMAGE_DVD-for-old-ubuntustudio into ubuntu-cdimage:main

Proposed by Michael Hudson-Doyle
Status: Merged
Approved by: Łukasz Zemczak
Approved revision: 942a5a76183eb40f0e5c01cd69bf177d0f24880c
Merged at revision: 942a5a76183eb40f0e5c01cd69bf177d0f24880c
Proposed branch: ~mwhudson/ubuntu-cdimage:CDIMAGE_DVD-for-old-ubuntustudio
Merge into: ubuntu-cdimage:main
Diff against target: 27 lines (+5/-1)
2 files modified
bin/cron.daily-live (+4/-0)
lib/cdimage/germinate.py (+1/-1)
Reviewer Review Type Date Requested Status
Łukasz Zemczak Approve
Review via email: mp+467624@code.launchpad.net

Commit message

Set CDIMAGE_DVD=1 for <= noble ubuntustudio images

Otherwise debian-cd does not name the ISO how the publication code
expects.

To post a comment you must log in.
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

This is looking correct, especially that the additional check for DVD was since long unnecessary. There would be bonus points for unit tests, but this is good anyway.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/bin/cron.daily-live b/bin/cron.daily-live
index a1c4e21..a6dfd82 100755
--- a/bin/cron.daily-live
+++ b/bin/cron.daily-live
@@ -52,6 +52,10 @@ def main():
52 image_type = "daily-live"52 image_type = "daily-live"
5353
54 config = Config(IMAGE_TYPE=image_type)54 config = Config(IMAGE_TYPE=image_type)
55 if project == "ubuntustudio" and series <= "noble":
56 # Without this, debian-cd does not name the ISO the way the
57 # publication code expects.
58 config["CDIMAGE_DVD"] = "1"
55 config["CDIMAGE_LIVE"] = "1"59 config["CDIMAGE_LIVE"] = "1"
56 if not build_image_set(config, options):60 if not build_image_set(config, options):
57 sys.exit(1)61 sys.exit(1)
diff --git a/lib/cdimage/germinate.py b/lib/cdimage/germinate.py
index 2cc50e2..a7dade0 100644
--- a/lib/cdimage/germinate.py
+++ b/lib/cdimage/germinate.py
@@ -202,7 +202,7 @@ class GerminateOutput:
202 self._seeds.append(line.split(":", 1)[0])202 self._seeds.append(line.split(":", 1)[0])
203203
204 def pool_seeds(self):204 def pool_seeds(self):
205 if not (self.config["CDIMAGE_DVD"] or self.config["CDIMAGE_LIVE"]):205 if not self.config["CDIMAGE_LIVE"]:
206 raise NoMasterSeeds("No seeds found for master task!")206 raise NoMasterSeeds("No seeds found for master task!")
207 project = self.config.project207 project = self.config.project
208208

Subscribers

People subscribed via source and target branches