[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-22 Thread reynaldo-tortoledo-hs
Github user reynaldo-tortoledo-hs commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-142457227 `npm install --save q && npm install --save nopt` Thanks ;-) --- If your project is set up for it, you can reply to this email and have your

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141538218 Nice of you to end the debate. And our answer is, update your platform, it is fixed in a newer update. --- If your project is set up for it, you can reply to this

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141528062 I reported the bug. You deal with it how you want. Anyone who uses 4.3.0 and follows instructions on numerous WWW sites are going to run into this.

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141525802 Not sure how you do software dev -- that's not how our releases work, mschartz - if a bug goes in, it goes in a new release with a new version. We don't alter a so

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141525374 Fix it in the earlier tag for iOS platform :) That's my suggestion. On Fri, Sep 18, 2015 at 11:20 AM, Shazron Abdullah wrote:

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141524921 Not sure what else we can do, it is fixed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141524412 The problem is fixed, you have to update, you should read about cordova cli and platform versions, cordova iOS 3.9.1 shouldn't break any plugin, cordova iOS 4.

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141519811 Doesn't Cordova 5 have breaking API changes? I found a number of plugins I was using on a project started crashing or failing to make callbacks to JavaScri

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141519174 Update your cordova to get the latest pinned cordova-ios platform 3.9.1 or you can add it explicitly ios@3.9.1 --- If your project is set up for it, you can reply

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141518970 Whichever iOS platform is installed by cordova 4.3.0 has the wrong "Q" case in the file. On Fri, Sep 18, 2015 at 10:43 AM, Julio César

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread jcesarmobile
Github user jcesarmobile commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141517225 Latest cordova (lib) version is 5.3.1, there is no cordova ios 4.3.0, latest is 3.9.1 --- If your project is set up for it, you can reply to this email and ha

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-18 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141507304 The issue exists with cordova 4.3.0 Any time you do cordova platform add ios, you get a check_reqs.js with the wrong case Q in it. O

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-17 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141239022 Not sure - if ionic is not up to date, a post in the Ionic forums would clear things up. --- If your project is set up for it, you can reply to this email and have

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-17 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-141091008 I just followed the instructions to install the ionic framework. The check_req.js in the newly installed iOS platform was the wrong case. I believ

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-16 Thread shazron
Github user shazron commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-140981509 @mschwartz it's definitely fixed, not sure what version of the platform you have? See: https://github.com/apache/cordova-ios/blob/ebbc6d05995e50312c044dd1ae11d18e1f

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-16 Thread mschwartz
Github user mschwartz commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-140803747 This is fixed for android, but not for iOS. I had to edit platform/ios/cordova/lib/check_reqs.js and change the case of the Q filename to lowercase. --

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-09-03 Thread appjitsu
Github user appjitsu commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-137566856 remove/add works for me on mac --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-08-28 Thread mixersoft
Github user mixersoft commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-135665090 I had the same problem. Did this to fix ``` ionic platform remove ios ionic platform add ios # NOW make the change from 'Q' > 'q' ionic build io

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-07-28 Thread tohagan
Github user tohagan commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-12582 +1 Still can't build. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not hav

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-06-16 Thread jhampton
Github user jhampton commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-112565149 Ditto - our team is currently unable to build a project because of this bug. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-06-15 Thread killswitch
Github user killswitch commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-112232774 This package hasn't been published since before this pull request was merged. Can we get this code published to npm? --- If your project is set up for it, you

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-04-15 Thread sendittokeith
Github user sendittokeith commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-93607980 Thank you, that was very helpful - I could not figure out the error because I am case insensitive and not prejudice like apple or linux - although I do think u

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-04-09 Thread rbelouin
Github user rbelouin commented on the pull request: https://github.com/apache/cordova-ios/pull/132#issuecomment-91174417 :+1: I cannot actually build my project for iOS. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-03-02 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cordova-ios/pull/132 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] cordova-ios pull request: Fix Q include in check_reqs.js

2015-02-28 Thread dpogue
GitHub user dpogue opened a pull request: https://github.com/apache/cordova-ios/pull/132 Fix Q include in check_reqs.js The npm package is q (lowercase) but was being required as Q (uppercase). For most people this probably works fine due to case insensitive filesystems, but if you