[many] BUILD_ROOT environment var to minimise pollution in the source…

… repo
shish · Apr 8, 2024 · 1339ce3 · 1339ce3
1 parent 0c1a821
commit 1339ce3
Show file tree

Hide file tree

Showing 16 changed files with 43 additions and 18 deletions.
diff --git a/all.py b/all.py
@@ -259,6 +259,8 @@ def main() -> int:
                 sub = "release"
                 if match := re.match("rosettaboy-(.*)", runner):
                     sub = match.group(1)
+                if lang_runner.suffix == ".nimble":
+                    continue  # rosettaboy.nimble is detected as executable??
                 if not os.access(lang_runner, os.X_OK):
                     continue
                 if args.default and sub != "release":
@@ -286,6 +288,8 @@ def main() -> int:
                 sub = "release"
                 if match := re.match("rosettaboy-(.*)", runner):
                     sub = match.group(1)
+                if lang_runner.suffix == ".nimble":
+                    continue  # rosettaboy.nimble is detected as executable??
                 if not os.access(lang_runner, os.X_OK):
                     continue
                 if args.default and sub != "release":

diff --git a/c/build.sh b/c/build.sh
@@ -2,7 +2,7 @@
 set -eu
 
 cd $(dirname $0)
-BUILDDIR=build/release/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 cmake -DCMAKE_BUILD_TYPE=Release -B $BUILDDIR .
 cmake --build $BUILDDIR -j
 cp $BUILDDIR/rosettaboy-c ./rosettaboy-release
diff --git a/c/build_debug.sh b/c/build_debug.sh
@@ -2,7 +2,7 @@
 set -eu
 
 cd $(dirname $0)
-BUILDDIR=build/debug/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 cmake -DCMAKE_BUILD_TYPE=Debug -B $BUILDDIR .
 cmake --build $BUILDDIR -j
 cp $BUILDDIR/rosettaboy-c ./rosettaboy-debug
diff --git a/c/build_lto.sh b/c/build_lto.sh
@@ -2,7 +2,7 @@
 set -eu
 
 cd $(dirname $0)
-BUILDDIR=build/lto/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 cmake -DCMAKE_BUILD_TYPE=Release -DENABLE_LTO=On -B $BUILDDIR .
 cmake --build $BUILDDIR -j
 cp $BUILDDIR/rosettaboy-c ./rosettaboy-lto
diff --git a/cpp/build.sh b/cpp/build.sh
@@ -2,7 +2,7 @@
 set -eu
 
 cd $(dirname $0)
-BUILDDIR=build/release/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 cmake -DCMAKE_BUILD_TYPE=Release -B $BUILDDIR .
 cmake --build $BUILDDIR -j
 cp $BUILDDIR/rosettaboy-cpp ./rosettaboy-release
diff --git a/cpp/build_debug.sh b/cpp/build_debug.sh
@@ -2,7 +2,7 @@
 set -eu
 
 cd $(dirname $0)
-BUILDDIR=build/debug/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 cmake -DCMAKE_BUILD_TYPE=Debug -B $BUILDDIR .
 cmake --build $BUILDDIR -j
 cp $BUILDDIR/rosettaboy-cpp ./rosettaboy-debug
diff --git a/cpp/build_lto.sh b/cpp/build_lto.sh
@@ -2,7 +2,7 @@
 set -eu
 
 cd $(dirname $0)
-BUILDDIR=build/lto/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 cmake -DCMAKE_BUILD_TYPE=Release -DENABLE_LTO=On -B $BUILDDIR .
 cmake --build $BUILDDIR -j
 cp $BUILDDIR/rosettaboy-cpp ./rosettaboy-lto
diff --git a/pxd/build.sh b/pxd/build.sh
@@ -5,7 +5,7 @@ cd $(dirname $0)
 
 source py_env.sh
 
-BUILD_DIR=build
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)-build
 CORES=$(python3 -c 'import os; print(os.cpu_count() or 0)')
 
 if [ -t "$CORES" ]; then
@@ -15,12 +15,20 @@ else
 	echo "Found $CORES cores." 1>&2
 fi
 
-python3 setup.py build "$@" --build-base "$BUILD_DIR" --build-purelib "$BUILD_DIR" --build-lib "$BUILD_DIR" --build-scripts "$BUILD_DIR" --build-temp build_temp --parallel $CORES
+python3 setup.py build "$@" \
+	--build-base "$BUILDDIR/base" \
+	--build-purelib "$BUILDDIR/purelib" \
+	--build-lib "$BUILDDIR/lib" \
+	--build-scripts "$BUILDDIR/scripts" \
+	--build-temp "$BUILDDIR/temp" \
+	--parallel $CORES
 
 cat >rosettaboy-release <<EOD
 #!/usr/bin/env bash
 set -eu
