[CB-4528] Plugman v0.10.0 fails to publish due to path.join error

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.0.0
• Component/s: cordova-plugman

Environment

OS X 10.8.4
node v0.10.2
npm v1.2.15
cordova (cli) v3.0.4
plugman v0.10.0

• Assignee: Anis Kadri
• Reporter: Steve Jansen
• Created: 07/Aug/13 17:55
• Updated: 23/Aug/18 07:27
• Resolved: 07/Aug/13 19:41

Description

Overview
--------
Publishing to the cordova registry @ http://registry.cordova.io/ fails due to a problem parsing the `dir` argument to `plugman publish dir`

Steps To Reproduce
------------------
sudo npm install -g plugman
plugman adduser steve-jansen
plugman publish --debug --plugin ./cordova-ios-emailcomposer/

Expected Results
----------------
This should publish the ./cordova-ios-emailcomposer/ plugin to http://registry.cordova.io/

$ ls -l ./cordova-ios-emailcomposer/plugin.xml
rw-rr- 1 me staff 1676 Aug 6 22:47 ./cordova-ios-emailcomposer/plugin.xml

This is a local clone of https://github.com/steve-jansen/cordova-ios-emailcomposer

Actual Results
--------------
TypeError: Arguments to path.join must be strings
at path.js:360:15
at Array.filter (native)
at Object.exports.join (path.js:358:36)
at Object.generatePackageJsonFromPluginXml (/usr/local/lib/node_modules/plugman/src/registry/manifest.js:16:57)
at /usr/local/lib/node_modules/plugman/src/registry/registry.js:124:22
at initSettings (/usr/local/lib/node_modules/plugman/src/registry/registry.js:207:5)
at Object.module.exports.publish (/usr/local/lib/node_modules/plugman/src/registry/registry.js:121:9)
at Object.module.exports [as publish] (/usr/local/lib/node_modules/plugman/src/publish.js:5:14)
at Object.plugman.commands.publish (/usr/local/lib/node_modules/plugman/plugman.js:84:17)
at Object.<anonymous> (/usr/local/lib/node_modules/plugman/main.js:75:26)

Comments
--------
It appears the dir argument passed on the command line is not being passed into registry.js line 121, which cause the downstream error.