[Bug 504405] Review Request: jaxodraw - A Java program for drawing Feynman diagrams

[bugzilla at redhat.com]

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=504405





--- Comment #14 from Orion Poplawski <orion at cora.nwra.com>  2009-10-22 17:12:41 EDT ---
Some other stuff:

- On KDE it appears in two different menus:  Graphics, and Science & Math.  I
don't have a problem with that, but don't know if it is intentional or not.

- Does not run on F12:

Exception in thread "main" java.lang.ExceptionInInitializerError
Caused by: java.lang.IllegalArgumentException: Consecutive slashes in path
        at
java.util.prefs.AbstractPreferences.node(AbstractPreferences.java:826)
        at
java.util.prefs.AbstractPreferences.node(AbstractPreferences.java:843)
        at
java.util.prefs.AbstractPreferences.node(AbstractPreferences.java:817)
        at net.sf.jaxodraw.util.JaxoPrefs.initSessionPrefs(JaxoPrefs.java:446)
        at net.sf.jaxodraw.JaxoDraw.<clinit>(JaxoDraw.java:34)
Could not find the main class: net/sf/jaxodraw/JaxoDraw. Program will exit.

- I know it is more of an application than a library, but perhaps you want to
build and package the javadocs?

%build
ant jar javadoc

puts javadocs in build/javadoc

- I think you should run the unit tests:

BuildRequires: ant-junit

%check
ant test

which for me results in:

    [junit] Testcase:
testGroupMarkedObjects(net.sf.jaxodraw.graph.JaxoGraphTest):      Caused an
ERROR
    [junit] missing initial moveto in path definition
    [junit] java.awt.geom.IllegalPathStateException: missing initial moveto in
path definition
    [junit]     at java.awt.geom.Path2D$Float.needRoom(Path2D.java:280)

-- 
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.