Skip to content

Commit 233aa11

Browse files
sbryngelsonclaude
andcommitted
Delete install/ on workspace path change to fix stale binaries
Updating .pc and .cmake config files with sed is insufficient — the MFC executables (simulation, pre_process, post_process) and static libraries have the old runner workspace path baked in at compile time. When the cache moves between runner instances, these binaries fail at runtime. Replace the install/ sed fix with rm -rf install/ so CMake re-links and re-installs all binaries with correct paths. The staging/ object files remain valid, so this is a re-link, not a full rebuild. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 4172553 commit 233aa11

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

.github/scripts/setup-build-cache.sh

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -82,10 +82,10 @@ if [ -f "$_workspace_marker" ]; then
8282
-o -name "*.make" -o -name "Makefile" \
8383
-o -name "build.ninja" \) \
8484
-exec sed -i "s|${_old_workspace}|$(pwd)|g" {} + 2>/dev/null || true
85-
# Update pkg-config and cmake config files in install/
86-
find "$_cache_dir/install" -type f \
87-
\( -name "*.pc" -o -name "*.cmake" \) \
88-
-exec sed -i "s|${_old_workspace}|$(pwd)|g" {} + 2>/dev/null || true
85+
# Compiled binaries have stale paths baked in — delete install/
86+
# so CMake rebuilds and re-installs them with correct paths.
87+
echo " Clearing install/ to force rebuild of binaries..."
88+
rm -rf "$_cache_dir/install"
8989
fi
9090
fi
9191
echo "$(pwd)" > "$_workspace_marker"

0 commit comments

Comments
 (0)