raulcd opened a new pull request, #2742:
URL: https://github.com/apache/iceberg-python/pull/2742

    Closes #2741
   
   # Rationale for this change
   
   `make install` is currently trying to install `uv` even when is present 
locally:
   
   ```bash
   $ make install
   uv not found. Installing...
   /home/raulcd/.local/bin/uv
   ^Cmake: *** [Makefile:62: install-uv] Interrupt
   
   $ uv --version
   uv 0.9.7
   ```
   
   ## Are these changes tested?
   
   No, only validated locally on Debian 14 that it does not try to install:
   ```bash
   $ PYTHON=3.12 make install
   /home/raulcd/.local/bin/uv
   uv is already installed.
   uv venv --python 3.12
   Using CPython 3.12.12
   Creating virtual environment at: .venv
   ```
   
   And validated in a clean docker container that if `uv` is not present it 
installs it:
   ```bash
   root@9e4870fda91e:/app/iceberg-python# make install
   uv not found. Installing...
   downloading uv 0.9.8 x86_64-unknown-linux-gnu
   no checksums to verify
   installing to /root/.local/bin
   ```
   
   ## Are there any user-facing changes?
   
   No
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to