Hi Lukas,
On 09/19/2014 08:51 AM, Lukas Slebodnik wrote:
The mock config with name default is usually symbolic link
to the configuration file of local architecture. The side effect
of this patch is that we will not try to rebuild on old architectures
src.rpm for new architectures(fedora). It caused issues with mock tmpfs
plugin.
There is one other issue. This way we won't run RHEL6 mock builds locally
normally, as, AFAIK, people usually don't work on RHEL6.
Still, we would have our CI servers.
+ stage mock-build mock_privileged_deps
"default" \
+ --resultdir ci-mock-result \
rpmbuild/SRPMS/*.src.rpm
Would it be better to perhaps name the stage just "mock"? "mock-build"
seems
redundant.
Otherwise, the patch looks and works fine to me.
Thank you, Lukas.
There is still the problem of Clang running on all hosts. I think it will be
better to run it just on one. OTOH, developers might run CI on different
distros and Clang might produce different issues there. I see that on Fedora
Rawhide at least. Anyway, we can live with it for now and solve it later.
Nick