[GitHub] cordova-plugin-battery-status issue #38: CB-11256: (ubuntu) Qt API change br...

2016-09-08 Thread david-barth-canonical
Github user david-barth-canonical commented on the issue:

https://github.com/apache/cordova-plugin-battery-status/pull/38
  
Just to confirm LGTM and +1 from me


---
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-plugin-inappbrowser issue #174: CB-3360 Fix Oxide version & support ...

2016-09-08 Thread david-barth-canonical
Github user david-barth-canonical commented on the issue:

https://github.com/apache/cordova-plugin-inappbrowser/pull/174
  
Just to confirm LGTM and +1 from me


---
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-docs pull request #610: CB 11328 - Ubuntu platform documentation upd...

2016-06-08 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-docs/pull/610

CB 11328 - Ubuntu platform documentation update

Update the platform guide to reflect the status as of cordova-ubuntu@4.3.4


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-docs CB-11328

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/610.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 #610


commit 013903be2af809d47c5b64dbd1232c61dbbdd06f
Author: David Barth 
Date:   2016-06-01T16:00:23Z

update plugin support information for Ubuntu

commit b5d73b795a4d3167359bdd4324410b9684a2
Author: David Barth 
Date:   2016-06-01T16:00:54Z

Update of the Ubuntu platform guide

commit a7390bcb5194e394f41fdb07653fcbb7d7115600
Author: David Barth 
Date:   2016-06-01T16:07:00Z

typos

commit 725248cf5bd023a043360fad4e75534592721c11
Author: David Barth 
Date:   2016-06-02T09:30:47Z

add details on debugging and publishing an app for Ubuntu




---
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-docs pull request: Cordova Ubuntu 4.3.4 release announceme...

2016-05-26 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-docs/pull/607

Cordova Ubuntu 4.3.4 release announcement



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-docs 
cordova-ubuntu-4.3.4

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/607.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 #607


commit 159a63941af3c12fe52c77cb876b67e2109965b4
Author: David Barth 
Date:   2016-05-26T14:08:08Z

Cordova Ubuntu 4.3.4 release announcement




---
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-docs pull request: Add Cordova Ubuntu 4.3.3 blog post

2016-05-24 Thread david-barth-canonical
Github user david-barth-canonical commented on the pull request:

https://github.com/apache/cordova-docs/pull/497#issuecomment-221223223
  
Yeah, probably. I'll redo an announcement for 4.3.4


---
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-docs pull request: Add Cordova Ubuntu 4.3.3 blog post

2016-05-24 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-docs/pull/497


