This patch makes tests not depend on specific platforms where tests run.
This patch also fixes prep, install, compile, local, verify-files where not modify rpmdefines by adding _builddir.
Signed-off-by: Chenxiong Qi cqi@redhat.com
I agree changing self.rpmdefines here seems wrong. However, this changes the order. Since the rpmdefines contain _builddir, it will override what is appended here. I think it breaks the --builddir cli option.
self.rpmdefines
_builddir
--builddir
Typo here: warining. Given that the command is not actually executed, does it make sense to mangle the specfile? I think just testing the --info option would be enough.
warining
--info
rebased
@lsedlar Fixed. Thanks for pointing out these.
Looks good to me. :thumbsup:
Looks good to me.
I'll rebase this patch, then merge it.
Pull-Request has been merged by cqi
This patch makes tests not depend on specific platforms where tests
run.
This patch also fixes prep, install, compile, local, verify-files where
not modify rpmdefines by adding _builddir.
Signed-off-by: Chenxiong Qi cqi@redhat.com