add linux32 to i686 builds (#338)

* add linux32 to i686 builds * Add test. Fix bug with space. * Apply suggestions from code review apply suggestions Co-authored-by: Joe Rickerby <joerick@mac.com> * change uname to platform.machine Co-authored-by: Joe Rickerby <joerick@mac.com>
2020-05-04 17:06:47 +01:00
parent 328a6714e9
commit 3f3effe25d
2 changed files with 13 additions and 1 deletions
@@ -109,6 +109,7 @@ def build(options: BuildOptions):
        container_name = 'cibuildwheel-{}'.format(uuid.uuid4())

        try:
+            shell_cmd = ['linux32', '/bin/bash'] if platform_tag.endswith("i686") else ['/bin/bash']
            call(['docker', 'create',
                  '--env', 'CIBUILDWHEEL',
                  '--name', container_name,
@@ -136,7 +137,7 @@ def build(options: BuildOptions):
                        )

                call(
-                    ['docker', 'exec', '-i', container_name, '/bin/bash'],
+                    ['docker', 'exec', '-i', container_name] + shell_cmd,
                    universal_newlines=True,
                    input='''
                        # give xtrace output an extra level of indent inside docker