[fix](build) Fix build and test scripts failing due to missing executable permission#63330
Open
heguanhui wants to merge 1 commit into
Open
[fix](build) Fix build and test scripts failing due to missing executable permission#63330heguanhui wants to merge 1 commit into
heguanhui wants to merge 1 commit into
Conversation
…able permission ### What problem does this PR solve? Issue Number: close apache#63225 Problem Summary: Several shell scripts are invoked directly (e.g., ./build.sh) without executable permission, causing "Permission denied" errors during build, FE UT, BE UT, and broker build processes. This PR changes all such invocations to use `bash` prefix to ensure they execute correctly regardless of file permission settings. ### Release note Fix build and test scripts failing due to missing executable permission by using `bash` prefix for shell script invocations. ### Check List (For Author) - Test: Manual test - Successfully built Doris with `bash build.sh --be --fe` - Successfully ran FE unit tests with `bash run-fe-ut.sh` - Successfully built broker module - Behavior changed: No - Does this need documentation: No
Contributor
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
Contributor
Author
Manual Test ReportTest type: Manual test Test report attachment: 问题及修复.docx The test report documents the build errors encountered and the fixes applied. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What problem does this PR solve?
Issue Number: close #63225
Problem Summary: Several shell scripts are invoked directly (e.g.,
./build.sh) without executable permission, causing "Permission denied" errors during build, FE UT, BE UT, and broker build processes. This PR changes all such invocations to usebashprefix to ensure they execute correctly regardless of file permission settings.Release note
Fix build and test scripts failing due to missing executable permission by using
bashprefix for shell script invocations.Check List (For Author)
Test
bash build.sh --be --febash run-fe-ut.shBehavior changed:
Does this need documentation?