-
-
Notifications
You must be signed in to change notification settings - Fork 308
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
Test Autogen: Reset JCK_GIT_REPO every loop #2425
Test Autogen: Reset JCK_GIT_REPO every loop #2425
Conversation
Could you resolve the conflicts? Thanks |
2a902b8
to
6db5d13
Compare
Rebased, and cleaned up some whitespace. |
if (GROUP == "jck" && JCK_GIT_REPO_PREFIX && !JCK_GIT_REPO) { | ||
JCK_GIT_REPO = "${JCK_GIT_REPO_PREFIX}/JCK${JDK_VERSION}-unzipped.git" | ||
ACTUAL_JCK_GIT_REPO = "${JCK_GIT_REPO_PREFIX}/JCK${JDK_VERSION}-unzipped.git" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the original code supports two cases:
- the user provides
JCK_GIT_REPO
repo. The script uses it to generate test jobs - the user does not provide
JCK_GIT_REPO
repo. The script setsJCK_GIT_REPO
to${JCK_GIT_REPO_PREFIX}/JCK${JDK_VERSION}-unzipped.git
// this is ideal for running in a loop
Also, the change in this PR will cause user-provided JCK_GIT_REPO
to be ignored.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I think I've fixed it now.
6db5d13
to
b14233e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
rebase required. |
Similar to what is done for ACTUAL_TEST_JOB_NAME, use and reset the ACTUAL_JCK_GIT_REPO every loop in order to hit the condition everytime. Otherwise JCK_GIT_REPO would be set on the first loop for the duration of the build. Signed-off-by: Adam Brousseau <[email protected]>
b14233e
to
78ea06a
Compare
rebased |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Similar to what is done for ACTUAL_TEST_JOB_NAME,
use and reset the ACTUAL_JCK_GIT_REPO every loop in
order to hit the condition everytime. Otherwise
JCK_GIT_REPO would be set on the first loop for the
duration of the build.
Signed-off-by: Adam Brousseau [email protected]
@llxia for review
Tested internal Test_Job_Auto_Gen_Sandbox/104