github-actions[bot] commented on code in PR #63330:
URL: https://github.com/apache/doris/pull/63330#discussion_r3339036358
##########
build.sh:
##########
@@ -1142,7 +1142,7 @@ fi
if [[ "${BUILD_BE_CDC_CLIENT}" -eq 1 ]]; then
install -d "${DORIS_OUTPUT}/be/lib/cdc_client"
cd "${DORIS_HOME}/fs_brokers/cdc_client"
- ./build.sh
+ bash ./build.sh
rm -rf "${DORIS_OUTPUT}/be/lib/cdc_client"/*
Review Comment:
This still leaves `build.sh --be` exposed to the same permission failure.
This changed call now runs `fs_brokers/cdc_client/build.sh` via `bash`, but
that script immediately invokes `"${DORIS_HOME}"/generated-source.sh noclean`
directly. In a checkout where executable bits are missing, the full/BE build
reaches this CDC client step and fails with `Permission denied` on
`generated-source.sh`, even though the top-level `build.sh` and `run-fe-ut.sh`
paths were fixed. Please update the nested CDC client invocation to use `bash
"${DORIS_HOME}"/generated-source.sh noclean` as well.
--
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]
