David Caro has posted comments on this change. Change subject: Use static repos for a new layout ......................................................................
Patch Set 4: (2 comments) http://gerrit.ovirt.org/#/c/25295/4/jobs/packaging/repo_closure_check.sh File jobs/packaging/repo_closure_check.sh: Line 21: CENTOS_MIRROR="http://centos.mirror.constant.com/"; Line 22: EPEL_MIRROR="http://linux.mirrors.es.net/"; Line 23: FEDORA_MIRROR="http://mirrors.kernel.org/"; Line 24: GLUSTER_MIRROR="http://download.gluster.org/"; Line 25: JPACKAGE_MIRROR="ftp://jpackage.hmdc.harvard.edu/"; > its inside the script, why do I need it ? To avoid having it overwritten by environment variables (unless that is what you want) Line 26: Line 27: die() { Line 28: local m="${1}" Line 29: echo "FATAL: ${m}" Line 114: if [ "${DISTRIBUTION}" = "el" ] \ Line 115: || [ "${DISTRIBUTION}" = "Centos" ]; then Line 116: repoclosure \ Line 117: -t \ Line 118: --repofrompath=check-custom,"${CUSTOM_URL}" ${STATIC_RP} \ > exists in line 109 I see, then I'll just put it on it's own line, and maybe reorder the options to be grouped by repo repoclosure \ -t \ --repofrompath=check-custom,"${CUSTOM_URL}" \ -l check-custom \ ${STATIC_RP} \ --repofrompath=check-base, ... \ -l check-base \ ... Line 119: --repofrompath=check-base,"${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/os/x86_64/ \ Line 120: --repofrompath=check-base-i386,"${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/os/i386/ \ Line 121: --repofrompath=check-updates,"${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/updates/x86_64/ \ Line 122: --repofrompath=check-extras,"${CENTOS_MIRROR}/${DISTRIBUTION_VERSION}"/extras/x86_64/ \ -- To view, visit http://gerrit.ovirt.org/25295 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I44087b1d6f5aafa965ad8c0b9b23e8f8d4b949b2 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Kiril Nesenko <knese...@redhat.com> Gerrit-Reviewer: David Caro <dcaro...@redhat.com> Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
