[Bug 857843] Review Request: lessjs - Less.js The dynamic stylesheet language

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

 



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

--- Comment #3 from Matthias Runge <mrunge@xxxxxxxxxx> ---
lessjs uses cssmin.js to shrink css code.

The usage of cssmin.js is harmless; yui compressor is deprecated by its
upstream, and currently lessjs is the only package using that:
yum search */cssmin.js
Loaded plugins: langpacks, presto, refresh-packagekit
Warning: No matches found for: */cssmin.js
No Matches found

Reference upstream:
- http://www.yuiblog.com/blog/2012/10/16/state-of-yui-compressor/

Also, one may assume, there's a general exception for JavaScript libraries from
strictly no bundling policy:

http://fedoraproject.org/wiki/Packaging:Guidelines#Duplication_of_system_libraries

Nevertheless, I did some minor tweaks:

SPEC: http://www.matthias-runge.de/fedora/lessjs.spec
SRPM: http://www.matthias-runge.de/fedora/lessjs-1.3.1-2.fc18.src.rpm

it builds in koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=4802144

and still has some spelling warnings:
[mrunge@turing SPECS]$ rpmlint ./lessjs.spec
../RPMS/noarch/lessjs-1.3.1-2.fc18.noarch.rpm
../SRPMS/lessjs-1.3.1-2.fc18.src.rpm 
./lessjs.spec:12: W: macro-in-comment %{version}
./lessjs.spec: W: invalid-url Source0: less.js-1.3.1.tar.gz
lessjs.noarch: W: spelling-error Summary(en_US) js -> dis, ks, j
lessjs.noarch: W: spelling-error Summary(en_US) stylesheet -> style sheet,
style-sheet, stylishness
lessjs.noarch: W: spelling-error %description -l en_US mixins -> mixing, mix
ins, mix-ins
lessjs.noarch: W: spelling-error %description -l en_US js -> dis, ks, j
lessjs.noarch: W: no-documentation
lessjs.noarch: W: no-manual-page-for-binary lessc
lessjs.src: W: spelling-error Summary(en_US) js -> dis, ks, j
lessjs.src: W: spelling-error Summary(en_US) stylesheet -> style sheet,
style-sheet, stylishness
lessjs.src: W: spelling-error %description -l en_US mixins -> mixing, mix ins,
mix-ins
lessjs.src: W: spelling-error %description -l en_US js -> dis, ks, j
lessjs.src:12: W: macro-in-comment %{version}
lessjs.src: W: invalid-url Source0: less.js-1.3.1.tar.gz
2 packages and 1 specfiles checked; 0 errors, 14 warnings.

-- 
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=roRyrQEbgz&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]