Skip to content

Commit

Permalink
Remove duplicate options and dependencies from respective variables
Browse files Browse the repository at this point in the history
Duplicate options e.g. "+FOO +FOO" was causing the wanted and remote
options comparison to fail, causing poudriere to build packages
unnecessarily.

Duplicate dependencies e.g. "foo/bar foo/bar" was causing the wanted
and remote deps comparison to fail, causing poudriere to skip packages.
  • Loading branch information
patmaddox authored and bapt committed Nov 7, 2023
1 parent 2fc920a commit b4c9458
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/share/poudriere/common.sh
Original file line number Diff line number Diff line change
Expand Up @@ -3676,7 +3676,7 @@ download_from_repo_check_pkg() {
$1 == pkgbase && $3 == "off" {print "-"$2;printed=1}
$1 != pkgbase && printed == 1 {exit}
' \
"${remote_all_options}" | sort -k1.2 | paste -s -d ' ' -)
"${remote_all_options}" | sort -k1.2 -u | paste -s -d ' ' -)

shash_get pkgname-options "${pkgname}" selected_options || \
selected_options=
Expand All @@ -3702,7 +3702,7 @@ download_from_repo_check_pkg() {
$1 == pkgbase {print $2;printed=1}
$1 != pkgbase && printed == 1 {exit}
' \
"${remote_all_deps}" | sort | paste -s -d ' ' -)
"${remote_all_deps}" | sort -u | paste -s -d ' ' -)
case "${local_deps}" in
${remote_deps}) ;;
*)
Expand Down

0 comments on commit b4c9458

Please sign in to comment.