What steps will reproduce the problem?
- git clone git://github.com/douglascrockford/JSLint.git
- add this config to the pom in maven: <jslintSource>${basedir}/src/tests/jslint/jslint.js</jslintSource>
- mvn jslint4java:lint
What is the expected output? What do you see instead?
[ERROR] Failed to execute goal com.googlecode.jslint4java:jslint4java-maven-plugin:2.0.2:lint (default-cli) on project: Execution default-cli of goal com.googlecode.jslint4java:jslint4java-maven-plugin:2.0.2:lint failed: TypeError: Cannot read property "length" from undefined (/src/tests/jslint/jslint.js#6336) -> [Help 1]
What version of the product are you using? On what operating system?
jslint4java-2.0.2 on Mac OS X 10.7.3 with jslint.js-2012-05-09
Please provide any additional information below.
Works great with jslint.js-2012-04-15, which is the previous version. Once cloned, you can get it at: git checkout 4f2bf65ed59a0ba16fb22a6a90f76fc0060a2c22
Fails with jslint.js-2012-05-09 which you can get at: git checkout 097ba532870c9c358f6c535d0712171d56a3f2a6
It appears the main difference between the two versions is the HTML report changed from JSLINT.report() to JSLINT.error_report() and now requires JSLINT.data() to be passed into it vs. the bool for errors_only.
https://github.com/douglascrockford/JSLint/commit/097ba532870c9c358f6c535d0712171d56a3f2a6#diff-3
Comment #1
Posted on Jun 5, 2012 by Happy BearGood catch! I really need to get a CI build up and running…
Comment #2
Posted on Jun 6, 2012 by Happy Bearhttps://github.com/happygiraffe/jslint4java/compare/34e3f83319...a490ae118e
Status: Fixed
Labels:
Type-Defect
Priority-Medium
Milestone-Release2.0.3