---
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-plugin-battery-status pull request: CB-11256 (was Fix due ...

2016-05-12 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-plugin-battery-status/pull/35


---
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-plugin-battery-status pull request: CB-11256 (was Fix due ...

2016-05-12 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-plugin-battery-status/pull/35

CB-11256 (was Fix due to Qt SystemInfo API updates)



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-plugin-battery-status 
fix-systeminfo-api-update

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-plugin-battery-status/pull/35.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 #35


commit 515b4ca6e9d9bafe378874968081b3f3f9dd7cb3
Author: Alexandre Abreu 
Date:   2016-05-09T16:10:50Z

Fix due to Qt SystemInfo API updates




---
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: Fix plugin platform_www & cordova_plugin...

2016-05-09 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-lib/pull/438


---
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: Fix plugin platform_www & cordova_plugin...

2016-05-09 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-lib/pull/438

Fix plugin platform_www & cordova_plugins.json copy



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-lib 
fix-cordova-plugins-copy

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/438.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 #438


commit b6ad96ffd5ef88b3c3d5fb5a0a689ae1a40cc8df
Author: Alexandre Abreu 
Date:   2016-05-09T13:21:23Z

Fix plugin platform_www & cordova_plugins.json copy




---
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-docs pull request: Add Cordova Ubuntu 4.3.3 blog post

2016-02-10 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-docs/pull/497

Add Cordova Ubuntu 4.3.3 blog post

Blog post to announce cordova-ubuntu 4.3.3

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/apache/cordova-docs cordova-ubuntu-4.3.3

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/497.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 #497


commit caafe227cc02b6a0b33187054f43f41c2adba716
Author: David Barth 
Date:   2016-02-09T16:17:13Z

Add Cordova Ubuntu 4.3.3 blog post




---
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-ubuntu pull request: Set of fixes for Cordova-Ubuntu 4.3.3

2016-01-28 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-ubuntu/pull/21


---
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-ubuntu pull request: Set of fixes for Cordova-Ubuntu 4.3.3

2016-01-28 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-ubuntu/pull/21

Set of fixes for Cordova-Ubuntu 4.3.3



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-ubuntu master

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-ubuntu/pull/21.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 #21


commit 8136602f23a004cf5dc7c80aeffbc4d7ca8f6159
Author: Peter Bakondy 
Date:   2016-01-20T10:07:54Z

fix typo

commit 31ba4b18ca7c8abac37962ecfa0cd310bc4a3288
Author: david-barth-canonical 
Date:   2016-01-20T11:02:03Z

Merge pull request #7 from pbakondy/master

fix typo

commit 0c2c3e222d9e2b2928d92fa2dc8fafb0bb89f1a3
Author: Alexandre Abreu 
Date:   2016-01-26T16:25:39Z

Fix reliance on deprecated Oxide's onLoadingChanged signal

commit 876674b37167524dbd0ad9100e45504d5632dff8
Author: david-barth-canonical 
Date:   2016-01-26T18:47:47Z

Merge pull request #8 from AlexandreAbreu/fix-loading-changed

Fix reliance on deprecated Oxide's onLoadingChanged signal

commit af17a8628613d70368cbf5e53f48e732f12c9cb2
Author: Alexandre Abreu 
Date:   2016-01-26T21:17:25Z

Fix debugging enabled flag & remove webkit specific bit

commit 827212f339cc75ec25b1078c7355f3152763bbfa
Author: david-barth-canonical 
Date:   2016-01-27T08:42:30Z

Merge pull request #9 from AlexandreAbreu/fix-devtools

Fix debugging enabled flag & remove webkit specific bit




---
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-docs pull request: Add Cordova Ubuntu 4.3.2 blog post

2016-01-08 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-docs/pull/452

Add Cordova Ubuntu 4.3.2 blog post



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-docs 
cordova-ubuntu-4.3.2

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-docs/pull/452.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 #452


commit cc5aa1e7ed2ae9c3f3e51e2af98f5f045dd3ee03
Author: David Barth 
Date:   2016-01-08T10:41:33Z

Add Cordova Ubuntu 4.3.2 blog post




---
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-ubuntu pull request: CB-10119 - Can't install click chroot...

2015-12-17 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-ubuntu/pull/20

CB-10119 - Can't install click chroot 14.10

Fix for CB-10119, switching the default framework to 15.04
Also a fix for check_reqs which was referencing a function that has been 
removed since.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-ubuntu 4.3.2-pre

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-ubuntu/pull/20.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 #20


commit 9a10c78bb75ec7c5a9175d960803f4c265ae92ef
Author: David Barth 
Date:   2015-11-19T08:30:04Z

Set VERSION to 4.4.0-dev (via coho)

commit 178832e268b6cb66f16e02ce718063692d6893b3
Author: David Barth 
Date:   2015-11-19T15:31:24Z

fix for update script

commit 4efeff2c275709ff617c8d68c05ab5a187f24f91
Author: David Barth 
Date:   2015-11-19T15:31:24Z

add www sub-directory to let the unit tests run properly

(cherry picked from commit 0f97fd32eaadcc03e918c9598205125ec215b492)

commit 2700423ec06c146800e1c58c1bda770320b3b1f3
Author: David Barth 
Date:   2015-11-24T10:57:58Z

Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/cordova-ubuntu

commit d9456a90e1b2824b0059ceecaff65b3b316e20cd
Author: Alexandre Abreu 
Date:   2015-12-07T21:30:55Z

Fix nodejs deps install which refers to an invalid ubuntu deps check 
function

commit 418947792d573812a3056092221f39ea99dfddf6
Author: David Barth 
Date:   2015-12-09T13:44:40Z

Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/cordova-ubuntu

commit 15141640f16ae94657de652d1a478c0a571b8e46
Author: David Barth 
Date:   2015-12-11T16:00:12Z

CB-10119 - default framework upgraded to 15.04

commit cc2cec984b188269f7a0d2fa4dfb487a36124d6c
Author: David Barth 
Date:   2015-12-11T16:07:24Z

update RELEASENOTES

commit e854a04b553767e0f6625ba61e83e90a64f5bc97
Author: david-barth-canonical 
Date:   2015-12-11T16:17:10Z

Merge pull request #6 from AlexandreAbreu/fix-build-ubuntu-deps

Fix nodejs deps install which refers to an invalid ubuntu deps check …

commit 70c5c51eec3cdcbb90948d3509a143f058b501d2
Author: David Barth 
Date:   2015-12-11T16:19:23Z

update RELEASENOTES




---
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 9590 - Ubuntu support for the new plu...

2015-12-03 Thread david-barth-canonical
Github user david-barth-canonical commented on a diff in the pull request:

https://github.com/apache/cordova-lib/pull/349#discussion_r46569876
  
--- Diff: cordova-lib/src/plugman/platforms/ubuntu.js ---
@@ -29,6 +29,54 @@ function toCamelCase(str) {
 }).join('');
 }
 
