Sandro Bonazzola has posted comments on this change. Change subject: reorg repoclosure script for ease of maintenance ......................................................................
Patch Set 6: (1 comment) https://gerrit.ovirt.org/#/c/39249/6/jobs/packaging/repo_closure_check.sh File jobs/packaging/repo_closure_check.sh: Line 407: Line 408: case ${DISTRIBUTION} in Line 409: el|Centos) Line 410: # TODO: extract these into files? Line 411: repos+=( Looks like += works only on bash 3. it doesn't work with bash 4.2.53 ( fedora 20) Take a look at http://stackoverflow.com/questions/1951506/bash-add-value-to-array-without-specifying-a-key for alternatives Line 412: "check-custom-${distid},baseurl,${CUSTOM_URL}" Line 413: "check-base-${distid},baseurl,${CENTOS_MIRROR}/${distro_ver}/os/x86_64/" Line 414: "check-updates-${distid},baseurl,${CENTOS_MIRROR}/${distro_ver}/updates/x86_64/" Line 415: "check-extras-${distid},baseurl,${CENTOS_MIRROR}/${distro_ver}/extras/x86_64/" -- To view, visit https://gerrit.ovirt.org/39249 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I207c5c59ca3149c17897c231e391e15e7eec4363 Gerrit-PatchSet: 6 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Max Kovgan <[email protected]> Gerrit-Reviewer: David Caro <[email protected]> Gerrit-Reviewer: Max Kovgan <[email protected]> Gerrit-Reviewer: Sandro Bonazzola <[email protected]> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
