[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 08/24] python: Add pipenv support
From: |
Cleber Rosa |
Subject: |
Re: [PATCH v4 08/24] python: Add pipenv support |
Date: |
Tue, 16 Feb 2021 22:02:46 -0500 |
On Tue, Feb 16, 2021 at 09:59:47PM -0500, Cleber Rosa wrote:
> On Thu, Feb 11, 2021 at 01:58:40PM -0500, John Snow wrote:
> > pipenv is a tool used for managing virtual environments with pinned,
> > explicit dependencies. It is used for precisely recreating python
> > virtual environments.
> >
> > pipenv uses two files to do this:
> >
> > (1) Pipfile, which is similar in purpose and scope to what setup.py
> > lists. It specifies the requisite minimum to get a functional
> > environment for using this package.
> >
> > (2) Pipfile.lock, which is similar in purpose to `pip freeze >
> > requirements.txt`. It specifies a canonical virtual environment used for
> > deployment or testing. This ensures that all users have repeatable
> > results.
> >
> > The primary benefit of using this tool is to ensure repeatable CI
> > results with a known set of packages. Although I endeavor to support as
> > many versions as I can, the fluid nature of the Python toolchain often
> > means tailoring code for fairly specific versions.
> >
> > Note that pipenv is *not* required to install or use this module; this is
> > purely for the sake of repeatable testing by CI or developers.
> >
> > Here, a "blank" pipfile is added with no dependencies, but specifies
> > Python 3.6 for the virtual environment.
> >
> > Pipfile will specify our version minimums, while Pipfile.lock specifies
> > an exact loudout of packages that were known to operate correctly. This
>
> Layout? Loadout?
>
> > latter file provides the real value for easy setup of container images
> > and CI environments.
> >
> > Signed-off-by: John Snow <jsnow@redhat.com>
> > ---
> > python/Pipfile | 11 +++++++++++
> > 1 file changed, 11 insertions(+)
> > create mode 100644 python/Pipfile
> >
>
> Other than that,
>
> Reviewed-by: Cleber Rosa <crosa@redhat.com>
Actually, just one suggestion: bump the position of this patch twice.
It makes it easier to understand its purpose if it is placed right
before the "python: add pylint to pipenv" patch.
Cheers,
- Cleber.
signature.asc
Description: PGP signature
- [PATCH v4 01/24] python/console_socket: avoid one-letter variable, (continued)
- [PATCH v4 01/24] python/console_socket: avoid one-letter variable, John Snow, 2021/02/11
- [PATCH v4 03/24] python: create qemu packages, John Snow, 2021/02/11
- [PATCH v4 05/24] python: add qemu package installer, John Snow, 2021/02/11
- [PATCH v4 08/24] python: Add pipenv support, John Snow, 2021/02/11
- Re: [PATCH v4 08/24] python: Add pipenv support, John Snow, 2021/02/16
[PATCH v4 06/24] python: add VERSION file, John Snow, 2021/02/11
[PATCH v4 17/24] python/qemu: add isort to pipenv, John Snow, 2021/02/11
[PATCH v4 09/24] python: add pylint import exceptions, John Snow, 2021/02/11