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?


>
> 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.


> I have a fix in my tree for the UnboundLocalError, just didn't get around
> to committing it yet.
>
>
This is fine.


> -- Brane
>
>

-- 
Timofei Zhakov

Reply via email to