[I] [Bug] CI build script accepts flags as values for required options [paimon-cpp]

["slfan1989 (via GitHub)" <gi...@apache.org>]

slfan1989 opened a new issue, #293:
URL: https://github.com/apache/paimon-cpp/issues/293

   ### Search before asking
   
   - [x] I searched in the [issues](https://github.com/apache/paimon-cpp/issues) and found nothing similar.
   
   
   ### Paimon-cpp version
   
   ### Paimon version
   
   Current `main` branch.
   
   ### Minimal reproduce step
   
   Run one of the following commands:
   
   ```bash
   ci/scripts/build_paimon.sh --source_dir --enable_asan
   
   ci/scripts/build_paimon.sh \
     --source_dir . \
     --build_type --enable_asan
   
   ci/scripts/build_paimon.sh \
     --source_dir . \
     --lint_git_target_commit --enable_asan
   ```
   
   The script treats the following option as the value of the preceding option and continues until a later command fails or CMake is invoked with an invalid value.
   
   The script also accepts unsupported build types:
   
   ```
   ci/scripts/build_paimon.sh \
     --source_dir . \
     --build_type Profile
   ```
   
   ### What doesn't meet your expectations?
   
   Options that require a value should reject an empty value or another `--option` with a clear error message.
   
   The `--build_type` option should only accept the standard CMake build types used by the project:
   - `Debug`
   - `Release`
   - `RelWithDebInfo`
   - `MinSizeRel`
   Invalid arguments should be rejected before creating build directories or invoking CMake.
   
   The argument validation behavior is currently not covered by dedicated script-level tests.
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [x] I'm willing to submit a PR!


-- 
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: issues-unsubscribe@paimon.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] [Bug] CI build script accepts flags as values for required options [paimon-cpp]

["SteNicholas (via GitHub)" <gi...@apache.org>]

SteNicholas closed issue #293: [Bug] CI build script accepts flags as values for required options
URL: https://github.com/apache/paimon-cpp/issues/293


-- 
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: issues-unsubscribe@paimon.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org