-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve postinstall script resilience #69
Conversation
* Allow postinstall scripts to be executed with any Python interpreter (not just the deployed base runtime interpreter) * Generate layer config file as part of layers * Use a common postinstall script in all layers * Generate the deployed `sitecustomize.py` file from the layer config in the postinstall script * Add unit tests for the common postinstall script Closes #66. Implements initial steps towards #19.
Update expected test output files from automated/expected-output/20241105-113208-d995f3c (source PR: #69)
Need to avoid dependency on
|
Latest Windows failures are due to the fake "absolute" paths in the postinstall tests lacking a drive letter, so they're not considered absolute on Windows. I also worked out some new checks to add to the
|
Update expected test output files from automated/expected-output/20241107-075720-9b79f96 (source PR: #69)
Even with the new build and deployed environment sys.path checks passing, there appears to still be a problem with the layered environments not correctly picking up the items that are already installed in the lower layers. |
There are some relative paths that are not being correctly resolved (they're being resolved relative to the environment path rather than the base build path) |
Update expected test output files from automated/expected-output/20241107-105447-a10d4bc (source PR: #69)
Update expected test output files from automated/expected-output/20241107-105447-a10d4bc (source PR: #69) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
sitecustomize.py
file from the layer config in the postinstall scriptCloses #66. Implements initial steps towards #19.