Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failing Windows Tests #502

Closed
muuki88 opened this issue Feb 22, 2015 · 7 comments
Closed

Failing Windows Tests #502

muuki88 opened this issue Feb 22, 2015 · 7 comments
Labels

Comments

@muuki88
Copy link
Contributor

muuki88 commented Feb 22, 2015

On AppVeyor the following tests fail for strange reasons

  • scripted jar/*
  • scripted windows/test-bat-template

More Details

cc @nazoking @bjuric , if you have any ideas :)

@nazoking
Copy link
Contributor

wmm..

this error cause is _JAVA_OPTS.

Java HotSpot(TM) 64-Bit Server VM warning: ignoring option MaxPermSize=2g; support was removed in 8.0

it set here.

https://ci.appveyor.com/project/muuki88/sbt-native-packager/build/6#L15

can you remove this MaxPermSize option ?

muuki88 added a commit that referenced this issue Feb 23, 2015
@muuki88
Copy link
Contributor Author

muuki88 commented Feb 23, 2015

Thanks for the fast response 👍
I remove the whole JAVA_OPTS line, with the same result.

https://ci.appveyor.com/project/muuki88/sbt-native-packager/build/12

@nazoking
Copy link
Contributor

Merge miss occured. on 3577a55#diff-0893ecb3791881c87ff523facf8b1880R140

This 2 lines removed on b5a259d

@nazoking
Copy link
Contributor

And maybe java change ProcessBuilder cmd escape algorithm .. surely https://blogs.oracle.com/thejavatutorials/entry/changes_to_runtime_exec_problems ...

@muuki88
Copy link
Contributor Author

muuki88 commented Feb 24, 2015

Did you test this on java 6? Maybe I should try this as well :/

@nazoking
Copy link
Contributor

i create #507. it success on jdk1.7.0_25, jdk1.8.0_20

@nazoking
Copy link
Contributor

I tried on jdk1.6.0_27, but it has compile error. jdk6 dont have java.nio.file

muuki88 added a commit that referenced this issue Feb 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants