Skip to content

Commit

Permalink
apacheGH-41119: [Archery][Packaging][CI] Avoid using --progress flag …
Browse files Browse the repository at this point in the history
…on Docker on Windows on archery (apache#41120)

### Rationale for this change

Windows wheels are currently failing due to the change on `ARCHERY_DEBUG=1` by default. This uses `--progress` on `docker build` which is not supported on Windows.

### What changes are included in this PR?

Do not use `--progress` on the Windows builds.

### Are these changes tested?

Yes on CI via archery.

### Are there any user-facing changes?
No
* GitHub Issue: apache#41119

Lead-authored-by: Raúl Cumplido <raulcumplido@gmail.com>
Co-authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
  • Loading branch information
2 people authored and vibhatha committed May 25, 2024
1 parent 1d0d59f commit d29d044
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions dev/archery/archery/docker/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@ def _build(service, use_cache):
self._execute_docker("buildx", "build", *args)
elif self.config.using_docker:
# better for caching
if self.config.debug:
if self.config.debug and os.name != "nt":
args.append("--progress=plain")
for k, v in service['build'].get('args', {}).items():
args.extend(['--build-arg', '{}={}'.format(k, v)])
Expand All @@ -324,7 +324,7 @@ def _build(service, use_cache):
])
self._execute_docker("build", *args)
else:
if self.config.debug:
if self.config.debug and os.name != "nt":
args.append("--progress=plain")
self._execute_compose("build", *args, service['name'])

Expand Down

0 comments on commit d29d044

Please sign in to comment.