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

The tns debug ios command hangs on launch screen #2225

Closed
vchimev opened this issue Nov 14, 2016 · 10 comments
Closed

The tns debug ios command hangs on launch screen #2225

vchimev opened this issue Nov 14, 2016 · 10 comments
Assignees
Milestone

Comments

@vchimev
Copy link

vchimev commented Nov 14, 2016

The tns debug ios command hangs on the launch screen and the Resources tab does not fill in with the files of the application

Setup:
Xcode 8.1
{N} CLI 2.4.0

@vchimev
Copy link
Author

vchimev commented Nov 14, 2016

It turned out that the python package six is required in order LLDB to operate properly. It could be installed as follows:

pip install six

@vchimev
Copy link
Author

vchimev commented Nov 16, 2016

Perhaps we need to add verification of the six package in the tns doctor, as well as include it into the setup script?

@KristinaKoeva, @jasssonpet, @rosen-vladimirov, others, what do you think?

@rosen-vladimirov
Copy link
Contributor

In case there's no other way of bypassing the missing six python package, we should definitely add it to doctor.
Does this occur only when XCode 8.x is used?

@jasssonpet
Copy link
Contributor

jasssonpet commented Nov 17, 2016

The six module is installed by default in /System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python.

The issue should only happen if the user is using a non-default Python version (e.g. installed from Brew).

@tsonevn
Copy link
Contributor

tsonevn commented Dec 5, 2016

Can we add this package to setup script?

@tsonevn tsonevn added this to the 2.6.0 (TBD) milestone Dec 5, 2016
@pkoleva pkoleva modified the milestone: 2.6.0 (TBD) Jan 26, 2017
@pkoleva pkoleva modified the milestone: 3.0.0-RC Feb 27, 2017
@yyosifov yyosifov self-assigned this Mar 14, 2017
@pkoleva pkoleva added the bug label Mar 30, 2017
@toddanglin
Copy link

Just encountered the new warning about 'six' package with 3.0 RC. Never had a problem (as far as I know) in the past. Wondering if the new check is accurate...

If the check is accurate and 'six' is missing, suggest that the official documentation page provide info for installing/configuring this correctly. By default on Mac, the pip command is not available.

@pkoleva
Copy link
Contributor

pkoleva commented Apr 3, 2017

Hey, @toddanglin, we encountered an issue with debugging on iOS on our machines, where we have custom python installed with brew, so we proposed to have such a check. By default in the system python in OSX 'six' is installed, so you don't have to do anything to get it.

About pip - it isn't installed in most of python versions, so we expect people that installed custom python to be aware of how to get pip and use it since this is the most commonly used package manager.

About the accuracy of out check in tns doctor could you verify you have 'six' installed and you still get the message you don't by running:

python
import six

and then run tns doctor.

Thanks :)

@toddanglin
Copy link

Well...as a result of this message, I installed python via brew and ran pip install six...which made the tns doctor warning go away. :)

So...if you're suggesting OS X should not need to perform these steps, perhaps there is some way to improve the check so that the warning is not displayed on systems that do not need to run pip install six? If that's not possible, we should at least document this nuance in the "official documentation" page that the CLI opens when tns doctor finds a problem.

@dtopuzov
Copy link
Contributor

Hi @toddanglin

Missing six may cause issues when running or on some real iOS devices (it looks it is device/iOS version dependent), but unfortunately we can't predict what iOS device user can connect to its mac host, so we explicitly check if six is available.

By default it is available, so we don't expect a lot of cases when users will hit this warning.
However may be we should add some notes to documentation for debugging.

@toddanglin
Copy link

Yeah, at the very least, improving the tns doctor detail OR the documentation that the CLI links to when there are installation errors would help. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

8 participants