[Bug 1228172] Review Request: jedis - A redis Java client

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

 



https://bugzilla.redhat.com/show_bug.cgi?id=1228172

Hans de Goede <hdegoede@xxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
              Flags|                            |fedora-review?



--- Comment #2 from Hans de Goede <hdegoede@xxxxxxxxxx> ---
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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
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]