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

[TIMOB-24485] Revert TIMOB-23664 - Skip Windows virtual adapter #99

Merged
merged 1 commit into from May 16, 2017
Merged

[TIMOB-24485] Revert TIMOB-23664 - Skip Windows virtual adapter #99

merged 1 commit into from May 16, 2017

Conversation

infosia
Copy link
Contributor

@infosia infosia commented Mar 24, 2017

TIMOB-24485

Revert #94 because this adapter may be used to communicate to emulators. From what I see on the latest Win 10 SDK & emulators this adapter is actually used. So I think under the variety of user's network configurations we should not skip specific interface automatically but should provide a way to select preferred interface to use - which is implemented in #96 (CLI-1075).

Copy link
Contributor

@garymathews garymathews left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CR: PASS

@infosia
Copy link
Contributor Author

infosia commented May 15, 2017

@garymathews Do you think we want to pass it to in-qe-testing, or we just merge this now?

@garymathews
Copy link
Contributor

I think we should mark for QE, since it could re-introduce https://jira.appcelerator.org/browse/TIMOB-23664

@ewanharris
Copy link
Contributor

FR PASSED!

Windows 10 Pro
Ti SDK:  6.1.0.v20170514022959
Appc Core: 6.2.2-master.9
Appc NPM: 4.2.9
Node: 6.10.2

Able to build a classic Titanium app for wp-emulator, ws-local, and wp-device targets. On wp-emulator, a user must pass in the ip of the Internal Switch

Alloy app connects to server and fails on a file, will be fixed in: TIMOB-24699

Added TIDOC-2875 to review documentation to ensure it correctly reflects usage Windows platform (i.e. wp-emulator requiring --liveview-ip

@ewanharris ewanharris merged commit f2e4e54 into tidev:master May 16, 2017
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.

None yet

3 participants