+function getPluginXml(plugin_dir) {
+var et = require('elementtree'),
+fs = require('fs'),
+path = require('path');
+
+var pluginxml;
+var config_path = path.join(plugin_dir, 'plugin.xml');
+
+if (fs.existsSync(config_path)) {
+// Get the current plugin.xml file
+pluginxml = et.parse(fs.readFileSync(config_path, 'utf-8'));
+}
+ 
+return pluginxml;
+}
+
+function findClassName(pluginxml, plugin_id) {
+var class_name;
+
+// first check if we have a class-name parameter in the plugin config
+if (pluginxml) {
+   var platform = pluginxml.find("./platform/[@name='ubuntu']/");
+   if (platform) {
+   var param = 
platform.find("./config-file/[@target='config.xml']/feature/param/[@name='ubuntu-package']");
+   if (param && param.attrib) {
+   class_name = param.attrib.value;
+   return class_name;
+   }
+   }
+}
+
+// fallback to guess work, based on the plugin package name
+
+if (plugin_id.match(/\.[^.]+$/)) {
+// old-style plugin name
+class_name = plugin_id.match(/\.[^.]+$/)[0].substr(1);
+class_name = toCamelCase(class_name);
+} else {
+   match = plugin_id.match(/cordova\-plugin\-([\w\-]+)$/);
+if (match && match.length > 0)
+   class_name = match[0].substr(15);
+   else
+class_name = toCamelCase(class_name);
+}
+
+return class_name;
+}
--- End diff --

There is a new PR at https://github.com/apache/cordova-lib/pull/349 with
your suggested fixes. Thanks for the review.

On Thu, Dec 3, 2015 at 12:45 AM, Tim Barham 
wrote:

> In cordova-lib/src/plugman/platforms/ubuntu.js
> <https://github.com/apache/cordova-lib/pull/349#discussion_r46494465>:
>
> > +// fallback to guess work, based on the plugin package name
> > +
> > +if (plugin_id.match(/\.[^.]+$/)) {
> > +// old-style plugin name
> > +class_name = plugin_id.match(/\.[^.]+$/)[0].substr(1);
> > +class_name = toCamelCase(class_name);
> > +} else {
> > +   match = plugin_id.match(/cordova\-plugin\-([\w\-]+)$/);
> > +if (match && match.length > 0)
> > +       class_name = match[0].substr(15);
    > > +   else
> > +class_name = toCamelCase(class_name);
> > +}
> > +
> > +return class_name;
> > +}
>
> Thanks @david-barth-canonical <https://github.com/david-barth-canonical>!
> Some comments:
>
>- Generally the Cordova codebase uses curly braces even around single
>line blocks.
>- There are some tabs in there that should be converted to spaces.
>- The match variable needs a var.
>- Should be calling toCamelCase() for the scenario where we match
>cordova-plugin-...
>- Where we don't match cordova-plugin-..., class_name is not defined
>(should be using plugin_id).
>- Since we will always end up calling toCamelCase(), the logic for the
>second half of this method can be simplified to:
>
> // fallback to guess work, based on the plugin package name
> if (plugin_id.match(/\.[^.]+$/)) {
> // old-style plugin name
> class_name = plugin_id.match(/\.[^.]+$/)[0].substr(1);
> } else {
> var match = plugin_id.match(/cordova\-plugin\-([\w\-]+)$/);
> if (match && match.length > 0) {
> class_name = match[0].substr(15);
> } else {
> class_name = plugin_id;
> }
> }
>
> return toCamelCase(class_name);
>
> Also, since the original PR has been merged, can you recreate this as a
> new PR which will just include the changes in your last commit (and 
changes
> in response to feedback, of course).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/cordova-lib/pull/349/files#r46494465>.
>



