Build fails while compiling code generated by Swig about simpleelastix HOT 7 CLOSED

I am able to reproduce the problem on my mac with SimpleElastix and the SimpleITK release, master and next branches. May I humbly ask you @blowekamp if you have any suggestions?

from simpleelastix.

What version of Java are you using?

from simpleelastix.

I am running

$ java -version
java version "1.8.0_51"
Java(TM) SE Runtime Environment (build 1.8.0_51-b16)
Java HotSpot(TM) 64-Bit Server VM (build 25.51-b03, mixed mode)

on Mac OS X El Capitan Version 10.11.3.

from simpleelastix.

I have seen this error on several systems including windows, and I am able to get it on my updated OS X version. Currently there are not too many build systems on the SimpleITK dashboard with Java being built on it. Those that have it are either version 1.7, or 1.6 for distribution.

I am not sure if this is a SWIG or a Java problem. Perhaps that may be a flag that needs to be passed to either SWIG or Java to fix this problem.

Help with the issue in the upstream would be appreciated.

from simpleelastix.

Sure, I'll make a pull request. The problem seems to be related to the Java 8 javadoc compilation as nicost suspected. There are two fixes according to this stack overflow question (summarized here for your convenience):

• Disable the doclint feature by adding -Xdoclint:none to the invocation of javadoc
• Fix javadoc errors. Don't know if this is feasible (do you?). Depends if the comments are pulled from the json files or generated by SWIG.

Which one do you prefer?

from simpleelastix.

If you want to improve the java documentation output to make it compliant, it certainly would be welcome :) But I don't think that is a reasonable task.

Please add the flag when Java >= 1.8, and provide clear comments for the reason for the flag. A pull request through SimpleITK/ITK gerrit preferred, but if done through github it's still welcomed.

Also please note that upstreamed has refactored ( still a little more todo there) the Wrapping directory.

Thank you!

from simpleelastix.

PR submitted and accepted. Changes will be merged into SimpleElastix with the next SimpleITK release.

from simpleelastix.