yaooqinn opened a new pull request, #11515:
URL: https://github.com/apache/incubator-gluten/pull/11515

   ## What changes are proposed in this pull request?
   
   This PR migrates GitHub Actions workflows and Dockerfiles to use the 
`./build/mvn` Maven wrapper instead of manually installing Maven via 
`setup-helper.sh`. The wrapper automatically downloads Maven 3.9.12 when 
needed, simplifying the CI setup and reducing maintenance overhead.
   
   **Changes:**
   - Delete `.github/workflows/util/setup-helper.sh` (no longer needed)
   - Update workflows to use `./build/mvn`:
     - `build_bundle_package.yml`
     - `velox_nightly.yml` (6 jobs)
     - `velox_weekly.yml`
     - `scala_code_format.yml` (remove unused SETUP env var)
   - Update Dockerfiles to use `./build/mvn`:
     - `dev/docker/Dockerfile.centos7-static-build`
     - `dev/docker/Dockerfile.centos7-gcc13-static-build`
     - `dev/docker/Dockerfile.centos8-static-build`
     - `dev/docker/Dockerfile.centos8-gcc13-static-build`
     - `dev/docker/Dockerfile.centos8-dynamic-build`
     - `dev/docker/Dockerfile.centos9-static-build`
     - `dev/docker/Dockerfile.centos9-dynamic-build`
     - `dev/docker/cudf/Dockerfile`
   - Remove `wget` package installation where it was only used for Maven 
download
   - Remove `PATH=/usr/lib/maven/bin` environment variables
   
   ## How was this patch tested?
   
   - Local Docker build tests on CentOS 8 and CentOS 9 Stream confirmed that 
`./build/mvn` correctly auto-downloads Maven 3.9.12 and executes successfully
   - Verified no remaining references to `setup-helper.sh` or `install_maven`
   - Code review confirmed all mvn → ./build/mvn migrations are correct
   
   ## Was this patch authored or co-authored using generative AI tooling?
   
   Generated-by: GitHub Copilot (Claude Opus 4.5)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to