Bug#630542: yui-compressor: mangles strings containing backslash

2011-06-25 Thread Jakub Wilk 

* Damien Raude-Morvan draz...@drazzib.com, 2011-06-20, 22:24:

Could you please try yui-compressor package I've put here with your jQuery
build :
http://people.debian.org/~drazzib/yui/


Sorry for the late answer. With 2.4.6+rhino17R3-1, jquery fails to 
build:


| dh build
|dh_testdir
|dh_auto_configure
|debian/rules override_dh_auto_build
| make[1]: Entering directory `/tmp/jquery-1.6.1'
| `yui-compressor dist/jquery.js -o dist/jquery.min.js`
|
| [ERROR] 8936:11:unterminated comment
|
| [ERROR] 8936:11:missing ) after formal parameters
|
| [ERROR] 8936:11:missing } after function body
|
| [ERROR] 1:0:Compilation produced 3 syntax errors.
| org.mozilla.javascript.EvaluatorException: Compilation produced 3 syntax 
errors.
|   at 
com.yahoo.platform.yui.compressor.YUICompressor$1.runtimeError(YUICompressor.java:154)
|   at org.mozilla.javascript.Parser.parse(Parser.java:597)
|   at org.mozilla.javascript.Parser.parse(Parser.java:530)
|   at 
com.yahoo.platform.yui.compressor.JavaScriptCompressor.parse(JavaScriptCompressor.java:313)
|   at 
com.yahoo.platform.yui.compressor.JavaScriptCompressor.init(JavaScriptCompressor.java:536)
|   at 
com.yahoo.platform.yui.compressor.YUICompressor.main(YUICompressor.java:131)
|   at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
|   at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
|   at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
|   at java.lang.reflect.Method.invoke(Method.java:616)
|   at com.yahoo.platform.yui.compressor.Bootstrap.main(Bootstrap.java:21)
| make[1]: *** [override_dh_auto_build] Error 2
| make[1]: Leaving directory `/tmp/jquery-1.6.1'
| make: *** [build] Error 2
| dpkg-buildpackage: error: debian/rules build gave error exit status 2

--
Jakub Wilk



--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org

Bug#630542: yui-compressor: mangles strings containing backslash

2011-06-20 Thread Damien Raude-Morvan 
Hi Jakub,

Le mercredi 15 juin 2011 22:59:41, Damien Raude-Morvan a écrit :
 Le mercredi 15 juin 2011 01:22:12, Jakub Wilk a écrit :
  $ cat tmp.js
  x = \\;
  
  $ yui-compressor tmp.js; echo
  x=\;
 
 Thanks for you feedback, I can confirm that this break badly with jQuery.
 It seems regression testsuite in yui-compressor is not sufficient to check
 this case.

Could you please try yui-compressor package I've put here with your jQuery 
build :
http://people.debian.org/~drazzib/yui/

Thanks in advance,
-- 
Damien


signature.asc
Description: This is a digitally signed message part.

Bug#630542: yui-compressor: mangles strings containing backslash

2011-06-15 Thread Damien Raude-Morvan 
Hi Jakub,

Le mercredi 15 juin 2011 01:22:12, Jakub Wilk a écrit :
 Package: yui-compressor
 Version: 2.4.6+debian1-1
 Severity: grave
 
 $ cat tmp.js
 x = \\;
 
 $ yui-compressor tmp.js; echo
 x=\;

Thanks for you feedback, I can confirm that this break badly with jQuery. It 
seems regression testsuite in yui-compressor is not sufficient to check this 
case.

Can you provide me a working compressed-jQuery so I can compare and check 
there is no others regressions ?

For now, you can just use previous release (2.4.6-1) from testing.

 (Severity grave, because this badly breaks jQuery, and possibly lots of
 other code.)

ACK.

Cheers,
-- 
Damien


signature.asc
Description: This is a digitally signed message part.

Bug#630542: yui-compressor: mangles strings containing backslash

2011-06-14 Thread Jakub Wilk 

Package: yui-compressor
Version: 2.4.6+debian1-1
Severity: grave

$ cat tmp.js
x = \\;

$ yui-compressor tmp.js; echo
x=\;


(Severity grave, because this badly breaks jQuery, and possibly lots of 
other code.)


--
Jakub Wilk



--
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org