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.

Sign up for GitHub

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

Jump to bottom

Travis PR builds broken #116

Closed
dnakamura opened this issue Sep 22, 2017 · 2 comments
Closed

Travis PR builds broken #116

dnakamura opened this issue Sep 22, 2017 · 2 comments
Labels
comp:build

Comments

@dnakamura
Copy link
Contributor

dnakamura commented Sep 22, 2017 •
edited
Loading

Travis pull request builds are broken. They time out cloning openj9-openjdk-jdk9

remote: Counting objects: 565618, done.
remote: Compressing objects: 100% (13/13), done.
Receiving objects:  98% (558063/565618), 776.86 MiB | 160.00 KiB/s   
The job exceeded the maximum time limit for jobs, and has been terminated.

https://travis-ci.org/eclipse/openj9/builds/278682619

Builds on branches still seem ok I have seen the issue strike on branch builds but much less often than on PR builds
DanHeidinga added a commit to DanHeidinga/openj9 that referenced this issue Sep 23, 2017
@DanHeidinga
Use shallow clone of the openj9-openjdk-jdk9 repo in travis build
3e44e45 
Attempt to work around timeouts cloning this repo in the CI builds
by cloning less material.  This is fine since the build doesn't
commit back to that repo (or other repos for that matter)

Issue eclipse-openj9#116

Signed-off-by: Dan Heidinga <[email protected]>
@DanHeidinga DanHeidinga mentioned this issue Sep 23, 2017
Merged
@AdamBrousseau
Copy link
Contributor

Can this be closed?

@DanHeidinga
Copy link
Member

I think so. There has been other travis build flakiness but that's been related to source forge downloads of free marker.

This was referenced Feb 5, 2019
Merged
Closed
MarkQingGuo added a commit to MarkQingGuo/openj9 that referenced this issue Feb 5, 2019
@MarkQingGuo
A new target(java_spec_headers) is added in build-on-travis.mk
d2087b2 
A new target is necessary because it generates jvmti header files. Since we need to generate different jvmti header files for different JDKs and building with linter doesn't depend on extension repository, we have to modify makefiles so that it generates jvmti header files when build with linter.

PRs should be commited in such order:

* jvmti.h.m4 which generates jvmti header files(eclipse-openj9#4409)
* changes in extension repositories(JDK8, JDK11, JDK12)
* changes in JDK11(eclipse-openj9#116) in its specific extension repository
* changes in how openj9 are build with linter(eclipse-openj9#4605)

There are some changes:

* A new target(java_spec_headers) is added
* A new recipe of target java_spec_headers is added: it gets java_spec_version from j9cfg.h then uses it as a variable

Signed-off-by: MarkQingGuo <[email protected]>
tajila pushed a commit to tajila/openj9 that referenced this issue Jun 17, 2019
@gita-omr
Merge pull request eclipse-openj9#116 from Alen-Badel/static
8697607 
Fixing Clinit - Class initialization on P
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
comp:build
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@AdamBrousseau @DanHeidinga @dnakamura