Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=745515
Ville Skyttä <ville.skytta(a)iki.fi> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
CC| |ville.skytta(a)iki.fi
AssignedTo|nobody(a)fedoraproject.org |ville.skytta(a)iki.fi
Flag| |fedora-review?
--- Comment #1 from Ville Skyttä <ville.skytta(a)iki.fi> 2011-10-15 14:17:49 EDT ---
* LICENSE.TXT UTF-8 conversion should be from Windows-1252, not ISO-8859-1.
* Use %ant instead of plain ant.
* Upstream does not build or ship javadocs, should we? If you think it's a
good idea, please use %javadoc instead of plain javadoc, and crosslink them
with java javadocs (BuildRequires: java-javadoc + Requires: java-javadoc +
%javadoc -link %{_javadocdir}/java). Also, drop "-subpackages org" -
there's
nothing in it that ends up in javadocs with the currently used options.
* Add doc/README and doc/CHANGELOG to %docs.
* rhino dep and build dep needs to be versioned: >= 1.7R3
* Please include a script in the srpm that can be used to recreate the tarball,
for example like this one:
http://pkgs.fedoraproject.org/gitweb/?p=reptyr.git;a=blob;f=reptyr-snapsh...
* Unlike upstream 2.4.6 release, the packaged version seems to have some
comment related issues which I think are severe enough that they need to be
fixed before the package can be included, for example:
$ echo "/* hello */" | yuicompressor --type js
[ERROR] 1:11:unterminated comment
[ERROR] 1:0:Compilation produced 1 syntax errors.
org.mozilla.javascript.EvaluatorException: Compilation produced 1 syntax
errors.
at
com.yahoo.platform.yui.compressor.YUICompressor$1.runtimeError(YUICompressor.java:154)
[...]
--
Configure bugmail:
https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.