Skip to content

[VL] Replace wget w/ wget_and_untar to make dependency download more reliable#9444

Merged
philo-he merged 3 commits intoapache:mainfrom
yaooqinn:centos7
Apr 28, 2025
Merged

[VL] Replace wget w/ wget_and_untar to make dependency download more reliable#9444
philo-he merged 3 commits intoapache:mainfrom
yaooqinn:centos7

Conversation

@yaooqinn
Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

Using wget_and_untar from upstream that accepts options for download tools

How was this patch tested?

@yaooqinn yaooqinn requested a review from philo-he April 28, 2025 07:00
@github-actions github-actions bot added the VELOX label Apr 28, 2025
@github-actions
Copy link
Copy Markdown

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?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

Copy link
Copy Markdown
Member

@philo-he philo-he left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your work!

Copy link
Copy Markdown
Member

@philo-he philo-he left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@philo-he philo-he merged commit 304fb27 into apache:main Apr 28, 2025
44 of 45 checks passed
@yaooqinn yaooqinn deleted the centos7 branch April 29, 2025 02:44
baibaichen pushed a commit to baibaichen/gluten that referenced this pull request Apr 29, 2025
baibaichen pushed a commit to baibaichen/gluten that referenced this pull request Apr 29, 2025
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.

2 participants