---
If your project is set up for it, you can reply to this email and have your
repl

[GitHub] cordova-lib pull request: CB-9590 - Ubuntu support for the new plu...

2015-12-02 Thread david-barth-canonical
Github user david-barth-canonical commented on a diff in the pull request:

https://github.com/apache/cordova-lib/pull/294#discussion_r46471716
  
--- Diff: cordova-lib/src/plugman/platforms/ubuntu.js ---
@@ -29,6 +29,51 @@ function toCamelCase(str) {
 }).join('');
 }
 
+function getPluginXml(plugin_dir) {
+var et = require('elementtree'),
+fs = require('fs'),
+path = require('path');
+
+var pluginxml;
+var config_path = path.join(plugin_dir, 'plugin.xml');
+
+if (fs.existsSync(config_path)) {
+// Get the current plugin.xml file
+pluginxml = et.parse(fs.readFileSync(config_path, 'utf-8'));
+}
+ 
+return pluginxml;
+}
+
+function findClassName(pluginxml, plugin_id) {
+var class_name;
+
+// first check if we have a class-name parameter in the plugin config
+if (pluginxml) {
+   var platform = pluginxml.find("./platform/[@name='ubuntu']/");
+   if (platform) {
+   var param = 
platform.find("./config-file/[@target='config.xml']/feature/param/[@name='ubuntu-package']");
+   if (param && param.attrib) {
+   class_name = param.attrib.value;
+   return class_name;
+   }
+   }
+}
+
+// fallback to guess work, based on the plugin package name
+
+if (plugin_id.match(/\.[^.]+$/)) {
+// old-style plugin name
+class_name = plugin_id.match(/\.[^.]+$/)[0].substr(1);
+class_name = toCamelCase(class_name);
+} else {
+class_name = 
plugin_id.match(/cordova\-plugin\-([\w\-]+)$/)[0].substr(15);
--- End diff --

Thanks! I updated the branch to take that into account. I was forcing 
plugins to upgrade to adding their class name to the XML file, but there is a 
more gradual way ;)

If that PR can't be re-opened I opened 
https://github.com/apache/cordova-lib/pull/349 . Otherwise feel free to close 
this other one.


---
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 9590 - Ubuntu support for the new plu...

2015-12-02 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-lib/pull/349

Cb 9590 - Ubuntu support for the new plugin naming convention

Re-opening the pull request with hopefully a better fix that can handle all 
situations.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-lib CB-9590

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/349.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 #349


commit 187711dd7fb7cd21e9ea276239b2e8b87a9e1ee6
Author: David Barth 
Date:   2015-09-01T17:40:35Z

Ubuntu support for the new plugin naming convention

commit afde15c7e6748dfedbdd9816df199c14f2f7596e
Author: David Barth 
Date:   2015-09-04T11:41:13Z

more robust class name detection

commit 1ae6a8631a92f58020c3b0139931d784843c281e
Author: David Barth 
Date:   2015-11-03T17:13:49Z

take plugin class name from the plugin.xml to avoid class name guess work

commit 1bbf544f3f3cff709d9ee1618b585d4c57e8e62c
Author: David Barth 
Date:   2015-11-19T17:00:35Z

happier jshint

commit 877404bbcb2d84cbb9cade8e692396237cc586f3
Author: David Barth 
Date:   2015-12-02T20:18:28Z

don't crash when parsing general plugin names




---
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 9618 - Icon parameter should not be r...

2015-12-02 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-lib/pull/347

Cb 9618 - Icon parameter should not be relative to the www directory on 
Ubuntu

This patch fixes a long standing issue, whereby the icon parameter in the 
app manifest was prefixed with a www directory while it is not in other parts 
of the tools or runtime.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-lib CB-9618

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/347.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 #347


commit 4aa92493c36fef07415a0d0e6690464b2d21bd51
Author: David Barth 
Date:   2015-09-07T15:38:27Z

change iconPath to be relative to the project root directory, only; ie not 
to the www sub-directory anymore

commit 67d2950d6106eaba031ece18b45cf4f8dea96f21
Author: David Barth 
Date:   2015-09-23T10:50:05Z

sync icon path lookup with the code in the manifest generation

commit 704c13d8616be9036bbcce25238ecd1a3f6731b1
Author: David Barth 
Date:   2015-10-02T13:38:36Z

don't add a leading www to the icon path when generating the desktop file




---
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-ubuntu pull request: Cordova-Ubuntu Platform Release 4.3.1...

2015-11-19 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-ubuntu/pull/19

Cordova-Ubuntu Platform Release 4.3.1 

From the RELEASENOTES.md

* default icon name www/img/logo.png
* add defaut icon parameter to config.xml
* use QUrl::fromLocalFile()
* set mainUrl when the file exists
* fix critical issue introduced by general URL support changes earlier
* check if content src points to a url or local file
* fix typo
* better info msg
* don't check deps at create time and propose to auto-install once at build 
time; this is for the native/desktop build

See release preparation logs at 
https://issues.apache.org/jira/browse/CB-10030

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-ubuntu 4.3.x

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-ubuntu/pull/19.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 #19


commit d3a674f2a07ab269bfb2e2acaa60dc2ab25c4d50
Author: Alberto Mardegan 
Date:   2015-10-08T06:36:31Z

build: fix mechanism for additional dependencies

Instead of expecting plugins to store their configuration files in
a "configs/" directory, read the dependencies from the config.xml file.
The reason why the old method does not work is that the 
element of the plugin.xml file doesn't support a "target-dir" attribute.

By using the  element, instead, we don't need to do any
changes to cordova lib, and all the plugin configuration can now be
stored in the plugin.xml file.

Example:


  




  ["signond", "libsignon-qt5"]
  ["signond-dev:ARCH", "libsignon-qt5-dev:ARCH"]

  


commit 576fca86f5bc51df285ad5b3a94332039bf75161
Author: David Barth 
Date:   2015-09-30T16:20:19Z

add defaut icon parameter to config.xml

commit 7a5ac6dbc046bb9fd75c5de313723ee90907f7d6
Author: david-barth-canonical 
Date:   2015-10-27T11:24:55Z

Merge pull request #1 from mardy/additional-dependencies

build: fix mechanism for additional plugin dependencies

commit 33f61c7fb7d2370c907beec51a604026b62a57ea
Author: David Barth 
Date:   2015-10-28T14:30:39Z

default icon name www/img/logo.png

commit aaf201b289119f498866b4dbc16e6c581b15c2ac
Author: David Barth 
Date:   2015-11-12T16:42:04Z

Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/cordova-ubuntu

commit 75819c8186476c7d7534e6e19c6faebecd7fc9cd
Author: David Barth 
Date:   2015-09-30T14:51:05Z

check if content src points to a url or local file

commit 86e994affa1d7406295952bfac2d2af634485d2c
Author: David Barth 
Date:   2015-11-12T15:07:06Z

fix critical issue introduced by general URL support changes earlier

commit ac27c286c124776c2c4b938ccf86f2beaa20ac7e
Author: David Barth 
Date:   2015-11-12T16:02:26Z

set mainUrl when the file exists

commit 6fe31e7cb0568bee263cdf4c39c829f9e9bf4c97
Author: David Barth 
Date:   2015-10-28T18:50:38Z

don't check deps at create time and propose to auto-install once at build 
time; this is for the native/desktop build

commit 8eeea8d71dabfa43bbaab240ecf666d1e89efa88
Author: David Barth 
Date:   2015-11-12T15:06:16Z

better info msg

commit eea201221e2cc2cdaef06a9187f4c7f0a8af754d
Author: David Barth 
Date:   2015-11-18T10:29:12Z

Merge remote-tracking branch 'apache/master'

commit 91f655d03047583919582e841fb51c1f0df1ff63
Author: David Barth 
Date:   2015-11-18T11:19:02Z

Set VERSION to 4.4.0-dev (via coho)

commit 63d740326f2191de3bf568a06d8013395733ca2c
Author: David Barth 
Date:   2015-11-18T11:19:04Z

Update JS snapshot to version 4.4.0-dev (via coho)

commit 6630211fd507766176b2f1a1ba9a7619339a486a
Author: David Barth 
Date:   2015-11-18T13:51:34Z

fix typo

commit 51f8f3ce5204beb731d0ea8c708c9c0c533232c1
Author: david-barth-canonical 
Date:   2015-11-18T13:52:56Z

Merge pull request #4 from cordova-ubuntu/CB-9868

Cb 9868 - Should propose to install missing Ubuntu dependencies on platform 
add

commit cf5bbb6343aab3f0662550554c580bc464dc95d2
Author: David Barth 
Date:   2015-11-18T13:56:36Z

use QUrl::fromLocalFile()

commit e4a3c9e363b9d0ecda7536cc9b18342e3dfa1ab3
Author: david-barth-canonical 
Date:   2015-11-18T13:57:26Z

Merge pull request #3 from cordova-ubuntu/CB-9694
    
    Cb 9694 - Add support for HTTP urls

commit 346d43905e1fe498d74f374e0b6cb1c10e0aa5d9
Author: david-barth-canonical 
Date:   2015-11-18T13:57:44Z

Merge pull request #2 from cordova-ubuntu/CB-9806

Cb 9806 - Add icon to project defaults

commit 5cca7ca72c65882053266d22023fb7a9cf8194a1
Author: David

[GitHub] cordova-ubuntu pull request: add defaut icon parameter to config.x...

2015-11-13 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-ubuntu/pull/16


---
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-ubuntu pull request: Cb 9694 - Add support for HTTP urls

2015-11-13 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-ubuntu/pull/18


---
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-ubuntu pull request: Cb 9694 - Add support for HTTP urls

2015-11-12 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-ubuntu/pull/18

Cb 9694 - Add support for HTTP urls

The cordova-ubuntu container currently does not support loading the app 
from a network URL, only local file URLs. This prevents features like 
livereloading to work.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-ubuntu CB-9694

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-ubuntu/pull/18.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 #18


commit d3a674f2a07ab269bfb2e2acaa60dc2ab25c4d50
Author: Alberto Mardegan 
Date:   2015-10-08T06:36:31Z

build: fix mechanism for additional dependencies

Instead of expecting plugins to store their configuration files in
a "configs/" directory, read the dependencies from the config.xml file.
The reason why the old method does not work is that the 
element of the plugin.xml file doesn't support a "target-dir" attribute.

By using the  element, instead, we don't need to do any
changes to cordova lib, and all the plugin configuration can now be
stored in the plugin.xml file.

Example:


  




  ["signond", "libsignon-qt5"]
  ["signond-dev:ARCH", "libsignon-qt5-dev:ARCH"]

  
    

commit 7a5ac6dbc046bb9fd75c5de313723ee90907f7d6
Author: david-barth-canonical 
Date:   2015-10-27T11:24:55Z

Merge pull request #1 from mardy/additional-dependencies

build: fix mechanism for additional plugin dependencies

commit aaf201b289119f498866b4dbc16e6c581b15c2ac
Author: David Barth 
Date:   2015-11-12T16:42:04Z

Merge branch 'master' of 
https://git-wip-us.apache.org/repos/asf/cordova-ubuntu

commit 43e1c34f377d888599968cd42ee06751de09568b
Author: David Barth 
Date:   2015-09-30T14:51:05Z

check if content src points to a url or local file

commit 429971412c5903665942861552f3906a1f31ba71
Author: David Barth 
Date:   2015-11-12T15:07:06Z

fix critical issue introduced by general URL support changes earlier

commit 606cc331d5dc9410906ef3ddc7818555005834f0
Author: David Barth 
Date:   2015-11-12T16:02:26Z

set mainUrl when the file exists




---
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-ubuntu pull request: check if content src points to a url ...

2015-11-12 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-ubuntu/pull/14


---
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-9590 - Ubuntu support for the new plu...

2015-11-03 Thread david-barth-canonical
Github user david-barth-canonical commented on the pull request:

https://github.com/apache/cordova-lib/pull/294#issuecomment-153422772
  
I have update the pull request to add support for taking the class name 
from the plugin.xml file. For plugins using one of the 2 existing naming 
conventions, we can fallback to "guess work", otherwise new plugins using 
arbitrary names can just specify the 'package-ubuntu' parameter (like iOS or 
Android) to specify the class name directly. 
I decided to go ahead and keep on adding code to the cordova-lib for now, 
but will start adding support for that preparation step in cordova-ubuntu 
direclty. In 2 steps, I imagine we can migrate core plugins to using the new 
parameter and deprecate the specific code from the ubuntu/install stage in 
cordova-lib eventually. 


---
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-9590 - Ubuntu support for the new plu...

2015-10-16 Thread david-barth-canonical
Github user david-barth-canonical commented on the pull request:

https://github.com/apache/cordova-lib/pull/294#issuecomment-148743166
  
ok, i understand the problem now. In particular, the code to guess the
classname is nice, but other platforms resort to encapsulating this
information into the plugin.xml file which is safer.
We have some new code in cordova-ubuntu to handle dependencies, so I wonder
how much of that logic needs to be updated in cordova-lib, rather than
directly in cordova-ubuntu? See mardy's new branch

https://github.com/mardy/cordova-ubuntu/commit/d3a674f2a07ab269bfb2e2acaa60dc2ab25c4d50

On Thu, Oct 15, 2015 at 6:26 PM, Tim Barham 
wrote:

    > @david-barth-canonical <https://github.com/david-barth-canonical> - I
> think @stevengill <https://github.com/stevengill>'s point is that the
> intent of your change is to strip the cordova-plugin from the start of
> the plugin name to generate the class name, to get the equivalent of the
> old functionality. That logic won't work with plugins that don't start 
with
> cordova-plugin-. Perhaps an alternative would be to look for -plugin- in
> the name, and take everything after that (?otherwise just use the whole
> name).
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/cordova-lib/pull/294#issuecomment-148444797>.
>



---
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-ubuntu pull request: add defaut icon parameter to config.x...

2015-10-15 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-ubuntu/pull/16

add defaut icon parameter to config.xml



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-ubuntu CB-9806

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-ubuntu/pull/16.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 #16


commit 576fca86f5bc51df285ad5b3a94332039bf75161
Author: David Barth 
Date:   2015-09-30T16:20:19Z

add defaut icon parameter to config.xml




---
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-9590 - Ubuntu support for the new plu...

2015-10-15 Thread david-barth-canonical
Github user david-barth-canonical commented on the pull request:

https://github.com/apache/cordova-lib/pull/294#issuecomment-148342742
  
It does work on my system at least. I run this branch along with my other 3 
pull requests, but I don't think they have an influence on the plugin 
installation process.

dbarth@thinkmac:~/devel/wily/cordova/demo$ cordova -d plugin add 
phonegap-plugin-push
no version specified, retrieving version from config.xml
Calling plugman.fetch on plugin "phonegap-plugin-push"
Fetching plugin "phonegap-plugin-push" via npm
Copying plugin "/home/dbarth/.npm/phonegap-plugin-push/1.3.0/package" => 
"/home/dbarth/devel/wily/cordova/demo/plugins/phonegap-plugin-push"
Calling plugman.install on plugin 
"/home/dbarth/devel/wily/cordova/demo/plugins/phonegap-plugin-push" for 
platform "ubuntu
Installing "phonegap-plugin-push" for ubuntu
Running command: 
/home/dbarth/devel/wily/cordova/demo/platforms/ubuntu/cordova/version 
Command finished with error code 0: 
/home/dbarth/devel/wily/cordova/demo/platforms/ubuntu/cordova/version 
cordova has been detected as using a development branch. Attemping to 
install anyways.
Install start for "phonegap-plugin-push" on ubuntu.
Beginning processing of action stack for ubuntu project...
Action stack processing complete.
Install complete for phonegap-plugin-push on ubuntu.




---
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-ubuntu pull request: check if content src points to a url ...

2015-09-30 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-ubuntu/pull/14

check if content src points to a url or local file



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-ubuntu CB-9694

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-ubuntu/pull/14.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 #14


commit aa53b6652f56fbfcbc824140aee6457174316c15
Author: David Barth 
Date:   2015-09-30T14:51:05Z

check if content src points to a url or local file




---
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: icon parameter should not be relative to...

2015-09-23 Thread david-barth-canonical
Github user david-barth-canonical commented on the pull request:

https://github.com/apache/cordova-lib/pull/299#issuecomment-142564241
  
Yes, makes sense. Icon management needs more tweaks, but this way it will 
work better already Thanks for the review.


---
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: icon parameter should not be relative to...

2015-09-07 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-lib/pull/299

icon parameter should not be relative to the www directory on Ubuntu



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-lib CB-9618

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/299.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 #299


commit 4aa92493c36fef07415a0d0e6690464b2d21bd51
Author: David Barth 
Date:   2015-09-07T15:38:27Z

change iconPath to be relative to the project root directory, only; ie not 
to the www sub-directory anymore




---
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 9618

2015-09-07 Thread david-barth-canonical
Github user david-barth-canonical closed the pull request at:

https://github.com/apache/cordova-lib/pull/298


---
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 9618

2015-09-07 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-lib/pull/298

Cb 9618

Patch to change iconPath to be relative to the project root directory, 
only; ie not to the www sub-directory anymore.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-lib CB-9618

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/298.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 #298


commit 187711dd7fb7cd21e9ea276239b2e8b87a9e1ee6
Author: David Barth 
Date:   2015-09-01T17:40:35Z

Ubuntu support for the new plugin naming convention

commit afde15c7e6748dfedbdd9816df199c14f2f7596e
Author: David Barth 
Date:   2015-09-04T11:41:13Z

more robust class name detection

commit dcd77f3778a9755c7bd826a70eb084527fdcb028
Author: David Barth 
Date:   2015-09-07T15:23:22Z

change iconPath to be relative to the project root directory, only; ie not 
to the www sub-directory anymore




---
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-9590 - Ubuntu support for the new plu...

2015-09-07 Thread david-barth-canonical
Github user david-barth-canonical commented on the pull request:

https://github.com/apache/cordova-lib/pull/294#issuecomment-138310314
  
It does work both with old style plugin names, like 
"org.apache.cordova.battery-status" and new style names 
"cordova-plugin-media-capture". I can add and build a demo application  with 
the 2 of those here on my system.

The facebookconnect plugin failed compilation in my tests, but it seems 
more due to the absence of  ubuntu platform support code.

If there are still issues, can you give an example or a log of where it 
fails?


---
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-9590 - Ubuntu support for the new plu...

2015-09-01 Thread david-barth-canonical
GitHub user david-barth-canonical opened a pull request:

https://github.com/apache/cordova-lib/pull/294

CB-9590 - Ubuntu support for the new plugin naming convention

This patch allows ubuntu users to add plugins using the new naming 
convention and build properly. This also leaves support in place for the old 
style plugin names, during the transition.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/cordova-ubuntu/cordova-lib CB-9590

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/cordova-lib/pull/294.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 #294


commit 187711dd7fb7cd21e9ea276239b2e8b87a9e1ee6
Author: David Barth 
Date:   2015-09-01T17:40:35Z

Ubuntu support for the new plugin naming convention




---
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