Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

undeclared dependency on six package #2842

Closed
kapilt opened this issue May 19, 2021 · 5 comments · Fixed by #2863
Closed

undeclared dependency on six package #2842

kapilt opened this issue May 19, 2021 · 5 comments · Fixed by #2863

Comments

@kapilt
Copy link

kapilt commented May 19, 2021

installing docker python in a fresh py3, and attempt to use, fails due to undeclared dependency on the python six package (auth.py)

@cliffwoolley
Copy link

@ulyssessouza - This seems related to your recent change #2788 ?

@cliffwoolley
Copy link

There seem to still be quite a few files in the 5.0.0 package that import six.

docker/transport/npipeconn.py
docker/transport/unixconn.py
docker/transport/npipesocket.py
docker/transport/sshconn.py
docker/api/plugin.py
docker/api/secret.py
docker/api/image.py
docker/api/exec_api.py
docker/api/container.py
docker/api/swarm.py
docker/api/config.py
docker/credentials/store.py
docker/models/images.py
docker/types/healthcheck.py
docker/types/services.py
docker/types/base.py
docker/types/containers.py
docker/auth.py
docker/utils/build.py
docker/utils/json_stream.py
docker/utils/socket.py

@cliffwoolley
Copy link

Looks like this duplicates #2807 .

NitzKo95 added a commit to NitzKo95/docker-py that referenced this issue May 20, 2021
Signed-off-by: Nitzan Kohan <nitzan@cognata.com>
- fixes docker#2807
- fixes docker#2842
@MartinWallgren
Copy link

six was removed from setup.py in this commit

@geerlingguy
Copy link

Yeah... a lot of my automation has been failing since a couple weeks ago when this new release was made. Finally found the source of it :) (ref: geerlingguy/drupal-container#33)

geerlingguy added a commit to geerlingguy/fathom-container that referenced this issue Jun 3, 2021
geerlingguy added a commit to geerlingguy/php-apache-container that referenced this issue Jun 3, 2021
Pliner added a commit to aio-libs/aiopg that referenced this issue Jun 23, 2021
Pliner added a commit to aio-libs/aiopg that referenced this issue Jun 23, 2021
JonZeolla added a commit to SeisoLLC/easy_infra that referenced this issue Jul 7, 2021
jesse-r-s-hines added a commit to jesse-r-s-hines/ShellAdventure that referenced this issue Jul 11, 2021
See docker/docker-py#2842
We can undo this after the next release of the "docker" package, which
fixes the issue.
jesse-r-s-hines added a commit to jesse-r-s-hines/ShellAdventure that referenced this issue Jul 12, 2021
See docker/docker-py#2842
We can undo this after the next release of the "docker" package, which
fixes the issue.
knonm added a commit to knonm/kotlin-notebook that referenced this issue Aug 19, 2021
qtc-de added a commit to qtc-de/tricot that referenced this issue Sep 2, 2021
docker-py dropped it's python2 compatibility and six should be no longer
required (see docker/docker-py#2842).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants