Skip to content

Picked up _JAVA_OPTIONS: -Djava.io.tmpdir=/home/jenkins/tmp printed in partests #1712

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

Closed
nicolasstucki opened this issue Nov 15, 2016 · 9 comments

Comments

@nicolasstucki
Copy link
Contributor

It looks like when executing partest all tests fail because of Picked up _JAVA_OPTIONS: -Djava.io.tmpdir=/home/jenkins/tmp being printed at the start of each one. I've seen it in #1711 and #1685.

% diff /home/jenkins/workspace/dotty-master-validate-partest-bootstrapped/tests/partest-generated/run/view-headoption-run.log /home/jenkins/workspace/dotty-master-validate-partest-bootstrapped/tests/partest-generated/run/view-headoption.check
% 
@@ -1,4 +1,3 @@
-Picked up _JAVA_OPTIONS: -Djava.io.tmpdir=/home/jenkins/tmp
 fail
 success
 f1: Some(5)
@nicolasstucki
Copy link
Contributor Author

It has also affected #1700. All PRs that where updated this afternoon.

@smarter
Copy link
Member

smarter commented Nov 15, 2016

We have a disabled "filters" file for this, no clue how it's supposed to work: https://github.com/lampepfl/dotty/blob/master/tests/untried/filters#L6

@smarter
Copy link
Member

smarter commented Nov 15, 2016

In our case srcDir is tests/partest-generated/ so we have to put the filters file in there somehow

@nicolasstucki
Copy link
Contributor Author

I still can't reproduce it locally.

@nicolasstucki
Copy link
Contributor Author

Testing #1714 on CI to see if reverting the PRs merged today has some different results.

@liufengyun
Copy link
Contributor

It also cause test.TestDotc.buildAndRunHelloWorld junit test fail.

@liufengyun
Copy link
Contributor

liufengyun commented Nov 15, 2016

It can be reproduced as follows:

export _JAVA_OPTIONS=""
sbt "testOnly -- test.TestDotc.buildAndRunHelloWorld"

A related post at SO here.

I guess the settings on Jenkins have changed somehow.

Edited: update cmd to bash syntax

@DarkDimius
Copy link
Contributor

I have a fix available in #1715

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants