Skip to content

Conversation

black-square
Copy link

  • Fixed a problem when the noninteractive mode printed application logs in a wrong order. One line could be inserted into the middle of another one. (The order is very important for us as we parse the logs and retrieve some data automatically)

  • Fixed a problem when iosdeploy in noninteractive mode (ran with XCode since version 8.0 only) started to add (lldb) prefixes before it printed a game log. It added this prefixes even in the middle of the line sometimes

  • Added handling of additional states

…ogs in a wrong order. One line could be inserted into the middle of another one. (The order is very important for us as we parse the logs and retrieve some data automatically)

- Fixed a problem when iosdeploy in noninteractive mode (ran with XCode since version 8.0 only) started to add `(lldb) ` prefixes before it printed a game log. It added this prefix even in the middle of the line sometimes
- Added handling of additional states
@shazron
Copy link

shazron commented Jan 26, 2017

Hi @black-square I've started to merge all the outstanding PRs before doing testing, but now this PR has some conflicts. Doesn't seem to be a huge conflict but can you help sort it out? Thanks!

…LogsOrder

# Conflicts:
#	src/scripts/lldb.py
@black-square
Copy link
Author

black-square commented Jan 26, 2017

@shazron Conflicts have been resolved (had 2 PRs changing the same file). Let me know if you find any issues during the testing. I can try to help to fix them.

@shazron shazron merged commit 7a3c02e into ios-control:master Jan 26, 2017
@shazron shazron added this to the 1.9.1 milestone Jan 26, 2017
@shazron
Copy link

shazron commented Jan 27, 2017

Published 1.9.1 https://github.com/phonegap/ios-deploy/releases/tag/1.9.1 - if any issues arise we can do a 1.9.2 (I did a basic test of deploying a signed app)

@black-square black-square deleted the IncorrectLogsOrder branch July 28, 2017 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants