Skip to content
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

Inject into subprocesses of python #1058

Open
wants to merge 181 commits into
base: main
Choose a base branch
from

Conversation

plengauer
Copy link
Owner

@plengauer plengauer commented Jan 6, 2025

I needed to make several changes on to make this happen:

*) instrumenting the exec calls to inject
*) shebangs where incorrectly implemented, so with venvs changing paths and these paths being important on where to install pips, i needed to rethink shebangs. instead of doing some path magic, lets extract existing instrumentation and apply it to the path.
*) consequently, the renovate test fails because suddenly these spans have an absolute paths, which is more correct anyway
*) we need to mute all things pip because PYTHONPATH would make it install somewhere else or not install packages at all because its already available
*) the sizecustomize couldnt be used anymore because that fucked over PYTHONPATH even more

@plengauer plengauer self-assigned this Jan 6, 2025
@plengauer plengauer linked an issue Jan 6, 2025 that may be closed by this pull request
@plengauer plengauer marked this pull request as ready for review January 6, 2025 10:27
@plengauer plengauer enabled auto-merge (squash) January 6, 2025 10:27
@plengauer plengauer marked this pull request as draft January 7, 2025 09:42
auto-merge was automatically disabled January 7, 2025 09:42

Pull request was converted to draft

@plengauer plengauer marked this pull request as ready for review January 25, 2025 17:10
@plengauer plengauer enabled auto-merge (squash) January 25, 2025 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inject into subprocess of python
1 participant