Kiril Nesenko has posted comments on this change.

Change subject: Use static repos for a new layout
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.ovirt.org/#/c/25295/4/jobs/packaging/repo_closure_check.sh
File jobs/packaging/repo_closure_check.sh:

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} \
> I see, then I'll just put it on it's own line, and maybe reorder the option
Yes, but if ${STATIC_RP} is not defined will it work ?
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

Reply via email to