[VL] Fix weekly build job on Centos 7#9294
Conversation
|
Thanks for opening a pull request! Could you open an issue for this pull request on Github Issues? https://github.com/apache/incubator-gluten/issues Then could you also rename commit message and pull request title in the following format? See also: |
| cd "${DEPENDENCY_DIR}" | ||
| wget_and_untar https://github.com/davea42/libdwarf-code/archive/refs/tags/20210528.tar.gz dwarf | ||
| cd dwarf | ||
| pushd dwarf |
There was a problem hiding this comment.
it seems the dwarf is introduced by other dependency, not the one here
There was a problem hiding this comment.
@zhouyuan, thanks for your comment!
The installation for dwarf succeeds as you noted. But it doesn't go back to the original directory. Then, dwarf directory will be wrongly used when executing the build commands for other dependency. This issue was related to the recent pr #9207 which changes the script to use upstream Velox wget_and_untar function.
This pr has fixed the failure. See
https://github.com/apache/incubator-gluten/actions/runs/14413849930/job/40427128080?pr=9294. We can merge this pr now if no more comment (please ignore the latest CI failure since we need some code changes in yml file simply for pr validation and I have reverted those changes).
This reverts commit d243568.
2 participants
What changes were proposed in this pull request?
This issue was related to the recent pr #9207 which changes the script to use upstream Velox wget_and_untar function.