Sandro Bonazzola has posted comments on this change. Change subject: jobs: ovirt-hosted-engine-setup ......................................................................
Patch Set 8: (4 comments) http://gerrit.ovirt.org/#/c/30510/8/jobs/confs/shell-scripts/build_mock_srcrpm.sh File jobs/confs/shell-scripts/build_mock_srcrpm.sh: Line 6: # Name of the project it runs on, specifically the dir where the code Line 7: # has been cloned Line 8: # Line 9: # distro Line 10: # Distribution it should cre%ate the repms for (usually el6, el7, fc19 or > Spurious % there. Done Line 11: # fc20) Line 12: # Line 13: # arch Line 14: # Architecture to build the packages for Line 20: # extra-build-options Line 21: # space separated list of extra options to pass to the build.sh script Line 22: # Line 23: # extra-rpmbuild-options Line 24: # extra options to pass to rpmbuild as defines, as a spaceseparated list > Ironically missing a space in spaceseparated. Is that intentional? :) Done Line 25: # of key=value pairs Line 26: # Line 27: # extra-repos Line 28: # List of extra repositories to use when building, in a space separated list Line 67: --base="$distribution-$arch.cfg" \ Line 68: --option="basedir=$WORKSPACE/mock/" \ Line 69: $mock_repos \ Line 70: > "$mock_conf.cfg" Line 71: sudo touch /var/cache/mock/*/root_cache/cache.tar.gz || : > What does this? Doesn't this reuse the cache indefinitely since it'll alway Not sure, I took it from an existing script in this directory. Line 72: cat "$mock_conf.cfg" Line 73: popd Line 74: Line 75: ## prepare the command line Line 149: "${{rpmbuild_options[@]}}" \ Line 150: -ts *.gz Line 151: Line 152: Line 153: mkdir /tmp/SRCRPMS > Please use mktemp for this. this is executed inside the chroot, how do you pass it back to the calling script? Line 154: mv *.tar.gz /tmp/SRCRPMS/ Line 155: mv *src.rpm /tmp/SRCRPMS/ Line 156: EOF Line 157: -- To view, visit http://gerrit.ovirt.org/30510 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I10a908a2be067d38616c41a2c2e6cb3effc9c787 Gerrit-PatchSet: 8 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbona...@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: Ohad Basan <oba...@redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
