{ "id": "168956", "key": "TIMOB-24958", "fields": { "issuetype": { "id": "1", "description": "A problem which impairs or prevents the functions of the product.", "name": "Bug", "subtask": false }, "project": { "id": "10153", "key": "TIMOB", "name": "Titanium SDK/CLI", "projectCategory": { "id": "10100", "description": "Titanium and related SDKs used in application development", "name": "Client" } }, "fixVersions": [ { "id": "18414", "description": "", "name": "Release 6.2.0", "archived": false, "released": true, "releaseDate": "2017-09-13" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2017-08-25T12:35:47.000+0000", "created": "2017-07-14T12:30:39.000+0000", "priority": { "name": "High", "id": "2" }, "labels": [], "versions": [ { "id": "18414", "description": "", "name": "Release 6.2.0", "archived": false, "released": true, "releaseDate": "2017-09-13" } ], "issuelinks": [ { "id": "55735", "type": { "id": "10003", "name": "Relates", "inward": "relates to", "outward": "relates to" }, "outwardIssue": { "id": "164806", "key": "TIMOB-24189", "fields": { "summary": "Windows: Add support for Visual Studio 2017", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "priority": { "name": "High", "id": "2" }, "issuetype": { "id": "2", "description": "A new feature of the product, which has yet to be developed.", "name": "New Feature", "subtask": false } } } } ], "assignee": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "updated": "2017-08-29T18:37:02.000+0000", "status": { "description": "The issue is considered finished, the resolution is correct. Issues which are closed can be reopened.", "name": "Closed", "id": "6", "statusCategory": { "id": 3, "key": "done", "colorName": "green", "name": "Done" } }, "components": [ { "id": "12642", "name": "Windows", "description": "Windows authoring support" } ], "description": "h5.Description\r\n\r\nHistorically the winappdeploycmd exe used for Windows 10 device detection, app installation and such would always be installed 'globally' in {{C:\\Program Files (x86)\\Windows Kits\\10}\\bin\\x86}}. Since 10.0.15063.0 this appears to be no longer the case and it is packaged with the SDK, at {{C:\\Program Files (x86)\\Windows Kits\\10\\bin\\10.0.15063.0\\x86}}\r\n\r\nOur tooling makes the assumption that winappdeploycmd is in this global space, we need to adapt the tooling to inspect an SDK for winappdeploycmd, I'm unsure whether if we detect one we assign it to the overall VS info, overriding one that exists or we keep a winappdeploycmd tied to an SDK.\r\n\r\n\r\nPosted a question here to see whether this will be the norm from here\r\n\r\nhttps://docs.microsoft.com/en-us/windows/uwp/packaging/install-universal-windows-apps-with-the-winappdeploycmd-tool", "attachment": [], "flagged": false, "summary": "Windows: Tooling does not detect winappdeploycmd when only SDK 10.0.15063.0 is installed", "creator": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "subtasks": [], "reporter": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "environment": "Windows 10 Pro\r\nVisual Studio 2017\r\nWindows 10 SDK 10.0.15063.0", "closedSprints": [ { "id": 939, "state": "closed", "name": "2017 Sprint 17 SDK", "startDate": "2017-08-13T23:08:29.095Z", "endDate": "2017-08-27T23:08:00.000Z", "completeDate": "2017-08-27T21:10:54.125Z", "originBoardId": 114 } ], "comment": { "comments": [ { "id": "426609", "author": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "body": "Considering this a requirement for us to document support for VS2017 as without it will require installing multiple SDKs to build (n)", "updateAuthor": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "created": "2017-08-14T11:31:41.000+0000", "updated": "2017-08-14T11:31:41.000+0000" }, { "id": "426617", "author": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "body": "[~kiguchi] I've started to work on this but just wanted some input from you. I'm trying to keep the existing functionality as best as possible.\r\n\r\nHere's what I'm doing so far\r\n\r\n- After the detection, if deployCmd is null [here|https://github.com/appcelerator/windowslib/blob/58823680ebad9d100d5a3226f4cbf8b24868fc29/lib/windowsphone.js#L175-L180], starting with the highest SDK work through them and check for {{C:\\\\Program Files (x86)\\\\Windows Kits\\\\10\\\\bin\\\\\\\\x86\\\\winappdeploycmd.exe}}. Taking the highest available\r\n\r\nThe problem I have is, if I'm using SDK 15063 and 14393 we will take the globally installed winappdeploycmd, when I feel like I should be taking the latest under 10.0.15063, what're your thoughts?", "updateAuthor": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "created": "2017-08-14T15:23:17.000+0000", "updated": "2017-08-14T15:23:17.000+0000" }, { "id": "426666", "author": { "name": "kiguchi", "key": "kota", "displayName": "Kota Iguchi", "active": false, "timeZone": "America/Los_Angeles" }, "body": "[~eharris] Yes, it sounds natural to me to select latest version of winappdeploycmd tooling when target sdk is 10.0. (y)", "updateAuthor": { "name": "kiguchi", "key": "kota", "displayName": "Kota Iguchi", "active": false, "timeZone": "America/Los_Angeles" }, "created": "2017-08-15T01:06:08.000+0000", "updated": "2017-08-15T01:06:08.000+0000" }, { "id": "426676", "author": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "body": "Windowslib PR: https://github.com/appcelerator/windowslib/pull/75 \r\n\r\nTravis is failing because of a super old node version, I think we should nuke travis, (pr https://github.com/appcelerator/windowslib/pull/73) as imo it's not worth the hassle. Maybe we can look into appveyor if we want some sort of configurable ci as a service (which imo we should do to allow us to test diff versions of VS against windowslb)", "updateAuthor": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "created": "2017-08-15T13:46:01.000+0000", "updated": "2017-08-15T13:46:01.000+0000" }, { "id": "426764", "author": { "name": "kiguchi", "key": "kota", "displayName": "Kota Iguchi", "active": false, "timeZone": "America/Los_Angeles" }, "body": "Merged. need to update titanium_mobile then.", "updateAuthor": { "name": "kiguchi", "key": "kota", "displayName": "Kota Iguchi", "active": false, "timeZone": "America/Los_Angeles" }, "created": "2017-08-16T04:08:02.000+0000", "updated": "2017-08-16T04:08:02.000+0000" }, { "id": "427097", "author": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "body": "Master: https://github.com/appcelerator/titanium_mobile/pull/9349\r\n6_2_X: https://github.com/appcelerator/titanium_mobile/pull/9350", "updateAuthor": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "created": "2017-08-23T17:44:13.000+0000", "updated": "2017-08-23T17:44:13.000+0000" }, { "id": "427155", "author": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "body": "[~smohammed] [~amukherjee] Do you have a machine available that only has VS2017 installed?", "updateAuthor": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "created": "2017-08-24T13:14:32.000+0000", "updated": "2017-08-24T13:14:32.000+0000" }, { "id": "427182", "author": { "name": "smohammed", "key": "smohammed", "displayName": "Samir Mohammed", "active": true, "timeZone": "America/Los_Angeles" }, "body": "[~eharris] The machine we use for VS2017 has both VS2017 and VS2015 installed.", "updateAuthor": { "name": "smohammed", "key": "smohammed", "displayName": "Samir Mohammed", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-08-24T16:28:12.000+0000", "updated": "2017-08-24T16:28:12.000+0000" }, { "id": "427250", "author": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "body": "Passing review as it's merged into the SDK now, it is present in latest 6_2_X 6.2.0.v20170824144806 but not latest master 7.0.0.v20170823165814\r\n\r\nVerification \r\n\r\n# Ensure you have the 10.0.15063 tooling installed with VS2017\r\n# Run appc ti info -t windows -o json\r\n# Check the results.windowsphone[10.0] object for the deployCmd value, it should list a file under the 10.0.15063 bin dir like {{\"C:\\\\Program Files (x86)\\\\Windows Kits\\\\10\\\\bin\\\\10.0.15063.0\\\\x86\\\\WinAppDeployCmd.exe\"}}", "updateAuthor": { "name": "eharris", "key": "eharris", "displayName": "Ewan Harris", "active": true, "timeZone": "Europe/Dublin" }, "created": "2017-08-25T12:35:47.000+0000", "updated": "2017-08-25T12:36:28.000+0000" }, { "id": "427372", "author": { "name": "amukherjee", "key": "amukherjee", "displayName": "Abir Mukherjee", "active": true, "timeZone": "America/Los_Angeles" }, "body": "Closing ticket as changes present in SDKs:\r\n6.2.0.v20170828180251\r\n7.0.0.v20170829103419", "updateAuthor": { "name": "amukherjee", "key": "amukherjee", "displayName": "Abir Mukherjee", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2017-08-29T18:36:56.000+0000", "updated": "2017-08-29T18:36:56.000+0000" } ], "maxResults": 10, "total": 10, "startAt": 0 } } }