Skip to content

preserve relative-path at poetry run - #7963

Merged
radoering merged 1 commit into
python-poetry:masterfrom
dimbleby:fix-poetry-run
May 21, 2023
Merged

radoering merged 1 commit into
python-poetry:masterfrom
dimbleby:fix-poetry-run

Conversation

@dimbleby

@dimbleby dimbleby commented May 19, 2023 •

Copy link
Copy Markdown
Contributor

fixes #7959

a regression introduced in the effort to use Path for paths. Turns out that Env._bin() isn't really returning a path at all, it's returning a thing that you pass to subprocess.Popen()

In particular, Path("./foo.py") is the same as Path("foo.py")

@radoering radoering added impact/backport Requires backport to stable branch backport/1.5 labels May 21, 2023
Comment thread src/poetry/utils/env.py Resolved
@radoering
radoering merged commit ebadf8a into python-poetry:master May 21, 2023
poetry-bot Bot pushed a commit that referenced this pull request May 21, 2023
radoering pushed a commit that referenced this pull request May 21, 2023
@dimbleby
dimbleby deleted the fix-poetry-run branch May 21, 2023 15:51
@djch

djch commented May 22, 2023

Copy link
Copy Markdown

Nice to see this was addressed already 🙂

I'm hoping this one can go out in a bug fix release ASAP. I've got a lot of scripts in a lot of Django projects which call ./manage.py.

In the mean time, can I run poetry self update *something* to install poetry from master and get this fix?

@radoering

Copy link
Copy Markdown
Member

In the mean time, can I run poetry self update *something* to install poetry from master and get this fix?

Not sure about poetry self but if you use pipx, you can install the 1.5 branch (from which 1.5.1 will be built) via

pipx install git+https://github.com/python-poetry/poetry.git@1.5

@github-actions

github-actions Bot commented Mar 3, 2024

Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

impact/backport Requires backport to stable branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Poetry run no longer executes local bash scripts

3 participants