address review comments

Author: smanolloff

.github/workflows/rebuildDependencies.yml

@@ -225,6 +225,7 @@ jobs:
 
     - name: Remove build requirements' binaries
       run: |
+        set -x
         graphFile='graph.json'
         hostPackagesFile='packages-host.json'
         buildPackagesFile='packages-build.json'
@@ -252,13 +253,13 @@ jobs:
 
         # Some are both build and host requirements and must not be removed
         # => exclude them from the list
-        cat $buildPackagesFile $hostPackagesFile | jq -s '
-          .[0]["Local Cache"] as $build_packages
-          | (.[1]["Local Cache"] | keys) as $host_package_names
-          | $build_packages
-          | with_entries(select(.key as $k | $host_package_names | index($k) == null)) as $packages
-          | {"Local Cache": $packages}
-        ' > $buildFilteredPackagesFile
+        python3 -c '
+        import json, sys
+        build_dict = json.load(open(sys.argv[1]))["Local Cache"]  # => {"pkg1": {...}, "pkg2": {...}, ...}
+        host_list = json.load(open(sys.argv[2]))["Local Cache"]  # => ["pkg1", "pkg2", ...]
+        filtered_dict = {k: v for k, v in build_dict.items() if k not in host_list}
+        json.dump({"Local Cache": filtered_dict}, open(sys.argv[3], "w"))
+        ' "$buildPackagesFile" "$hostPackagesFile" "$buildFilteredPackagesFile"
 
         conan remove --list "$buildFilteredPackagesFile" --confirm
 

conan_profiles/base/common

@@ -226,7 +226,7 @@ sqlite3/*:build_executable=False
 opusfile/*:http=False
 zstd/*:build_programs=False
 
-# Prevents libcurl dependency (fails to build due to system zlib)
+# Prevents onnxruntime->date->libcurl dependency
 # NOTE: use_system_tz_db is specific to date/3.0.1
 # Newer date recipes would need the tz_db=system option instead.
 date/*:use_system_tz_db=True