https://bugzilla.redhat.com/show_bug.cgi?id=1286467 --- Comment #3 from gil cattaneo <puntogil@xxxxxxxxx> --- (In reply to Tomas Repik from comment #2) > ===== Issues ===== > > [!]: License headers in some files missing. I've seen that you have made a > pull request solving this issue, I'd wait a bit until it's processed, so the > package would be nice and tidy to ship. Reported @ https://github.com/virtuald/curvesapi/issues/1 PR https://github.com/virtuald/curvesapi/pull/2 This is a fork of https://sourceforge.net/projects/curves/ but now SF site seem not available. And i also open a "discussion" about this problem > ===== Non-blocking ===== > > I wonder how it's gonna be with the dependencies. Doesn't the > sun.awt.geom.Curve file need to be mentioned in requires somehow? No. requires are generated by maven-local/javapackages-tools > Also I noticed there is a GUI demo application and I wonder why you included > only the source files in the documentation. I'd rather build a binary so > that the could be executed. It's mentioned in readme.txt. No, is useless. The package is necessary only for upgrade apache-poi > I have to say you did a very good job on this package. 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@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review