scripts/verify-pack: exits 0 when no pack directory is given
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 25/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- bash
- Domaine
- cli
Piste de recherche
Read scripts/verify-pack around lines 401–405 and the usage() function at line 48. Check how the no-argument path handles output and exit status; done when it reports the error on stderr and exits nonzero. Pull request #141 is already open for this issue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
if [[ ${#TARGETS[@]} -eq 0 ]]; then
echo "Error: No pack directory specified."
echo ""
usage
fi
usage() ends with exit 0 (L48).
Problem: bash scripts/verify-pack with no argument prints an error but returns exit code 0. A script that calls verify-pack "$PACK" with an empty $PACK thinks that the check passed. The error also goes to stdout, not stderr.
Fix: At line 404, use usage >&2; exit 1 (or give usage() an exit-code argument).
Found by a code read at commit 1e50295. The code was read, but not run on a real instance.
- Langage dominant
- Shell
- Étoiles
- 73
- Forks
- 11
- Merge moyen
- 6 h 46 min
- PR mergées (30 j)
- 5
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de inceptionstack/lowkey
-
bug good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
inceptionstack/lowkey#135 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
inceptionstack/lowkey#134 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 94/100
inceptionstack/lowkey#133 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
inceptionstack/lowkey#131 ·
Les mainteneurs répondent en général sous 1 jour
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
inceptionstack/lowkey#130 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de inceptionstack/lowkey
Issues similaires
-
Sourced-library load-guards use a top-level `return` that errors on direct execution (repo-wide)Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
rossoctl/automation#115 ·
Les mainteneurs répondent en général sous 3 jours
-
loongarch64: `unsupported linker arg: --no-rosegment` with rustc 1.100Peut-être pris @monosans l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
rust-cross/cargo-zigbuild#503 ·
Les mainteneurs répondent en général sous 1 jour
-
netbox status: backlog topic: testing type: housekeeping
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
netbox-community/netbox#23395 ·
Les mainteneurs répondent en général sous 1 jour
-
test/command failure with filtered `listxattr` syscallPeut-être pris @Kiskae l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Difficulté 2/5 Moins d'une heure Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 1 jour