From 08f74d7814a60e0c796c42f63d739f38cf625d40 Mon Sep 17 00:00:00 2001
From: Doug Hellmann <dhellmann@redhat.com>
Date: Tue, 4 Jun 2024 18:18:08 -0400
Subject: [PATCH] feature: log the output of the build command

When we run the step to build the wheel, tell pip to write a log file
in the directory above where the source is located. Using this
location ensures the file is preserved when bootstrap cleans up the
source tree, and makes it easy to collect all of the files if we are
building a sequence.
---
 e2e/test_bootstrap.sh              | 6 ++++++
 e2e/test_build_with_build_order.sh | 6 ++++++
 src/fromager/wheels.py             | 1 +
 3 files changed, 13 insertions(+)

diff --git a/e2e/test_bootstrap.sh b/e2e/test_bootstrap.sh
index b1db98b5..5067fb65 100755
--- a/e2e/test_bootstrap.sh
+++ b/e2e/test_bootstrap.sh
@@ -42,6 +42,12 @@ sdists-repo/downloads/pbr-6.0.0.tar.gz
 
 work-dir/build-order.json
 work-dir/constraints.txt
+
+work-dir/flit_core-3.9.0/build.log
+work-dir/pbr-6.0.0/build.log
+work-dir/setuptools-70.0.0/build.log
+work-dir/stevedore-5.2.0/build.log
+work-dir/wheel-0.43.0/build.log
 "
 
 pass=true
diff --git a/e2e/test_build_with_build_order.sh b/e2e/test_build_with_build_order.sh
index fba26af0..bb7a70e4 100755
--- a/e2e/test_build_with_build_order.sh
+++ b/e2e/test_build_with_build_order.sh
@@ -126,6 +126,12 @@ sdists-repo/downloads/setuptools-70.0.0.tar.gz
 sdists-repo/downloads/wheel-0.43.0.tar.gz
 sdists-repo/downloads/flit_core-3.9.0.tar.gz
 sdists-repo/downloads/pbr-6.0.0.tar.gz
+
+work-dir/flit_core-3.9.0/build.log
+work-dir/pbr-6.0.0/build.log
+work-dir/setuptools-70.0.0/build.log
+work-dir/stevedore-5.2.0/build.log
+work-dir/wheel-0.43.0/build.log
 "
 
 pass=true
diff --git a/src/fromager/wheels.py b/src/fromager/wheels.py
index dbe688e5..ac12382c 100644
--- a/src/fromager/wheels.py
+++ b/src/fromager/wheels.py
@@ -34,6 +34,7 @@ def default_build_wheel(ctx, build_env, extra_environ, req, sdist_root_dir):
             '--wheel-dir', ctx.wheels_build,
             '--no-deps',
             '--index-url', ctx.wheel_server_url,  # probably redundant, but just in case
+            '--log', sdist_root_dir.parent / 'build.log',
             sdist_root_dir,
         ]
         external_commands.run(cmd, cwd=dir_name, extra_environ=extra_environ)