[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 binaries. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[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/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 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 `lib-file` and `framework` elements in the windows platform of plugin.xml. This allows plugin authors to target different references to different target platforms. Also adds support for `src` attribute as an alias for the `Include` attribute on the ` lib-file` element (since `src` is documented, but `Include` is used by existing plugins). Adds some tests to cover the new attributes. Updates existing plugin tests for windows8 platform to also test windows platform (left in windows8 tests to help verify backward compatibility with old windows8 platform). As part of this change, refactored `jsproj` to `jsprojManager` to reflect the fact that, with the windows platform, this class now manages multiple jsproj files. Broken into three commits to separate file renames from changes, so that git can successfully track the changes. You can merge this pull request into a Git repository by running: $ git pull https://github.com/MSOpenTech/cordova-lib CB-8123 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cordova-lib/pull/164.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #164 commit 94efb46eccbe0438dbede35b90c4c2255dac0b77 Author: Tim Barham tim.bar...@microsoft.com Date: 2015-02-11T00:53:36Z CB-8123 Plugin references can target specific windows platforms. Adds support for `target`, `versions` and `arch` attributes on `lib-file` and `framework` elements in the windows platform of plugin.xml. This allows plugin authors to target different references to different target platforms. Also adds support for `src` attribute as an alias for the `Include` attribute on the ` lib-file` element (since `src` is documented, but `Include` is used by existing plugins). Adds some tests to cover the new attributes. Updates existing plugin tests for windows8 platform to also test windows platform (left in windows8 tests to help verify backward compatibility with old windows8 platform). As part of this change, refactored `jsproj` to `jsprojManager` to reflect the fact that, with the windows platform, this class now manages multiple jsproj files. I will rename some windows8 files and folders to windows, and jsproj.js to jsprojManager.js in a subsequent commit. commit d7a09785bde58d11136653685c08ad148c1ae189 Author: Tim Barham tim.bar...@microsoft.com Date: 2015-02-11T01:11:40Z CB-8123 Rename windows platform related files. Renames `windows8` plugin platform folders in tests to `windows`. Renames `jsproj.js` to `jsprojManager.js`. commit 5b31cecf7677a53279ded52158dd3a6ea57d379c Author: Tim Barham tim.bar...@microsoft.com Date: 2015-02-11T01:12:53Z CB-8123 Rename further windows platform related files. Renames `windows8.spec.js` to `windows.spec.js`. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
RE: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
Thanks Jesse. Do you have any thoughts on the test failures? The failures appear to be related to my directory renaming - but the tests pass locally, and the commits look good (and identical to what I have locally). -Original Message- From: purplecabbage [mailto:g...@git.apache.org] 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 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
Re: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
The tests are only failing on Travis, which is running MacOS, so it is probably just related to path separators. @purplecabbage risingj.com On Thu, Jan 29, 2015 at 11:27 AM, Tim Barham tim.bar...@microsoft.com wrote: Thanks Jesse. Do you have any thoughts on the test failures? The failures appear to be related to my directory renaming - but the tests pass locally, and the commits look good (and identical to what I have locally). -Original Message- From: purplecabbage [mailto:g...@git.apache.org] 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 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 now because previously this code wasnât exercised by tests. Iâm looking to see if it is feasible/reasonable to update jsproj to be platform agnostic, to the extent it needs to be for tests to pass. Iâd prefer that to skipping the tests, so we get consistent coverage (the tests donât build projects â just verify that files are modified as expected). From: Jesse MacFadyen [mailto:notificati...@github.com] Sent: Thursday, January 29, 2015 12:12 PM To: apache/cordova-lib Cc: Tim Barham Subject: Re: [cordova-lib] CB-8123 Plugin references can target specific windows platforms. (#155) You should use path.join( ) in your tests when creating paths so the tests will still pass on MacOS, or check the environment and skip running your tests altogether. â Reply to this email directly or view it on GitHubhttps://github.com/apache/cordova-lib/pull/155#issuecomment-72095389. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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) under one
+ or more contributor license agreements. See the NOTICE file
+ distributed with this work for additional information
+ regarding copyright ownership. The ASF licenses this file
+ to you under the Apache License, Version 2.0 (the
+ License); you may not use this file except in compliance
+ with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing,
+ software distributed under the License is distributed on an
+ AS IS BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ KIND, either express or implied. See the License for the
+ specific language governing permissions and limitations
+ under the License.
+ */
+var windows = require('../../src/plugman/platforms/windows'),
+common = require('../../src/plugman/platforms/common'),
+install = require('../../src/plugman/install'),
+path = require('path'),
+fs = require('fs'),
+shell = require('shelljs'),
+et = require('elementtree'),
+os = require('osenv'),
+cordovaProjectDir = path.join(os.tmpdir(), 'plugman'),
+cordovaProjectWindowsPlatformDir = path.join(cordovaProjectDir,
'platforms', 'windows'),
+cordovaProjectPluginsDir = path.join(cordovaProjectDir, 'plugins'),
+xml_helpers = require('../../src/util/xml-helpers'),
+dummyplugin = path.join(__dirname, '..', 'plugins',
'org.test.plugins.dummyplugin'),
+faultyplugin = path.join(__dirname, '..', 'plugins',
'org.test.plugins.faultyplugin');
+
+var PluginInfo = require('../../src/PluginInfo');
+
+var dummyPluginInfo = new PluginInfo(dummyplugin);
+var dummy_id = dummyPluginInfo.id;
+var valid_source = dummyPluginInfo.getSourceFiles('windows');
+var valid_libfiles = dummyPluginInfo.getLibFiles('windows');
+var valid_frameworks = dummyPluginInfo.getFrameworks('windows');
+
+var faultyPluginInfo = new PluginInfo(faultyplugin);
+var faulty_id = faultyPluginInfo.id;
+var invalid_source = faultyPluginInfo.getSourceFiles('windows');
+var invalid_libfiles = faultyPluginInfo.getLibFiles('windows');
+
+function copyArray(arr) {
+return Array.prototype.slice.call(arr, 0);
+}
+
+beforeEach(function () {
+this.addMatchers({
+toContainXmlPath: function (xpath) {
+var xml = this.actual;
+var notText = this.isNot ? 'not ' : '';
+this.message = function () {
+return 'Expected xml \'' + et.tostring(xml) + '\' ' +
notText + 'to contain elements matching \'' + xpath + '\'.';
+};
+
+return xml.find(xpath) != null;
+}});
+});
+
+['windows', 'windows8'].forEach(function (platform) {
+var windows_project = path.join(__dirname, '..', 'projects', platform);
+
+shell.mkdir('-p', cordovaProjectWindowsPlatformDir);
+shell.cp('-rf', path.join(windows_project, '*'),
cordovaProjectWindowsPlatformDir);
+var proj_files =
windows.parseProjectFile(cordovaProjectWindowsPlatformDir);
+shell.rm('-rf', cordovaProjectDir);
+
+var platformProjects = {
+windows: {
+all: 'CordovaApp.projitems',
+phone: 'CordovaApp.Phone.jsproj',
+windows: 'CordovaApp.Windows.jsproj',
+windows8: 'CordovaApp.Windows80.jsproj'
+}, windows8: {
+all: 'TestApp.jsproj',
+windows8: 'TestApp.jsproj'
+}
+}[platform];
+
+describe(platform + ' project handler', function () {
+beforeEach(function () {
+shell.mkdir('-p', cordovaProjectWindowsPlatformDir);
+shell.mkdir('-p', cordovaProjectPluginsDir);
+});
+afterEach(function () {
+shell.rm('-rf', cordovaProjectDir);
+});
+
+describe('www_dir method', function () {
+it('should return cordova-windows project www location using
www_dir', function () {
+expect(windows.www_dir(path.sep)).toEqual(path.sep +
'www');
+});
+});
+describe('package_name method', function () {
+it('should return a windows project\'s proper package name',
function () {
+
expect(windows.package_name(windows_project)).toEqual(CordovaApp);
+});
+});
+
+describe('parseProjectFile method',
[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-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 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
RE: [GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
Oh, yeah, I see the problem. Paths in jsproj (now jsprojManager) are hard-coded to use \, which is not a problem in practice because this code would only normally executed on Windows, but is a problem when running tests (guess this particular code hasn't been exercised by tests 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 running MacOS, so it is probably just related to path separators. @purplecabbage risingj.com On Thu, Jan 29, 2015 at 11:27 AM, Tim Barham tim.bar...@microsoft.com wrote: Thanks Jesse. Do you have any thoughts on the test failures? The failures appear to be related to my directory renaming - but the tests pass locally, and the commits look good (and identical to what I have locally). -Original Message- From: purplecabbage [mailto:g...@git.apache.org] 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 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 this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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) under one
+ or more contributor license agreements. See the NOTICE file
+ distributed with this work for additional information
+ regarding copyright ownership. The ASF licenses this file
+ to you under the Apache License, Version 2.0 (the
+ License); you may not use this file except in compliance
+ with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing,
+ software distributed under the License is distributed on an
+ AS IS BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ KIND, either express or implied. See the License for the
+ specific language governing permissions and limitations
+ under the License.
+ */
+var windows = require('../../src/plugman/platforms/windows'),
+common = require('../../src/plugman/platforms/common'),
+install = require('../../src/plugman/install'),
+path = require('path'),
+fs = require('fs'),
+shell = require('shelljs'),
+et = require('elementtree'),
+os = require('osenv'),
+cordovaProjectDir = path.join(os.tmpdir(), 'plugman'),
+cordovaProjectWindowsPlatformDir = path.join(cordovaProjectDir,
'platforms', 'windows'),
+cordovaProjectPluginsDir = path.join(cordovaProjectDir, 'plugins'),
+xml_helpers = require('../../src/util/xml-helpers'),
+dummyplugin = path.join(__dirname, '..', 'plugins',
'org.test.plugins.dummyplugin'),
+faultyplugin = path.join(__dirname, '..', 'plugins',
'org.test.plugins.faultyplugin');
+
+var PluginInfo = require('../../src/PluginInfo');
+
+var dummyPluginInfo = new PluginInfo(dummyplugin);
+var dummy_id = dummyPluginInfo.id;
+var valid_source = dummyPluginInfo.getSourceFiles('windows');
+var valid_libfiles = dummyPluginInfo.getLibFiles('windows');
+var valid_frameworks = dummyPluginInfo.getFrameworks('windows');
+
+var faultyPluginInfo = new PluginInfo(faultyplugin);
+var faulty_id = faultyPluginInfo.id;
+var invalid_source = faultyPluginInfo.getSourceFiles('windows');
+var invalid_libfiles = faultyPluginInfo.getLibFiles('windows');
+
+function copyArray(arr) {
+return Array.prototype.slice.call(arr, 0);
+}
+
+beforeEach(function () {
+this.addMatchers({
+toContainXmlPath: function (xpath) {
+var xml = this.actual;
+var notText = this.isNot ? 'not ' : '';
+this.message = function () {
+return 'Expected xml \'' + et.tostring(xml) + '\' ' +
notText + 'to contain elements matching \'' + xpath + '\'.';
+};
+
+return xml.find(xpath) != null;
+}});
+});
+
+['windows', 'windows8'].forEach(function (platform) {
+var windows_project = path.join(__dirname, '..', 'projects', platform);
+
+shell.mkdir('-p', cordovaProjectWindowsPlatformDir);
+shell.cp('-rf', path.join(windows_project, '*'),
cordovaProjectWindowsPlatformDir);
+var proj_files =
windows.parseProjectFile(cordovaProjectWindowsPlatformDir);
+shell.rm('-rf', cordovaProjectDir);
+
+var platformProjects = {
+windows: {
+all: 'CordovaApp.projitems',
+phone: 'CordovaApp.Phone.jsproj',
+windows: 'CordovaApp.Windows.jsproj',
+windows8: 'CordovaApp.Windows80.jsproj'
+}, windows8: {
+all: 'TestApp.jsproj',
+windows8: 'TestApp.jsproj'
+}
+}[platform];
+
+describe(platform + ' project handler', function () {
+beforeEach(function () {
+shell.mkdir('-p', cordovaProjectWindowsPlatformDir);
+shell.mkdir('-p', cordovaProjectPluginsDir);
+});
+afterEach(function () {
+shell.rm('-rf', cordovaProjectDir);
+});
+
+describe('www_dir method', function () {
+it('should return cordova-windows project www location using
www_dir', function () {
+expect(windows.www_dir(path.sep)).toEqual(path.sep +
'www');
+});
+});
+describe('package_name method', function () {
+it('should return a windows project\'s proper package name',
function () {
+
expect(windows.package_name(windows_project)).toEqual(CordovaApp);
+});
+});
+
+describe('parseProjectFile method',
[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! --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
[GitHub] cordova-lib pull request: CB-8123 Plugin references can target spe...
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 `lib-file` and `framework` elements in the windows platform of plugin.xml. This allows plugin authors to target different references to different target platforms. Also adds support for `src` attribute as an alias for the `Include` attribute on the `lib-file` element (since `src` is documented, but `Include` is used by existing plugins). Adds some tests to cover the new attributes. Updates existing plugin tests for windows8 platform to also test windows platform (left in windows8 tests to help verify backward compatibility with old windows8 platform). As part of this change, refactored `jsproj` to `jsprojManager` to reflect the fact that, with the windows platform, this class now manages multiple jsproj files. You can merge this pull request into a Git repository by running: $ git pull https://github.com/MSOpenTech/cordova-lib CB-8123-final Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cordova-lib/pull/155.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #155 commit 8b6f7b9b443fe6b35e070bd1ad8ff81df53f559f Author: Tim Barham tim.bar...@microsoft.com Date: 2015-01-28T19:15:39Z CB-8123 Plugin references can target specific windows platforms. Adds support for `target`, `versions` and `arch` attributes on `lib-file` and `framework` elements in the windows platform of plugin.xml. This allows plugin authors to target different references to different target platforms. Also adds support for `src` attribute as an alias for the `Include` attribute on the `lib-file` element (since `src` is documented, but `Include` is used by existing plugins). Adds some tests to cover the new attributes. Updates existing plugin tests for windows8 platform to also test windows platform (left in windows8 tests to help verify backward compatibility with old windows8 platform). As part of this change, refactored `jsproj` to `jsprojManager` to reflect the fact that, with the windows platform, this class now manages multiple jsproj files. Note that I plan to rename some windows8 files and folders to windows, and jsproj.js to jsprojManager.js in a subsequent commit. commit 48852eefb20840149fa9ada28082e6cb0a2ff208 Author: Tim Barham tim.bar...@microsoft.com Date: 2015-01-28T19:30:27Z CB-8123 Rename windows platform related files. Renames `windows8` plugin platform folders in tests to `windows`. Renames `jsproj.js` to `jsprojManager.js`. commit 22a214f217014b8c2df0348025d734cdf03580a7 Author: Tim Barham tim.bar...@microsoft.com Date: 2015-01-28T19:31:48Z CB-8123 Rename further windows platform related files. Renames `windows8.spec.js` to `windows.spec.js`. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. --- - To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org For additional commands, e-mail: dev-h...@cordova.apache.org
