fix(setup-aur): fix variable expansion order in build_or_skip function
Fixed a bash variable expansion bug where $name was being referenced before assignment in a multi-variable local declaration. Changed d assignment to use $1 directly instead of $name to ensure the correct directory path is set. This prevents PKGBUILD lookup failures when processing dependent packages in AUR chains.
This commit is contained in:
+4
-1
@@ -236,7 +236,10 @@ publish_to_repo() {
|
|||||||
# Installe TOUJOURS le résultat (depuis local-repo si sauté) pour que les
|
# Installe TOUJOURS le résultat (depuis local-repo si sauté) pour que les
|
||||||
# paquets dépendants suivants résolvent leurs deps (chaînes AUR→AUR, anaconda…).
|
# paquets dépendants suivants résolvent leurs deps (chaînes AUR→AUR, anaconda…).
|
||||||
build_or_skip() {
|
build_or_skip() {
|
||||||
local name="$1" d="$BUILD_DIR/$name" ver
|
# NB : référencer "$1" et PAS "$name" ici — dans un `local a=… b=…` à une
|
||||||
|
# seule ligne, bash développe la 2ᵉ valeur AVANT que `name` ne soit affecté,
|
||||||
|
# donc "$name" y serait vide → d="/tmp/fws-aur-build/" (PKGBUILD introuvable).
|
||||||
|
local name="$1" d="$BUILD_DIR/$1" ver
|
||||||
[ -f "$d/PKGBUILD" ] || error "PKGBUILD introuvable pour $name ($d)"
|
[ -f "$d/PKGBUILD" ] || error "PKGBUILD introuvable pour $name ($d)"
|
||||||
ver="$(pkgbuild_fullver "$d")"
|
ver="$(pkgbuild_fullver "$d")"
|
||||||
if repo_has "$name" "$ver"; then
|
if repo_has "$name" "$ver"; then
|
||||||
|
|||||||
Reference in New Issue
Block a user