[python-wrapper] Re-populate REPLIT_LD_LIBRARY_PATH #410
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why
During the last refactor we lost the default fallback Python runtime, removing critical C++ libraries from the path. This was because the testing environment provided a default value, which did not adequately exercise the default case.
What changed
Mirror the setup between legacy and modern environment builders for clarity.
Test plan
This time, I tested by unsetting the value first...
then running the following (
main.py
):This replicates the lookup logic in
ld-audit
and verifies that we can find the library we need.Rollout
Describe any procedures or requirements needed to roll this out safely (or check the box below)