respond to review

- switch cases on `PYTHON_VARIANT`
- remove `minor_version` by using `python3`
- remove runtime-only asan options

Author: lucascolley

Tools/pixi-packages/asan/recipe.yaml

@@ -1,7 +1,6 @@
 context:
   # XXX: keep up to date
   version: "3.15"
-  minor_version: "3.15"
 
 package:
   name: python
@@ -17,8 +16,7 @@ build:
   script:
     file: ../build.sh
     env:
-      MINOR_VERSION: ${{ minor_version }}
-      PYTHON_ASAN: "1"
+      PYTHON_VARIANT: "asan"
 
 # XXX: maybe not all requirements necessary
 # derived from https://github.com/conda-forge/python-feedstock/blob/main/recipe/meta.yaml

Tools/pixi-packages/build.sh

@@ -1,10 +1,11 @@
 #!/bin/bash
 
-if [[ "${PYTHON_ASAN}" == 1 ]]; then
+if [[ "${PYTHON_VARIANT}" == "asan" ]]; then
     echo "BUILD TYPE: ASAN"
     BUILD_DIR="../build_asan"
     CONFIGURE_EXTRA="--with-address-sanitizer"
-    export ASAN_OPTIONS="detect_leaks=0:symbolize=1:strict_init_order=true:allocator_may_return_null=1:use_sigaltstack=0"
+    export PYTHON_ASAN="1"
+    export ASAN_OPTIONS="strict_init_order=true"
 else
     echo "BUILD TYPE: DEFAULT"
     BUILD_DIR="../build"
@@ -28,7 +29,7 @@ fi
 touch configure-done
 
 make -j"${CPU_COUNT}" install
-ln -sf "${PREFIX}/bin/python${MINOR_VERSION}" "${PREFIX}/bin/python"
+ln -sf "${PREFIX}/bin/python3" "${PREFIX}/bin/python"
 
 # https://github.com/prefix-dev/rattler-build/issues/2012
 if [[ ${OSTYPE} == "darwin"* ]]; then

Tools/pixi-packages/default/recipe.yaml

@@ -1,7 +1,6 @@
 context:
   # XXX: keep up to date
   version: "3.15"
-  minor_version: "3.15"
 
 package:
   name: python
@@ -16,8 +15,6 @@ build:
       - "*.o"
   script:
     file: ../build.sh
-    env:
-      MINOR_VERSION: ${{ minor_version }}
 
 # XXX: maybe not all requirements necessary
 # derived from https://github.com/conda-forge/python-feedstock/blob/main/recipe/meta.yaml