tools: fix fbq-add-source.sh exiting after first source with set -e
(( n++ )) returns exit code 1 when the pre-increment value is 0, which kills the script under set -euo pipefail. Replace with n=$(( n + 1 )) which always returns 0. https://claude.ai/code/session_014CCYqVwW6d6f5dw1qRokYt
This commit is contained in:
@@ -228,7 +228,7 @@ cmd_remove() {
|
|||||||
local name
|
local name
|
||||||
name=$(yq e ".server.sources[] | select(.path == \"$p\") | .name // \"(unnamed)\"" "$config" 2>/dev/null || echo "(unnamed)")
|
name=$(yq e ".server.sources[] | select(.path == \"$p\") | .name // \"(unnamed)\"" "$config" 2>/dev/null || echo "(unnamed)")
|
||||||
printf " %2d) %-20s %s\n" "$i" "${name:-?}" "$p"
|
printf " %2d) %-20s %s\n" "$i" "${name:-?}" "$p"
|
||||||
(( i++ ))
|
i=$(( i + 1 ))
|
||||||
done
|
done
|
||||||
echo ""
|
echo ""
|
||||||
|
|
||||||
@@ -268,7 +268,7 @@ cmd_show() {
|
|||||||
ro=$(yq e ".server.sources[] | select(.path == \"$p\") | .config.readOnly // false" "$config" 2>/dev/null || echo "false")
|
ro=$(yq e ".server.sources[] | select(.path == \"$p\") | .config.readOnly // false" "$config" 2>/dev/null || echo "false")
|
||||||
printf " %-20s %-35s defaultEnabled=%-5s readOnly=%s\n" \
|
printf " %-20s %-35s defaultEnabled=%-5s readOnly=%s\n" \
|
||||||
"${name}" "${p}" "${enabled}" "${ro}"
|
"${name}" "${p}" "${enabled}" "${ro}"
|
||||||
(( count++ ))
|
count=$(( count + 1 ))
|
||||||
done < <(list_sources "$config")
|
done < <(list_sources "$config")
|
||||||
|
|
||||||
[[ "$count" -eq 0 ]] && warn "No sources found."
|
[[ "$count" -eq 0 ]] && warn "No sources found."
|
||||||
|
|||||||
Reference in New Issue
Block a user