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

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

Hide file tree

Showing 18 changed files with 47 additions and 29 deletions.
diff --git a/.github/workflows/py.yml b/.github/workflows/py.yml
@@ -24,7 +24,7 @@ jobs:
     uses: ./.github/workflows/common.yml
     with:
       workdir: py
-      format-pkgs: python3.11-venv
+      format-pkgs: python3.11-venv black
       build-pkgs: python3.11-venv
       cache-paths: |
         py/venv

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
@@ -3,24 +3,27 @@ set -eu
 
 cd $(dirname $0)
 
-source py_env.sh
+BUILDDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)-build
+VENVDIR=${BUILD_ROOT:-build}/$(basename $(pwd))-$(echo $(basename $0) | sed 's/build_*\(.*\).sh/\1/')-$(uname)-$(uname -m)-venv
 
-BUILD_DIR=build
-CORES=$(python3 -c 'import os; print(os.cpu_count() or 0)')
-
-if [ -t "$CORES" ]; then
-	echo "Failed to find core count." 1>&2
-	CORES=1
-else
-	echo "Found $CORES cores." 1>&2
+if [ ! -d $VENVDIR ]; then
+	python3 -m venv $VENVDIR
+	$VENVDIR/bin/pip install pysdl2 pysdl2-dll mypy black Cython==3.0.0a11
 fi
+source $VENVDIR/bin/activate
 
-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 4
 
 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" \$*
+source $VENVDIR/bin/activate
+PYTHONPATH="$BUILDDIR/lib/" exec python3 "$BUILDDIR/scripts/main.py" \$*
 EOD
-chmod +x rosettaboy-release
+chmod +x rosettaboy-release
diff --git a/pxd/format.sh b/pxd/format.sh
@@ -3,6 +3,4 @@ set -eu
 
 cd $(dirname $0)
 
-source py_env.sh
-
 exec black src/*.py
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)
+            }
         }
     }
 }