Added hash for dacapo dependency.#710
Conversation
Created new element in 'my %base' for dacapo-23.11-MR2-chopin-minimal.zip with hash defined by zip download url, downloaded zip name, and sha1 value of the downloaded zip. Fixes: #6326 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>
|
In PR description/commit message, you can reference the aqa-test repo issue. For example, Fixes: adoptium/aqa-tests#6326 |
Changed the hash filename for dacapo from dacapo-23.11-MR2-chopin-minimal.zip to dacapo.zip. Fixes: adoptium/aqa-tests#6326 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>
Local indentation looked different from remote indentation due to VSCode settings. This should fix it. Fixes: adoptium/aqa-tests#6326 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>
Nevermind, spaces was not the issue. Might have had whitespace characters that I didn't see. Fixes: adoptium/aqa-tests#6326 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>
|
https://openj9-jenkins.osuosl.org/job/test.getDependency/ has been broken since Jun 6th. Issue opened at adoptium/aqa-tests#6336 |
|
https://ci.adoptium.net/job/test.getDependency/2507/console failed with the dacapo SHA. @MattyWeee123 could you take a look? Thanks |
|
In buildenv/Jenkins/getDependency, the perl command should include a redirect flag -L, since the link in the getDependencies.pl is not a destination link. I accounted for this in dacapo/build.xml for adoptium/aqa-tests/#6326. Is there a reason why we do not use -L? The original example code for adoptium/aqa-tests/#6326 also assumed no need for -L. |
|
We never had to use Update: On second thought, let's try updating dacapo url only. Change https://github.com/MattyWeee123/TKG/blob/67d2cec326c46e42f7d3f1705249846f48a04f0f/scripts/getDependencies.pl#L207 |
Fixes: adoptium#710 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>
Fixes: adoptium#710 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>

Created new element in 'my %base' for
dacapo-23.11-MR2-chopin-minimal.zip with hash
defined by zip download url, downloaded zip name,
and sha1 value of the downloaded zip.
Fixes: adoptium/aqa-tests#6326