On 15/10/21 14:38, Philippe Mathieu-Daudé wrote:
On 10/15/21 12:07, Paolo Bonzini wrote:
Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
---
  .gitlab-ci.d/crossbuild-template.yml | 2 +-
  configure                            | 3 ---
  2 files changed, 1 insertion(+), 4 deletions(-)

Maybe squash in patch #2 or place as #3 mentioning
"the previous commit"?

Anywhere you place it it's wrong. :) Squashing in #2 I dislike because of the functional change in .gilab-ci.d (in truth, the variable is *almost* dead!). Having the change as #4 makes it survive one patch longer than it should, on the other hand having it as #3 separates similar changes to "check-block.sh".

What I will actually do in the final submission is not include the TAP patch and submit it separately. Then for this one I can indeed add that it was used only in the implementation of check-block.

Paolo


Reply via email to