[Bug 1415164] Re: [MIR] slimit

2015-10-20 Thread Adam Conrad
Based on mterry's comments, and lamont's latest upload enabling the testsuite, I've promoted slimit to main now. ** Changed in: slimit (Ubuntu) Status: New => Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https:/

[Bug 1415164] Re: [MIR] slimit

2015-10-20 Thread Matthias Klose
see also https://lists.ubuntu.com/archives/ubuntu- devel/2015-October/038928.html -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1415164 Title: [MIR] slimit To manage notifications about this bug go

[Bug 1415164] Re: [MIR] slimit

2015-03-04 Thread Michael Terry
Those parse errors are https://github.com/yui/yuicompressor/issues/47, fixed by updating to a newer embedded version of rhino. Which yuicompressor doesn't support yet. Plus, embedded copies of rhino don't excite me anyway. So I'm warming to slimit, even though I'm nervous that it hasn't had a re

[Bug 1415164] Re: [MIR] slimit

2015-02-27 Thread Andres Rodriguez
We can look into that for sure, but that being said, this would unblock us as the new release in Vivid is coming soon. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1415164 Title: [MIR] slimit To m

[Bug 1415164] Re: [MIR] slimit

2015-02-04 Thread Michael Terry
:( OK. I'm curious how hard it would be to port the other packages to slimit. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1415164 Title: [MIR] slimit To manage notifications about this bug go t

[Bug 1415164] Re: [MIR] slimit

2015-02-04 Thread Blake Rouse
Using the latest release of yuicompressor fails. blake@blake-ubnt-laptop:/tmp$ java -jar yuicompressor-2.4.8.jar angular.js -o angular.min.js [ERROR] in angular.js 237:11:identifier is a reserved word [ERROR] in angular.js 237:21:illegal character [ERROR] in angular.js 237:23:syntax error [

[Bug 1415164] Re: [MIR] slimit

2015-02-03 Thread Michael Terry
That looks like https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=682069 Apparently updating to 2.4.8 will fix it. But ultimately, that stacktrace apparently results from invalid javascript. So here are some options: 1) Fix invalid javascript that is causing this problem (seems like a good idea

Re: [Bug 1415164] Re: [MIR] slimit

2015-02-03 Thread Andres Rodriguez
Hi Michael, When using yui-compressor, the package FTBFS with the following: http://pastebin.ubuntu.com/10041083/ We can definitely look at converting all the other packages to use slimit instead. Cheers. On Tue, Feb 3, 2015 at 9:44 PM, Michael Terry wrote: > Oh, unless angular.js uses the

[Bug 1415164] Re: [MIR] slimit

2015-02-03 Thread Michael Terry
Oh, unless angular.js uses the actual python library provided by slimit? -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1415164 Title: [MIR] slimit To manage notifications about this bug go to: http

[Bug 1415164] Re: [MIR] slimit

2015-02-03 Thread Michael Terry
It doesn't work? They both provide command lines that accept input files and spit out the minified javascript. How different can they be? Does angular.js rely on specific minimization techniques of slimit? If you do require slimit, is there a chance you could port the following reverse-depends o

[Bug 1415164] Re: [MIR] slimit

2015-02-02 Thread Andres Rodriguez
We have tried, yui-compressor does not work with angular.js, hence, we needed to add this new dependency. ** Changed in: slimit (Ubuntu) Status: Incomplete => New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.lau

[Bug 1415164] Re: [MIR] slimit

2015-02-02 Thread Michael Terry
We already have yui-compressor in main. Is there a reason that wouldn't work instead of slimit? ** Changed in: slimit (Ubuntu) Status: New => Incomplete -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.ne