Re: [PATCH v2 1/2] tests: libvirt-ci: Update to commit '51dad419'

[Paolo Bonzini]

On 4/17/23 14:46, Peter Krempa wrote:
> Update to commit which has fixes needed for OpenSUSE 15.4 and
> re-generate output files.
>
> Signed-off-by: Peter Krempa<pkrempa@redhat.com>
> ---
>   .gitlab-ci.d/cirrus/freebsd-12.vars           |  16 ++
>   .gitlab-ci.d/cirrus/freebsd-13.vars           |  16 ++
>   .gitlab-ci.d/cirrus/macos-12.vars             |  16 ++
>   tests/docker/dockerfiles/alpine.docker        | 127 +++++++++++++

Oops, I missed that '51dad419' is buggy and the next commit (2fa24dce8b) is
needed instead:

    commit 2fa24dce8bc2d0d57b02b111dbdbd24e25d4ae95
    Author: Erik Skultety <eskultet@redhat.com>
    Date:   Wed Apr 5 15:43:55 2023 +0200

    lcitool: __main__: Prevent running main() on import
    
    When commit f0cc11e3f introduced the module, it did it so that:
    1) it complies with PEP 517 [1]
    2) lcitool could be executed as a Python module
    
    The implementation is based on having a __main__.py module which Python
    can automatically import. The problem is that when the main function is
    imported from the module from within the automatic executable wrapper
    created when the package is installed, it executes the whole module on
    import (standard behaviour). At the end of the module there's an
    unguarded call to the main() function.
    Fix this with the '__name__ == "__main__"' idiom.
    
    Fixes: f0cc11e3f
    
    [1] https://peps.python.org/pep-0517/
    
    Signed-off-by: Erik Skultety <eskultet@redhat.com>

I adjusted the submodule and rerun tests/lcitool/refresh.

Paolo