[libvirt-jenkins-ci PATCH 6/7] lcitool: Use cross_commands[] for all distros

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



It's nicer to use, and consistent with how we're building the native
part of the Dockerfile.

Signed-off-by: Andrea Bolognani <abologna@xxxxxxxxxx>
---
 guests/lcitool | 33 +++++++++++++++++++--------------
 1 file changed, 19 insertions(+), 14 deletions(-)

diff --git a/guests/lcitool b/guests/lcitool
index fb0219b..117e1a5 100755
--- a/guests/lcitool
+++ b/guests/lcitool
@@ -868,25 +868,30 @@ class Application:
         sys.stdout.write(script.format(**varmap))
 
         if cross_arch:
+            cross_commands = []
+
             # Intentionally a separate RUN command from the above
             # so that the common packages of all cross-built images
             # share a Docker image layer.
             if package_format == "deb":
-                sys.stdout.write(textwrap.dedent("""
-                    RUN export DEBIAN_FRONTEND=noninteractive && \\
-                        dpkg --add-architecture {cross_arch} && \\
-                        {package_manager} update && \\
-                        {package_manager} dist-upgrade -y && \\
-                        {package_manager} install --no-install-recommends -y dpkg-dev && \\
-                        {package_manager} install --no-install-recommends -y {cross_pkgs} && \\
-                        {package_manager} autoremove -y && \\
-                        {package_manager} autoclean -y
-                """).format(**varmap))
+                cross_commands.extend([
+                    "export DEBIAN_FRONTEND=noninteractive",
+                    "dpkg --add-architecture {cross_arch}",
+                    "{package_manager} update",
+                    "{package_manager} dist-upgrade -y",
+                    "{package_manager} install --no-install-recommends -y dpkg-dev",
+                    "{package_manager} install --no-install-recommends -y {cross_pkgs}",
+                    "{package_manager} autoremove -y",
+                    "{package_manager} autoclean -y",
+                ])
             elif package_format == "rpm":
-                sys.stdout.write(textwrap.dedent("""
-                    RUN {package_manager} install -y {cross_pkgs} && \\
-                        {package_manager} clean all -y
-                """).format(**varmap))
+                cross_commands.extend([
+                    "{package_manager} install -y {cross_pkgs}",
+                    "{package_manager} clean all -y",
+                ])
+
+            cross_script = "\nRUN " + (" && \\\n    ".join(cross_commands)) + "\n"
+            sys.stdout.write(cross_script.format(**varmap))
 
         if "pip_pkgs" in varmap:
             sys.stdout.write(textwrap.dedent("""
-- 
2.25.1





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux