install-poetry.py fails to install poetry from tarball (using --path)

Open
#58 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
48/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Stale
Tech stack
python
Domain
tooling

Research direction

Start with install-poetry.py and reproduce the reported --path case in the python:3.7-slim Docker image using the Poetry 1.1.7 tarball. Compare its behavior with --version and the deprecated get-poetry.py --file path, then verify that the resulting virtual environment contains the poetry executable.

Written by the indexing model from the issue text.

Description

  • I am on the latest Poetry version.

  • I have searched the issues of this repo and believe that this is not a duplicate.

  • If an exception occurs when executing a command, I executed it again in debug mode (-vvv option).

  • OS version and name: Docker image python:3.7-slim

  • Poetry version: N/A

  • Link of a Gist with the contents of your pyproject.toml file: N/A

Issue

To reproduce the environment exactly, everything below should be run within this docker container:

docker run --rm -it python:3.7-slim bash

On top of that, the following needs to be run within the docker container, before the 3 cases described below:

POETRY_VERSION=1.1.7
curl -LO https://github.com/python-poetry/poetry/releases/download/$POETRY_VERSION/poetry-$POETRY_VERSION-linux.tar.gz

Now, this works (old get-poetry.py install script, passing --file):

curl -s https://raw.githubusercontent.com/python-poetry/poetry/master/get-poetry.py | \
  python - --file poetry-$POETRY_VERSION-linux.tar.gz

And so does this (new install-poetry.py install script, passing --version):

curl -s https://raw.githubusercontent.com/python-poetry/poetry/master/install-poetry.py | \
  python - --version $POETRY_VERSION

But this (new install script, passing --path) claims to successfully install poetry, but actually installs an empty venv, without poetry:

curl -s https://raw.githubusercontent.com/python-poetry/poetry/master/install-poetry.py | \
  python - --path poetry-$POETRY_VERSION-linux.tar.gz

I make use of --path so that I can cache the tarball between docker builds & CI executions, so I'll have to stay on the deprecated get-poetry.py script for now 🙃

Dominant language
Python
Stars
255
Forks
65
Avg merge
2h 48m
Merged PRs (30d)
1

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from python-poetry/install.python-poetry.org

All issues in python-poetry/install.python-poetry.org

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.