Skip to content

Set "-Xdoclint:none" only when the Java is 1.8 or more - #172

Merged
xerial merged 2 commits into
v07-developfrom
xdoclint_none_on_java18
Jan 14, 2015
Merged

Set "-Xdoclint:none" only when the Java is 1.8 or more#172
xerial merged 2 commits into
v07-developfrom
xdoclint_none_on_java18

Conversation

@komamitsu

Copy link
Copy Markdown
Member

When executing "publishLocal" on sbt, it failed with an error message [error] javadoc: error - invalid flag: -Xdoclint:none. I guess the reason why it failed is javadoc doesn't has the option in Java1.7 or before. If all the msgpack-java developers use Java1.8, we have nothing to do to fix this issue. But I don't think the limitation is so good. I fixed this issue but I'm not sure if this change is the best.

@xerial Can you review this PR?

@xuwei-k

xuwei-k commented Jan 12, 2015

Copy link
Copy Markdown
Contributor

@xerial

xerial commented Jan 13, 2015

Copy link
Copy Markdown
Member

@komamitsu
Use scala.util.isJavaAtLeast as @xuwei-k suggested.

Comment thread project/Build.scala Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

scala.util.Properties.isJavaAtLeast("1.8")

@komamitsu

Copy link
Copy Markdown
Member Author

@xuwei-k @xerial Thanks. I replaced it with scala.util.Properties.isJavaAtLeast()

xerial added a commit that referenced this pull request Jan 14, 2015
Set "-Xdoclint:none" only when the Java is 1.8 or more
@xerial
xerial merged commit 6c8d9da into v07-develop Jan 14, 2015
@xerial

xerial commented Jan 14, 2015

Copy link
Copy Markdown
Member

thanks!

@xerial
xerial deleted the xdoclint_none_on_java18 branch May 28, 2015 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants