Skip to content

Commit b082ae8

Browse files
author
Greger Stolt Nilsen
committed
Add option to manually name image.
1 parent 1b1de85 commit b082ae8

File tree

2 files changed

+11
-6
lines changed

2 files changed

+11
-6
lines changed

emu/containers/emulator_container.py

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,12 +30,11 @@ class EmulatorContainer(DockerContainer):
3030
"""
3131
NO_METRICS_MESSAGE = "No metrics are collected when running this container."
3232

33-
def __init__(
34-
self, emulator, system_image_container, repository=None, metrics=False, extra=""
35-
):
33+
def __init__(self, emulator, system_image_container, repository=None, metrics=False, extra="", name=None):
3634
self.emulator_zip = AndroidReleaseZip(emulator)
3735
self.system_image_container = system_image_container
3836
self.metrics = metrics
37+
self.name = name
3938

4039
if type(extra) is list:
4140
extra = " ".join([f'"{s}"' for s in extra])
@@ -100,6 +99,9 @@ def write(self, dest):
10099
self.emulator_zip.extract(os.path.join(dest, "emu"))
101100

102101
def image_name(self):
102+
if self.name:
103+
return self.name
104+
103105
name = "{}-{}-{}".format(
104106
self.props["ro.build.version.sdk"],
105107
self.props["qemu.short_tag"],

emu/emu_docker.py

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,8 @@ def create_docker_image(args):
7575

7676
emuzip = [args.emuzip]
7777
if emuzip[0] in ["stable", "canary", "all"]:
78-
emuzip = [x.download() for x in emu_downloads_menu.find_emulator(emuzip[0])]
78+
emuzip = [x.download()
79+
for x in emu_downloads_menu.find_emulator(emuzip[0])]
7980
elif re.match(r"\d+", emuzip[0]):
8081
# We must be looking for a build id
8182
logging.info("Treating %s as a build id", emuzip[0])
@@ -103,8 +104,7 @@ def create_docker_image(args):
103104
continue
104105

105106
emu_docker = EmulatorContainer(
106-
emulator, sys_docker, args.repo, cfg.collect_metrics(), args.extra
107-
)
107+
emulator, sys_docker, args.repo, cfg.collect_metrics(), args.extra, args.name)
108108
emu_docker.build(Path(args.dest) / "emulator")
109109

110110
if args.start:
@@ -255,6 +255,9 @@ def main():
255255
create_parser.add_argument(
256256
"--sys", action="store_true", help="Process system image layer only."
257257
)
258+
create_parser.add_argument(
259+
"--name", help="Name to give image when pushed.", default=None
260+
)
258261
create_parser.set_defaults(func=create_docker_image)
259262

260263
create_inter = subparsers.add_parser(

0 commit comments

Comments
 (0)