Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problems with Rhino #321

Open
foldi opened this issue Nov 14, 2012 · 18 comments
Open

Problems with Rhino #321

foldi opened this issue Nov 14, 2012 · 18 comments

Comments

@foldi
Copy link

foldi commented Nov 14, 2012

CSSLint seems to stall when running via Rhino from the command line.

I'm on os x 10.7.5 and have downloaded the latest Rhino (1.7R4), the latest csslint (v0.9.9 Build time: 12-September-2012 01:46:26), and have updated Java.

With three files in one folder, csslint-rhino.js, js.jar, and main.css, my command looks like this:

java -jar js.jar csslint-rhino.js main.css

The terminal stalls and never reports a success or fail.

Thanks.

@nzakas
Copy link
Contributor

nzakas commented Nov 14, 2012

Hmm, on my initial look, I'm able to get it to work correctly. I'll try on another machine.

@chase-seibert
Copy link

I am able to repo as well. Also on OSX.

java version "1.6.0_35"
rhino 1.7R4
csslint-rhino 12-September-2012 01:46:26

Hangs even using a nearly empty CSS file, with just one empty class in it.

Tried csslint-rhino 14-May-2012 10:24:48, same thing.
Latest Node.js version works fine.

@nzakas
Copy link
Contributor

nzakas commented Nov 28, 2012

Sorry, I'm still not able to reproduce. I've tried on a couple of different machines with both Java 1.6 and 1.7 and it works fine for me. How are you using it? Here's what I typed (from the csslint directory):

java -jar lib/js.jar release/csslint-rhino.js demos/demo.css

@chase-seibert
Copy link

Extract: https://dl.dropbox.com/u/422013/csslint-test.tar.gz

java -jar js.jar csslint-rhino.js test.css

@chase-seibert
Copy link

java -version

java version "1.6.0_35"
Java(TM) SE Runtime Environment (build 1.6.0_35-b10-428-11M3811)
Java HotSpot(TM) 64-Bit Server VM (build 20.10-b01-428, mixed mode)

@akmurray
Copy link

akmurray commented Dec 8, 2012

Same problem (hanging - 1 CPU core maxed) for me on Windows 7.

Java 1.7.0_09-b05
rhino1_7R4

java -jar rhino-js.jar csslint-rhino.js test.css

@akmurray
Copy link

akmurray commented Dec 8, 2012

This is a problem with Rhino. I downloaded an old version of csslint and it still hangs. I downloaded an old version of Rhino and it works

@nzakas
Copy link
Contributor

nzakas commented Dec 10, 2012

Can you post which versions of Rhino you tested?

On Fri, Dec 7, 2012 at 4:08 PM, Aaron Murray notifications@github.comwrote:

This is a problem with Rhino. I downloaded an old version of csslint and
it still hangs. I downloaded an old version of Rhino and it works


Reply to this email directly or view it on GitHubhttps://github.com//issues/321#issuecomment-11151744.


Nicholas C. Zakas
@SlickNet

Author, Professional JavaScript for Web Developers
Buy it at Amazon.com:
http://www.amazon.com/Professional-JavaScript-Developers-Nicholas-Zakas/dp/1118026691/ref=sr_1_3

@akmurray
Copy link

rhino1_7R4 (current from their site)

@nzakas
Copy link
Contributor

nzakas commented Dec 10, 2012

Is that the version that worked or the version that didn't? It would be
very helpful if you could provide both.

On Mon, Dec 10, 2012 at 2:53 PM, Aaron Murray notifications@github.comwrote:

rhino1_7R4 (current from their site)


Reply to this email directly or view it on GitHubhttps://github.com//issues/321#issuecomment-11222457.


Nicholas C. Zakas
@SlickNet

Author, Professional JavaScript for Web Developers
Buy it at Amazon.com:
http://www.amazon.com/Professional-JavaScript-Developers-Nicholas-Zakas/dp/1118026691/ref=sr_1_3

@akmurray
Copy link

Check a prior post for the version that didn't work - I already posted that once before. The version that works is from the html5boilerplate project - I'm not at my computer right now

On Dec 10, 2012, at 6:00 PM, "Nicholas C. Zakas" notifications@github.com wrote:

Is that the version that worked or the version that didn't? It would be
very helpful if you could provide both.

On Mon, Dec 10, 2012 at 2:53 PM, Aaron Murray notifications@github.comwrote:

rhino1_7R4 (current from their site)


Reply to this email directly or view it on GitHubhttps://github.com//issues/321#issuecomment-11222457.


Nicholas C. Zakas
@SlickNet

Author, Professional JavaScript for Web Developers
Buy it at Amazon.com:
http://www.amazon.com/Professional-JavaScript-Developers-Nicholas-Zakas/dp/1118026691/ref=sr_1_3

Reply to this email directly or view it on GitHub.

@jwmcleod
Copy link

I've encountered this too. It looks to be a problem with rhino:

The problem appears to be in regular expressions involving \uFFFF:

$ java -jar js.jar -e "print(/[a-z_\u0080-\uFFFF]/i.test(\"x\"))"

hangs, but

$ java -jar js.jar -e "print(/[a-z_\u0080-\uFFFE]/i.test(\"x\"))"
true

doesn't. As a temporary fix, I've just changed all occurrences of \uFFFF to \uFFFE in csslint-rhino.js.

@nschonni
Copy link
Member

nschonni commented Feb 4, 2013

any progress on this?

@nzakas
Copy link
Contributor

nzakas commented Feb 5, 2013

Nope. I've not been able to verify versions where it's working versus where
it's not working. If anyone else wants to take a stab at that, please let
me know and I'll be happy to update the docs to indicate the appropriate
versions to use.

On Sun, Feb 3, 2013 at 4:18 PM, Nick Schonning notifications@github.comwrote:

any progress on this?


Reply to this email directly or view it on GitHubhttps://github.com//issues/321#issuecomment-13058582.


Nicholas C. Zakas
@SlickNet

Author, Professional JavaScript for Web Developers
Buy it at Amazon.com:
http://www.amazon.com/Professional-JavaScript-Developers-Nicholas-Zakas/dp/1118026691/ref=sr_1_3

@cowwoc
Copy link

cowwoc commented Feb 9, 2013

I filed this bug report: mozilla/rhino#107

@jwmcleod
Copy link

I've grabbed the latest version of Rhino from the source code repository (it builds as 7R5pre), and the problem has been fixed. Now all we need to do is wait for an official release...

@RamiroCampa
Copy link

Rhino1_6R7 and csslint v0.9.10 work fine checked!

@ejzn
Copy link

ejzn commented Jun 20, 2014

Just wanted re-iterate that this is still Obviously a problem with the release candidate. This thread saved me hours of trouble.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants