Skip to content

[AURON #2439] Add git to yum install - #2442

Merged
cxzl25 merged 3 commits into
apache:masterfrom
cxzl25:auron_2439
Jul 30, 2026
Merged

[AURON #2439] Add git to yum install#2442
cxzl25 merged 3 commits into
apache:masterfrom
cxzl25:auron_2439

Conversation

@cxzl25

@cxzl25 cxzl25 commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Closes #2439

Rationale for this change

https://github.com/apache/auron/actions/runs/30325123247/job/90213501717

build-release-1  | [ERROR] [mvn-builder-auron-core] ./dev/vendors/setup-vendors.sh: line 135: git: command not found

What changes are included in this PR?

  1. dev/docker-build/centos7/Dockerfile: install git alongside the other build tools
    (it was previously missing entirely, causing setup-vendors.sh to fail with
    git: command not found).
  2. dev/vendors/setup-vendors.sh: replace git -C "${target_dir}" apply --3way "${patch}"
    with (cd "${target_dir}" && git apply --3way "${patch}"), so patch application no longer
    depends on a git version >= 1.8.5.
  3. .github/workflows/build-amd64-releases.yml: add dev/docker-build/centos7/Dockerfile to
    the paths trigger list so changes to the build image are validated by this workflow on PRs.

Are there any user-facing changes?

No

How was this patch tested?

GHA

Was this patch authored or co-authored using generative AI tooling?

  • Yes
  • No

@cxzl25
cxzl25 marked this pull request as draft July 28, 2026 13:26

@slfan1989 slfan1989 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the contribution! The changes look reasonable and address the missing git dependency. However, the Build amd64 Releases workflow is still failing, so could you please check the failure and confirm that the original git: command not found issue has been resolved? Also, please complete the “What changes are included” and generative AI disclosure sections in the PR description.

@cxzl25
cxzl25 marked this pull request as ready for review July 29, 2026 04:13
@cxzl25
cxzl25 requested a review from slfan1989 July 29, 2026 06:39
@cxzl25
cxzl25 merged commit ce460e1 into apache:master Jul 30, 2026
136 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: setup-vendors.sh: line 135: git: command not found

2 participants