[Bug] CI build script accepts flags as values for required options

Abierto
#293 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
74/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
cmake, shell
Área
build-system, cli

Línea de trabajo

Comienza con ci/scripts/build_paimon.sh y reproduce los comandos indicados para rastrear cómo se validan los valores de opción obligatorios y --build_type. Añade cobertura a nivel de script para valores ausentes o con apariencia de opción y tipos de compilación no compatibles; se considera terminado cuando los argumentos no válidos fallan antes de que se invoquen los directorios de compilación o CMake, mientras que los cuatro tipos de compilación de CMake indicados siguen siendo aceptados.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug
Search before asking
  • I searched in the issues and found nothing similar.
Paimon-cpp version
Paimon version

Current main branch.

Minimal reproduce step

Run one of the following commands:

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?
  • I'm willing to submit a PR!
Lenguaje dominante
C++
Estrellas
65
Forks
29
Merge medio
2 d 30 min
PR fusionados (30 d)
77

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de apache/paimon-cpp

Todos los issues de apache/paimon-cpp

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.