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



--- Comment #18 from gil cattaneo <punto...@libero.it> ---
(In reply to Tomas Repik from comment #17)


> ISSUES
> ======
> [!]: Requires correct, justified where necessary.
> - please justify the use of perl (in spec also)

I should add a comment in the spec file?
it is not obvious its use?

> QUESTIONS
> =========
> Are these two the only sources the package compiles?
> ECC-25519-Java/src/main/java/net/vrallev/java/ecc/Ecc25519Helper.java
> ECC-25519-Java/src/main/java/net/vrallev/java/ecc/KeyHolder.java

Yes, bundled code is removed in the  %prep section
Originally this library should contain:
ECC-25519-Java/src/main/java/djb curve25519-java or org.zeromq:curve25519-java
ECC-25519-Java/src/main/java/net/i2p ed25519-java or net.i2p.crypto:eddsa:0.1.0

thanks

-- 
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@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

Reply via email to