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

build_is_identical stabilization #398

Merged
merged 1 commit into from
Jan 19, 2018
Merged

Conversation

ittaiz
Copy link
Member

@ittaiz ittaiz commented Jan 18, 2018

No description provided.

@ittaiz
Copy link
Member Author

ittaiz commented Jan 18, 2018

@johnynek I was worried on how come the jmh problem never surfaced on CI and only locally and I have a theory that it's because build_is_identical ran before jmh and that inside the test itself it was "too quick". I moved build_is_identical to the end with a (hopefully) useful comment as well as added a 2 seconds sleep inside it to try and win different timestamps.
I also reordered a bit some of the tests like #219 so we can close it.

@ittaiz ittaiz mentioned this pull request Jan 18, 2018
@ittaiz
Copy link
Member Author

ittaiz commented Jan 19, 2018

@johnynek any objections?

@johnynek
Copy link
Member

no, this is great. Just frustrating that travis behaves so poorly.

@johnynek johnynek merged commit 7d02925 into bazelbuild:master Jan 19, 2018
@ittaiz
Copy link
Member Author

ittaiz commented Jan 19, 2018 via email

jmendiara added a commit to jmendiara/rules_scala that referenced this pull request Jun 20, 2018
Issues referenced for building in MAC (bazelbuild#136, bazelbuild#396 and bazelbuild#398)  are closed/merged

Having the comment `This doesn't currently pass on OS X (see bazelbuild#136 for details)` is not accurate. 

BTW, I cannot confirm it's true, cause in my env have bazel 0.14 and `test_all.sh` is failing
(`bazel test //test/...` is OK)
ittaiz pushed a commit that referenced this pull request Jun 20, 2018
Issues referenced for building in MAC (#136, #396 and #398)  are closed/merged

Having the comment `This doesn't currently pass on OS X (see #136 for details)` is not accurate. 

BTW, I cannot confirm it's true, cause in my env have bazel 0.14 and `test_all.sh` is failing
(`bazel test //test/...` is OK)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants