Skip to content

fix: Modified lldb command to fix remove_weak_ref issue #366

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

Merged
merged 1 commit into from
Jul 18, 2019
Merged

fix: Modified lldb command to fix remove_weak_ref issue #366

merged 1 commit into from
Jul 18, 2019

Conversation

jayb1239
Copy link

Modified lldb command to fix remove_weak_ref issue.

Modify lldb commant to fix remove_weak_ref issue
@shazron
Copy link

shazron commented Jan 8, 2019

what is the remove_weak_ref issue? I want to understand this change and I can't find exact docs on this

@shazron shazron changed the title Update ios-deploy.m fix: Modified lldb command to fix remove_weak_ref issue Jan 24, 2019
@shazron shazron added this to the 1.10.0 milestone Jul 17, 2019
@Ukalnins
Copy link

LLDB is built against python2. If it's run in an environment where the python3 and python2 get mixed up you get an error about these functions. I think this one was mostly fixed by fixing the environment variables in the system and I'm not sure if should be handled in this way as lldb comming with Xcode may not use system python or a compatible version in the future, as it ships it's own python. (And Apple has stated that they are deprecating it in the future).

@shazron
Copy link

shazron commented Jul 18, 2019

@Ukalnins I agree, but for now before this removal happens, this will do for now. I'll file a new issue regarding this.

btw this is from another contributor regarding this issue (not sure who is behind some anonymous Github names these days, it could be you for all I know!): https://stackoverflow.com/questions/47658596/strange-mixing-of-system-homebrew-python-with-lldb

@shazron shazron merged commit 38ecd67 into ios-control:master Jul 18, 2019
geekonion pushed a commit to geekonion/ios-deploy that referenced this pull request Jun 29, 2023
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.

3 participants