[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-17 Thread TimBarham
Github user TimBarham closed the pull request at: https://github.com/apache/cordova-lib/pull/164 --- 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

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-12 Thread stefangordon
Github user stefangordon commented on the pull request: https://github.com/apache/cordova-lib/pull/164#issuecomment-74116684 Great stuff! Any pointers on how users can easily install this via NPM now? I've only had success manually cloning the lib and cli repos and copying correct bin

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-11 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-lib/pull/164#issuecomment-73999167 Thanks Jesse! --- 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 fe

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-11 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-lib/pull/164#issuecomment-73998473 Thanks Tim! This is now merged, the mirror is just slow to update. This should close in a few minutes. --- If your project is set up for it, you can reply to

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-11 Thread sgrebnov
Github user sgrebnov commented on the pull request: https://github.com/apache/cordova-lib/pull/164#issuecomment-73859774 Reviewed and tested, LGTM :+1: --- 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-lib pull request: CB-8123 Plugin references can target spe...

2015-02-10 Thread TimBarham
GitHub user TimBarham opened a pull request: https://github.com/apache/cordova-lib/pull/164 CB-8123 Plugin references can target specific windows platforms. Adds support for `target`, `versions` and `arch` attributes on `` and `` elements in the windows platform of plugin.xml. This

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-10 Thread TimBarham
Github user TimBarham closed the pull request at: https://github.com/apache/cordova-lib/pull/155 --- 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

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-02-10 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-lib/pull/155#issuecomment-73799000 I'll close this PR, and open a new, cleaned up PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-lib/pull/155#issuecomment-72117279 yeah, whichever approach is easier. Ultimately the test in this case is invalid, but we don't want it appearing to break the build. --- If your project is set

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread TimBarham
Github user TimBarham commented on the pull request: https://github.com/apache/cordova-lib/pull/155#issuecomment-72114968 The tests use path.join(), but jsproj uses hard-coded backslashes (which is reasonable, because normally it would only run on Windows). This problem is showing up

RE: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread Tim Barham
ests previously). -Original Message- From: Jesse [mailto:purplecabb...@gmail.com] Sent: Thursday, January 29, 2015 12:04 PM To: dev@cordova.apache.org Subject: Re: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe... The tests are only failing on Travis, which is ru

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread purplecabbage
Github user purplecabbage commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/155#discussion_r23798529 --- Diff: cordova-lib/spec-plugman/platforms/windows.spec.js --- @@ -0,0 +1,415 @@ +/** + Licensed to the Apache Software Foundation (ASF) u

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread purplecabbage
Github user purplecabbage commented on a diff in the pull request: https://github.com/apache/cordova-lib/pull/155#discussion_r23798388 --- Diff: cordova-lib/spec-plugman/platforms/windows.spec.js --- @@ -0,0 +1,415 @@ +/** + Licensed to the Apache Software Foundation (ASF) u

Re: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread Jesse
e.org > Subject: [GitHub] cordova-lib pull request: CB-8123 Plugin references can > target spe... > > Github user purplecabbage commented on the pull request: > > https://github.com/apache/cordova-lib/pull/155#issuecomment-71907847 > > Very cool Tim! > > > ---

RE: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-29 Thread Tim Barham
] Sent: Wednesday, January 28, 2015 12:18 PM To: dev@cordova.apache.org Subject: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe... Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-lib/pull/155#issuecomment-71907847 Very

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-28 Thread purplecabbage
Github user purplecabbage commented on the pull request: https://github.com/apache/cordova-lib/pull/155#issuecomment-71907847 Very cool Tim! --- 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 th

[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...

2015-01-28 Thread TimBarham
GitHub user TimBarham opened a pull request: https://github.com/apache/cordova-lib/pull/155 CB-8123 Plugin references can target specific windows platforms. Adds support for `target`, `versions` and `arch` attributes on `` and `` elements in the windows platform of plugin.xml. This