-source "\$(dirname \$0)/py_env.sh"
-PYTHONPATH="\$(dirname \$0)/build/" exec python3 "\$(dirname \$0)/build/main.py" \$*
+cd "\$(dirname \$0)"
+source py_env.sh
+cd -
+PYTHONPATH="$BUILDDIR/lib/" exec python3 "$BUILDDIR/scripts/main.py" \$*
 EOD
-chmod +x rosettaboy-release
+chmod +x rosettaboy-release
diff --git a/pxd/py_env.sh b/pxd/py_env.sh
@@ -1,6 +1,6 @@
 # Instead of being run, this gets `source`d by `build.sh` and `format.sh`.
 
-BUILDDIR=venv/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)-venv
 PYIMPORT_EXIT=$(python3 -c 'import sdl2, black, mypy, cython' 1>&2; echo $?) # Use existing libraries (E.G. from system, or from Nix) if found.
 if [ $PYIMPORT_EXIT -eq 0 ] ; then
 	echo "Python packages found:"

diff --git a/py/py_env.sh b/py/py_env.sh
@@ -1,6 +1,6 @@
 # Instead of being run, this gets `source`d by `build.sh` and `format.sh`.
 
-BUILDDIR=$(dirname $0)/venv/$(uname)-$(uname -m)
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)-venv
 
 # Use existing libraries (E.G. from system, or from Nix) if found.
 if python3 -c 'import sdl2, black, mypy' 2>/dev/null ; then

diff --git a/rs/build.sh b/rs/build.sh
@@ -2,5 +2,7 @@
 set -eu
 
 cd $(dirname $0)
+BUILDDIR=${BUILD_ROOT:-target}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
+export CARGO_TARGET_DIR=$BUILDDIR
 cargo build --release
-cp ./target/release/rosettaboy-rs ./rosettaboy-release
+cp $BUILDDIR/release/rosettaboy-rs ./rosettaboy-release
diff --git a/rs/build_cranelift_debug.sh b/rs/build_cranelift_debug.sh
@@ -5,5 +5,5 @@ cd $(dirname $0)
 BUILDDIR=${BUILD_ROOT:-target}/$(basename $(pwd))-cranelift-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
 export CARGO_PROFILE_DEV_CODEGEN_BACKEND=cranelift
 export CARGO_TARGET_DIR=$BUILDDIR
-cargo +nightly build -Zcodegen-backend --release
+cargo +nightly build -Zcodegen-backend
 cp $BUILDDIR/debug/rosettaboy-rs ./rosettaboy-cranelift
diff --git a/rs/build_debug.sh b/rs/build_debug.sh
@@ -2,5 +2,7 @@
 set -eu
 
 cd $(dirname $0)
+BUILDDIR=${BUILD_ROOT:-target}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
+export CARGO_TARGET_DIR=$BUILDDIR
 cargo build
-cp ./target/debug/rosettaboy-rs ./rosettaboy-debug
+cp $BUILDDIR/debug/rosettaboy-rs ./rosettaboy-debug
diff --git a/rs/build_lto.sh b/rs/build_lto.sh
@@ -2,5 +2,7 @@
 set -eu
 
 cd $(dirname $0)
+BUILDDIR=${BUILD_ROOT:-target}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)
+export CARGO_TARGET_DIR=$BUILDDIR
 cargo build --profile release-lto
-cp ./target/release-lto/rosettaboy-rs ./rosettaboy-lto
+cp $BUILDDIR/release-lto/rosettaboy-rs ./rosettaboy-lto
diff --git a/rs/src/cart.rs b/rs/src/cart.rs
@@ -1,3 +1,5 @@
+#![allow(dead_code)]
+
 use crate::errors::UserException;
 use anyhow::{anyhow, Result};
 use num_enum::TryFromPrimitive;

diff --git a/rs/src/errors.rs b/rs/src/errors.rs
@@ -50,7 +50,12 @@ impl std::fmt::Display for UserException {
             UserException::UnsupportedCart(cart_type) => {
                 write!(f, "Unsupported cart type: {:?}", cart_type)
             }
-            _ => write!(f, "Unspecified user error: {:?}", self),
+            UserException::LogoChecksumFailed(bad_sum) => {
+                write!(f, "Logo checksum failed: {:?}", bad_sum)
+            }
+            UserException::HeaderChecksumFailed(bad_sum) => {
+                write!(f, "Header checksum failed: {:?}", bad_sum)
+            }
         }
     }
 }