Merge ~mwhudson/ubuntu-cdimage:preinstalled-certainty into ubuntu-cdimage:main

Proposed by Michael Hudson-Doyle
Status: Merged
Merged at revision: c25c334ec52c6afd0132071b6a921991bcd987f5
Proposed branch: ~mwhudson/ubuntu-cdimage:preinstalled-certainty
Merge into: ubuntu-cdimage:main
Diff against target: 35 lines (+11/-11)
1 file modified
lib/cdimage/build.py (+11/-11)
Reviewer Review Type Date Requested Status
Steve Langasek Approve
Review via email: mp+455253@code.launchpad.net

Commit message

assert that we do not run any preinstalled images through d-i

we do not build any images where live_fs_only(config) is false and
config["CDIMAGE_PREINSTALLED"] is true, and we sure as heck don't want
to start doing that by accident

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/lib/cdimage/build.py b/lib/cdimage/build.py
2index 503102e..ea9c507 100644
3--- a/lib/cdimage/build.py
4+++ b/lib/cdimage/build.py
5@@ -712,20 +712,20 @@ def build_image_set_locked(config, options):
6 elif is_live_fs_only(config):
7 build_livecd_base(config)
8 else:
9- if not config["CDIMAGE_PREINSTALLED"]:
10- log_marker("Germinating")
11- germination = Germination(config)
12- germination.run()
13+ assert not config["CDIMAGE_PREINSTALLED"]
14
15- log_marker("Generating new task lists")
16- germinate_output = germination.output(config.project)
17- germinate_output.write_tasks()
18+ log_marker("Germinating")
19+ germination = Germination(config)
20+ germination.run()
21
22- log_marker("Checking for other task changes")
23- germinate_output.update_tasks(date)
24+ log_marker("Generating new task lists")
25+ germinate_output = germination.output(config.project)
26+ germinate_output.write_tasks()
27
28- if (config["CDIMAGE_LIVE"] or config["CDIMAGE_SQUASHFS_BASE"] or
29- config["CDIMAGE_PREINSTALLED"]):
30+ log_marker("Checking for other task changes")
31+ germinate_output.update_tasks(date)
32+
33+ if (config["CDIMAGE_LIVE"] or config["CDIMAGE_SQUASHFS_BASE"]):
34 log_marker("Downloading live filesystem images")
35 download_live_filesystems(config)
36

Subscribers

People subscribed via source and target branches