Re: [kvm-unit-tests PATCH] Switch the order of the parameters in report() and report_xfail()

Re: [kvm-unit-tests PATCH] Switch the order of the parameters in report() and report_xfail()

[Thomas Huth]

On 17/10/2019 15.30, Andrew Jones wrote:
> 
> Hi Thomas and Paolo,
> 
> This will conflict badly with "[kvm-unit-tests PATCH v2 0/6] arm: Use
> stable test output lines" and likely with pretty much anything else
> in flight - since pretty much everything adds report() lines. I'm not
> sure why "[kvm-unit-tests PATCH v2 0/6] arm: Use stable test output
> lines" hasn't been merged yet. Paolo, do you want me to do PULL
> requests for all the arm-related patches? If so, I can do one tomorrow
> for what's currently reviewed and waiting. I'd prefer they all get
> merged before this patch. We should also confirm all other architectures
> have their reviewed and waiting patches merged first too, IMHO.

I'm not aware of any pending ppc or s390x patches. Anyway, could we
please revert the "lib: use an argument which doesn't require default
argument promotion" for now, since it breaks many tests? Or take Bill's
fix for the problem (we still can rework it later, I guess).

 Thanks,
  Thomas

Re: [kvm-unit-tests PATCH] Switch the order of the parameters in report() and report_xfail()

[Paolo Bonzini]

On 17/10/19 15:30, Andrew Jones wrote:
> Paolo, do you want me to do PULL
> requests for all the arm-related patches?

Yes, that's why it's not merged. :)

This patch is mostly automatically generated, so Thomas can send me v2
after your pull request is in, and I'll apply it.

Paolo