Bug 1228172

I'll take care of reviewing this one.

Hi,

I'm afraid that (after building and installing the package from bug 1228169) it does not build for me:

[ERROR] Exit code: 1 - /home/hans/rpmbuild/BUILD/jedis-jedis-2.7.2/src/main/java/redis/clients/jedis

<snip>

[ERROR] /home/hans/rpmbuild/BUILD/jedis-jedis-2.7.2/src/main/java/redis/clients/jedis/BinaryJedis.java:326: error: malformed HTML
[ERROR] * @see <ahref="http://code.google.com/p/redis/wiki/ExpireCommand">ExpireCommand</a>
[ERROR] ^
[ERROR] /home/hans/rpmbuild/BUILD/jedis-jedis-2.7.2/src/main/java/redis/clients/jedis/BinaryJedis.java:326: error: bad use of '>'

And a few more of these (and also lots of javadoc warnings), I've had build failures of several of my own java packages during the F-23 rebuild, these are caused by javadoc in the F-23/rawhide jdk being stricter.

I've managed to reproduce this issue on F-22 by installing the latest openjdk from F-23 on F-22:
http://koji.fedoraproject.org/koji/buildinfo?buildID=651231

This works fine on F-22 and gives you the stricter javadoc, which should allow you to reproduce and then fix this problem.

Also I've a question about the .spec file:

# take too much time and use web connection
%mvn_build -f

Can you make the comment better so that it explains things a bit more, also use a web connection? I know maven normally does that, but I thought that %mvn_build disables that ?

Regards,

Hans

(In reply to Hans de Goede from comment #2)
> Hi,
> 
> I'm afraid that (after building and installing the package from bug 1228169)
> it does not build for me:
> 
> [ERROR] Exit code: 1 -
> /home/hans/rpmbuild/BUILD/jedis-jedis-2.7.2/src/main/java/redis/clients/jedis
> 
> <snip>
> 
> [ERROR]
> /home/hans/rpmbuild/BUILD/jedis-jedis-2.7.2/src/main/java/redis/clients/
> jedis/BinaryJedis.java:326: error: malformed HTML
> [ERROR] * @see
> <ahref="http://code.google.com/p/redis/wiki/ExpireCommand">ExpireCommand</a>
> [ERROR] ^
> [ERROR]
> /home/hans/rpmbuild/BUILD/jedis-jedis-2.7.2/src/main/java/redis/clients/
> jedis/BinaryJedis.java:326: error: bad use of '>'
> 
> And a few more of these (and also lots of javadoc warnings), I've had build
> failures of several of my own java packages during the F-23 rebuild, these
> are caused by javadoc in the F-23/rawhide jdk being stricter.
for a maven build you should remove maven-javadoc-plugin from POM file/s
for a ant build you should add:
e.g.
               version="true"
               use="true"
               windowtitle="Classycle"
-              failonerror="true"/>
+              failonerror="true">
+      <arg value="-Xdoclint:none"/>
+    </javadoc>
or
e.g.
     <javadoc destdir="${javadoc.dir}" source="1.6" failonerror="true"
       access="protected" 
+      additionalparam="-Xdoclint:none"
       windowtitle="JGraphT : a free Java graph library"
       overview="${src.dir}/overview.html">
       <classpath refid="build.classpath" />

> I've managed to reproduce this issue on F-22 by installing the latest
> openjdk from F-23 on F-22:
> http://koji.fedoraproject.org/koji/buildinfo?buildID=651231
FiXED, remove maven-javadoc-plugin to use default javadoc setting to override doclint issues

> This works fine on F-22 and gives you the stricter javadoc, which should
> allow you to reproduce and then fix this problem.
> 
> Also I've a question about the .spec file:
> 
> # take too much time and use web connection
> %mvn_build -f
> 
> Can you make the comment better so that it explains things a bit more, also
> use a web connection? I know maven normally does that, but I thought that
> %mvn_build disables that ?
The test suite require unavailable resources, especially if the builder is a ARM arch based.
These builders are pretty slow compared to other arch.
Moreover, we don't have JIT enabled on this architecture, so everything
runs in interpreted mode. Package builds can be one to three orders of
magnitude slower on armv7hl compared to x86_64 and often fail due to
timeouts or OutOfMemoryError.
For "real" builds the only known workaround is
resubmitting the build until you get non-ARM builder.
Is a waste of time and resources.
> Regards,
> 
> Hans

Spec URL: https://gil.fedorapeople.org/jedis.spec
SRPM URL: https://gil.fedorapeople.org/jedis-2.7.2-2.fc20.src.rpm

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=10242377

Spec URL: https://gil.fedorapeople.org/jedis.spec
SRPM URL: https://gil.fedorapeople.org/jedis-2.7.2-2.fc22.src.rpm

Hi,

(In reply to gil cattaneo from comment #6)
> Spec URL: https://gil.fedorapeople.org/jedis.spec
> SRPM URL: https://gil.fedorapeople.org/jedis-2.7.2-2.fc22.src.rpm

Thanks, sorry for being a bit slow to respond.

Full review done:

Good:
- rpmlint checks return:
  3 packages and 0 specfiles checked; 0 errors, 0 warnings.
- package meets naming guidelines
- package meets packaging guidelines
- license (MIT) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file 

Everything looks good: Approved.

Regards,

Hans

Thanks for the review!

New Package SCM Request
=======================
Package Name: jedis
Short Description: A redis Java client
Upstream URL: https://github.com/xetorthio/jedis
Owners: gil
InitialCC: java-sig

Git done (by process-git-requests).

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=10329821

Package Change Request
======================
Package Name: jedis
New Branches: f22
Owners: gil
InitialCC: java-sig

Git done (by process-git-requests).

jedis-2.7.2-2.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2015-14667

jedis-2.7.2-2.fc22 has been pushed to the Fedora 22 testing repository. If problems still persist, please make note of it in this bug report.\nIf you want to test the update, you can install it with \n su -c 'yum --enablerepo=updates-testing update jedis'. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2015-14667

jedis-2.7.2-2.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report.