[
https://issues.apache.org/jira/browse/CB-13451?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16213034#comment-16213034
]
ASF GitHub Bot commented on CB-13451:
-------------------------------------
Github user stevengill commented on a diff in the pull request:
https://github.com/apache/cordova-lib/pull/597#discussion_r146039059
--- Diff: src/plugman/init-defaults.js ---
@@ -55,25 +55,26 @@ function readDeps (test) {
};
}
-var name = pkg.name || defaults.id || basename;
+/* eslint-disable */
--- End diff --
Do you need this `/* eslint-disable */`
> "pkg not defined" exception when running plugman with createpackagejson
> command-line
> ------------------------------------------------------------------------------------
>
> Key: CB-13451
> URL: https://issues.apache.org/jira/browse/CB-13451
> Project: Apache Cordova
> Issue Type: Bug
> Reporter: Roland
> Priority: Minor
>
> NOTE: I am about to submit a pull request with a fix for this issue.
> Description follows:
> A global js-lint pass on the cordova-lib codebase on the 30/8/2017 caused
> references to the global 'package' variable in init-default.js to be renamed
> to 'pkg', presumably because the cordova-lib source doesn't declare a global
> variable called 'package' but _does_ declare a global variable called 'pkg'
> (in cordova/info.js). However in this case, the 'package' variable refers to
> the one declared in the plugman source, specifically main.js, so it should
> have stayed as 'package'. To test the fix, run:
> plugman createpackagejson .
> Without the fix, this will trigger the following exception: 'pkg is not
> defined'. Having patched in the fix, running the command line should work as
> expected, prompting the user with questions and then spitting out a
> package.json file.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
