officialasishkumar opened a new pull request, #11903:
URL: https://github.com/apache/gluten/pull/11903
## What changes are proposed in this pull request?
Fixes #11885.
`dev/gluten-build-info.sh` was hardcoding the Velox checkout path when
writing build metadata, so builds done against a custom Velox source tree still
reported the default bundled path.
This change makes the script resolve the Velox checkout in the following
order:
- explicit `--backend_home` / `--velox_home` override
- exported `VELOX_HOME`
- `VELOX_HOME` recorded in `cpp/build/CMakeCache.txt`
- the previous default bundled path
The script now also parses arguments before writing the properties file so
backend-home overrides are order-independent.
## How was this patch tested?
- `bash -n dev/gluten-build-info.sh`
- local shell validation with a temporary git repo acting as a fake Velox
checkout, covering:
- explicit `--backend_home`
- `VELOX_HOME` environment fallback
- `cpp/build/CMakeCache.txt` fallback
## Was this patch authored or co-authored using generative AI tooling?
No
--
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]
