On 26. 5. 25 21:12, Timofei Zhakov wrote:
On Mon, May 26, 2025 at 8:54 PM Branko Čibej <br...@apache.org> wrote:

    On 26. 5. 25 15:40, Timofei Zhakov wrote:
    Would this be helpful?

    (see the patch in the attachments)


    This just throws out all the dependency setup and special-cases
    schema verification, it doesn't actually solve the underlying
    problem. You'll notice from the logs that creating and using the
    virtual environment is *not* the reason for the test failures.


Yes, but this is the purpose of the patch. Since I want to avoid downloading dependencies, I'm suggesting adding an option and asking users to install it.

If I am not mistaken, it takes a single `pip install` invocation to setup all dependencies, so what are the reasons to download them at run-time?


It takes more than a single pip install to set up a virtual environment so that you don't pollute your system. Like I said, enabling offline testing is not my priority right now, there are other issues with the XML output, and with the semantics of schema validation, that are more important.

I wouldn't mind at all if you could be a little more patient and not expect everything to be fixed right this second.



    The first issue is that the CMake build does not not set
    PYTHONPATH the way run_tests.py does, but that's not causing the
    failures, either.


I think it does. According to the documentation <https://cmake.org/cmake/help/latest/module/FindPython.html>, you just need to set |Python_EXECUTABLE| option to the path to the correct program.

That's not the case at all and not what we're doing. You do not have to use a different python in order to use packages installed in a virtual environment. Setting PYTHONPATH is enough.

Anyway, I'll have to see what's happening in the CMake build, it's refreshing the venv far too often.

-- Brane

Reply via email to