Ewoud Kohl van Wijngaarden has posted comments on this change.

Change subject: Add generic code to check repoclosure
......................................................................


Patch Set 12: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/24802/12/jobs/packaging/repo_closure_check.sh
File jobs/packaging/repo_closure_check.sh:

Line 102:                       -l check-jpackage-rhel5 \
Line 103:                       -l check-jpackage-generic \
Line 104:                       -r check-custom
Line 105:       elif [ "${DISTRIBUTION}" = "fc" ] \
Line 106:               [ "${DISTRIBUTION}" = "Fedora" ]; then
Missing || here
Line 107:               custom_url="${BASE_URL}/${REPO_NAME}/rpm/${repo}"
Line 108:               repoclosure \
Line 109:                       -t \
Line 110:                       --repofrompath=check-custom,"${custom_url}" \


-- 
To view, visit http://gerrit.ovirt.org/24802
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I762abf97c9020301a388f7d83c7d1e41bdf1f062
Gerrit-PatchSet: 12
Gerrit-Project: jenkins
Gerrit-Branch: master
Gerrit-Owner: Kiril Nesenko <knese...@redhat.com>
Gerrit-Reviewer: David Caro <dcaro...@redhat.com>
Gerrit-Reviewer: Ewoud Kohl van Wijngaarden <ew...@kohlvanwijngaarden.nl>
Gerrit-Reviewer: Eyal Edri <ee...@redhat.com>
Gerrit-Reviewer: Kiril Nesenko <knese...@redhat.com>
Gerrit-Reviewer: Ohad Basan <oba...@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

Reply via email to