[Bug 921847] Review Request: ycssmin - CSS minification tool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=921847

--- Comment #4 from T.C. Hollingsworth <tchollingsworth@xxxxxxxxx> ---
(In reply to comment #3)
> The %description needs a bit of work:
> javascript->JavaScript
> css->CSS
> "for of"
> And I think some commas are missing.

Sorry, that's obivously a bad README copy/paste job.  I'll do it justice soon.

> /usr/bin/cssmin could also do with a man page.

I have a to-do list of a couple man pages.  I was waiting for nodejs-ronn to
get in so I could do them in Markdown.  I figure most nodejs upstreams will be
happier with that than groff.  ;-)

> Right, so let me get this straight...
> 
> ycssmin is Yahoo's fork of Johan Bleuzen's node-cssmin, which is a fork of
> Stoyan Stefanov's cssmin.js, which is a JavaScript port of YUICompressor's
> cssmin utility, which is a Java port of Isaac Schlueter's cssmin.
> 
> Forks are fine but my only concern is that /usr/bin/cssmin may clash if
> somebody wants to package eg Johan's version. It does look like ycssmin is
> the most healthy project so we probably want to make it the "default" cssmin
> anyway.

I consdiered it more of an unfork of Johan's cssmin, the upstream for YUI
Compressor just took back over the node fork.  YUI Compressor cssmin is written
in JS too, it just bundles the Mozilla Rhino server-side JS engine.  Are you
sure it was a fork of some izs cssmin?  I figured he just wrote the original
YUI version (along with a fair chunk of the rest of YUI  ;-).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=s62BdeL4bx&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review



[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]