{ "id": "138690", "key": "TIMOB-19470", "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": "16925", "description": "WatchKit Support--all going into 5.0 now", "name": "Release 5.0.0", "archived": true, "released": true, "releaseDate": "2015-09-16" } ], "resolution": { "id": "1", "description": "A fix for this issue is checked into the tree and tested.", "name": "Fixed" }, "resolutiondate": "2015-09-09T16:13:06.000+0000", "created": "2014-10-29T09:09:59.000+0000", "priority": { "name": "Critical", "id": "1" }, "labels": [ "setup" ], "versions": [], "issuelinks": [], "assignee": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "updated": "2015-09-17T17:09:51.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": "10207", "name": "Tooling" } ], "description": "When I run {{ti setup}} and select *check* I get:\r\n\r\n{code}\r\nNode.js\r\n ✓ node up-to-date (v0.10.33)\r\n ★ npm new version v2.1.6 available! (currently v2.1.5)\r\n{code}\r\n\r\nHowever, on https://www.npmjs.org/package/npm and via {{npm info npm}} it has *2.1.5* as the last version. The {{versions}} array does have 2.1.6 but the CLI should look at {{version}} not {{versions}}.\r\n\r\nh2. Related code\r\n\r\nhttps://github.com/appcelerator/titanium/blob/master/lib/commands/setup.js#L516-L522\r\n\r\n{code}\r\nif (info && info.versions) {\r\n for (var i = 0, vers = info.versions, l = vers.length; i < l; i++) {\r\n if (appc.version.gt(vers[i], r.npm.latest)) {\r\n r.npm.latest = vers[i];\r\n }\r\n }\r\n}\r\n{code}\r\n\r\nh2. Proposed chage\r\n\r\nThe above lines with:\r\n\r\n{code}\r\nif (info && info.version) {\r\n r.npm.latest = info.version;\r\n}\r\n{code}", "attachment": [], "flagged": false, "summary": "CLI: ti setup doesn't properly select last available versions from npm", "creator": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "subtasks": [], "reporter": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "environment": "3.4.0.GA", "closedSprints": [ { "id": 487, "state": "closed", "name": "2015 Sprint 18 Tooling", "startDate": "2015-08-31T15:59:21.784Z", "endDate": "2015-09-12T00:59:00.000Z", "completeDate": "2015-09-14T23:09:59.452Z", "originBoardId": 121 } ], "comment": { "comments": [ { "id": "348668", "author": { "name": "rtlechuga", "key": "rtlechuga", "displayName": "Radamantis Torres-Lechuga", "active": false, "timeZone": "Asia/Dubai" }, "body": "[~fokkezb] is this still an issue?", "updateAuthor": { "name": "rtlechuga", "key": "rtlechuga", "displayName": "Radamantis Torres-Lechuga", "active": false, "timeZone": "Asia/Dubai" }, "created": "2015-04-06T23:08:45.000+0000", "updated": "2015-04-06T23:08:45.000+0000" }, { "id": "348741", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "[~rtlechuga] yes:\r\n\r\n{{ti setup}} > check\r\n\r\n{code}\r\n * npm new version v2.7.6 available! (currently v1.4.28)\r\n{code}\r\n\r\n{{npm info npm}}\r\n\r\n{code}\r\n version: '2.7.5',\r\n{code}", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2015-04-07T07:41:18.000+0000", "updated": "2015-04-07T07:41:18.000+0000" }, { "id": "362589", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "Verified this is a bug. @fokke is correct, we should be looking at {{version}}, not {{versions}}, then showing the proper label based on whether they are up-to-date, behind, or bleeding edge.\r\n\r\nThis is pretty low priority as this is just a report screen. Titanium CLI doesn't really do anything meaningful with NPM post installation.", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2015-09-08T22:21:42.000+0000", "updated": "2015-09-08T22:21:42.000+0000" }, { "id": "362636", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "Well, low prio except for that it now kind of suggests to upgrade to a new version we don't support:\r\n\r\n{code}\r\nNode.js\r\n ★ node new version v4.0.0 available! (currently v0.10.40)\r\n ★ npm new version v3.3.2 available! (currently v1.4.28)\r\n{code}\r\n\r\nWe might want to use [vendorDependencies|https://github.com/appcelerator/titanium_mobile/blob/master/package.json#L63] to determine what we show here.", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2015-09-09T08:42:31.000+0000", "updated": "2015-09-09T08:42:31.000+0000" }, { "id": "362660", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "Um, and how exactly do we \"not support\" NPM v3.3.2 or any other specific version? What compatibility issues are there? You see, once Titanium is installed, it doesn't care about NPM. It's never invoked. This screen is just a report. Titanium CLI doesn't have \"vendorDependencies\". That's a TiSDK thing and Titanium SDK doesn't reference NPM either.\r\n\r\nSo, all this ticket needs to do is report the latest stable.", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2015-09-09T14:52:45.000+0000", "updated": "2015-09-09T14:52:45.000+0000" }, { "id": "362663", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "PR: https://github.com/appcelerator/titanium/pull/232", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2015-09-09T15:08:58.000+0000", "updated": "2015-09-09T15:08:58.000+0000" }, { "id": "363592", "author": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "body": "What I mean is node v4.0.0 that we don't (yet) support. Since I think the message suggest people to upgrade to the latest, even now we've fixed that to be the latest stable that might mean they upgrade to a version we don't support yet.", "updateAuthor": { "name": "fokkezb", "key": "fokke", "displayName": "Fokke Zandbergen", "active": true, "timeZone": "Europe/Amsterdam" }, "created": "2015-09-11T13:58:25.000+0000", "updated": "2015-09-11T13:58:25.000+0000" }, { "id": "363596", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "I see your point. I'm fine with ripping Node.js and NPM out of \"ti setup check\". The intent is if someone has issues with their system, they can copy/paste the results of \"ti setup check\" and that would help us help them.", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2015-09-11T15:59:52.000+0000", "updated": "2015-09-11T15:59:52.000+0000" }, { "id": "363598", "author": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "body": "I'd be fine leaving it in but only showing what you have and not encouraging an upgrade", "updateAuthor": { "name": "ingo", "key": "ingo", "displayName": "Ingo Muschenetz", "active": true, "timeZone": "America/Los_Angeles" }, "created": "2015-09-11T16:09:21.000+0000", "updated": "2015-09-11T16:09:21.000+0000" }, { "id": "363599", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "That works too.", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2015-09-11T16:25:30.000+0000", "updated": "2015-09-11T16:25:30.000+0000" }, { "id": "363627", "author": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "body": "PR 2: https://github.com/appcelerator/titanium/pull/233", "updateAuthor": { "name": "cbarber", "key": "cbarber", "displayName": "Chris Barber", "active": true, "timeZone": "America/Chicago" }, "created": "2015-09-11T20:29:40.000+0000", "updated": "2015-09-11T20:29:40.000+0000" }, { "id": "364221", "author": { "name": "htbryant", "key": "htbryant", "displayName": "Harry Bryant", "active": true, "timeZone": "Europe/London" }, "body": "Verified fixed, running \" ti setup \" and selecting \" check \" displays versions without encouraging updates.\r\n\r\n\r\nTi.SDK: 5.0.0.v20150915230722\r\nAppc Studio: 4.3.0.201509140942\r\nAppc CLI: 5.0.0", "updateAuthor": { "name": "htbryant", "key": "htbryant", "displayName": "Harry Bryant", "active": true, "timeZone": "Europe/London" }, "created": "2015-09-17T17:09:45.000+0000", "updated": "2015-09-17T17:09:45.000+0000" } ], "maxResults": 16, "total": 16, "startAt": 0 } } }