qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH 1/2] configure: Add 'mkdir build' check


From: Dinah B
Subject: Re: [PATCH 1/2] configure: Add 'mkdir build' check
Date: Thu, 16 Feb 2023 00:48:48 -0500

ping

Patch series: 20230208233111.398577-1-dinahbaum123@gmail.com/">https://lore.kernel.org/qemu-devel/20230208233111.398577-1-dinahbaum123@gmail.com/

-Dinah

On Wed, Feb 8, 2023 at 6:31 PM Dinah Baum <dinahbaum123@gmail.com> wrote:
QEMU configure script goes into an infinite error printing loop
when in read only directory due to 'build' dir never being created.

Checking if 'mkdir dir' succeeds prevents this error.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/321
---
 configure | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/configure b/configure
index 64960c6000..3b384914ce 100755
--- a/configure
+++ b/configure
@@ -31,10 +31,11 @@ then
         fi
     fi

-    mkdir build
-    touch $MARKER
+    if mkdir build
+    then
+        touch $MARKER

-    cat > GNUmakefile <<'EOF'
+        cat > GNUmakefile <<'EOF'
 # This file is auto-generated by configure to support in-source tree
 # 'make' command invocation

@@ -56,8 +57,12 @@ force: ;
 GNUmakefile: ;

 EOF
-    cd build
-    exec "$source_path/configure" "$@"
+        cd build
+        exec "$source_path/configure" "$@"
+    else
+        echo "ERROR: Unable to use ./build dir, try using a ../qemu/configure build"
+        exit 1
+    fi
 fi

 # Temporary directory used for files created while
--
2.30.2


reply via email to

[Prev in Thread] Current Thread [Next in Thread]