Skip to content

Uninstall enum34 in python3.6 docker image#8103

Merged
balloob merged 2 commits into
home-assistant:devfrom
jeanregisser:dockerfile-remove-enum34
Jun 24, 2017
Merged

Uninstall enum34 in python3.6 docker image#8103
balloob merged 2 commits into
home-assistant:devfrom
jeanregisser:dockerfile-remove-enum34

Conversation

@jeanregisser
Copy link
Copy Markdown
Contributor

@jeanregisser jeanregisser commented Jun 19, 2017

This is a short term fix for #7733

What's happening is the following dependencies are pulling enum34:

  • pygatt
  • libsoundtouch
  • yeelight

However, enum34 is not meant to be installed in Python versions 3.4+
and causing the AttributeError: module 'enum' has no attribute 'IntFlag'

I've submitted patches to these projects so we don't have to do this
manual uninstall in the future.

Let me know what you think:

This is a short term fix for home-assistant#7733

What's happening is the following dependencies are pulling enum34:
- pygatt
- libsoundtouch
- yeelight
However, enum34 is not meant to be installed in Python versions 3.4+
and causing the `AttributeError: module 'enum' has no attribute 'IntFlag'``

I've submitted patches to these projects so we don't have to do this
manual uninstall in the future.
@mention-bot
Copy link
Copy Markdown

@jeanregisser, thanks for your PR! By analyzing the history of the files in this pull request, we identified @balloob, @michaelarnauts and @mathewpeterson to be potential reviewers.

@homeassistant
Copy link
Copy Markdown
Contributor

Hi @jeanregisser,

It seems you haven't yet signed a CLA. Please do so here.

Once you do that we will be able to review and accept this pull request.

Thanks!

@balloob balloob added this to the 0.47.2 milestone Jun 24, 2017
@balloob
Copy link
Copy Markdown
Member

balloob commented Jun 24, 2017

Thanks for this fix! If we make a 0.47.2 I'll make sure to include this.

@balloob
Copy link
Copy Markdown
Member

balloob commented Jun 24, 2017

Also thanks for trying to get the broken dependencies fixed!

@balloob balloob merged commit 45f6f44 into home-assistant:dev Jun 24, 2017
balloob added a commit that referenced this pull request Jun 24, 2017
balloob added a commit that referenced this pull request Jun 24, 2017
* Revert "Uninstall enum34 in python3.6 docker image (#8103)"

This reverts commit 45f6f44.

* Revert "Add libboost-python1.62-dev (fixes #7851) (#7868)"

This reverts commit f1290d3.
@balloob balloob mentioned this pull request Jul 1, 2017
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
* Uninstall enum34 in python3.6 docker image

This is a short term fix for home-assistant#7733

What's happening is the following dependencies are pulling enum34:
- pygatt
- libsoundtouch
- yeelight
However, enum34 is not meant to be installed in Python versions 3.4+
and causing the `AttributeError: module 'enum' has no attribute 'IntFlag'``

I've submitted patches to these projects so we don't have to do this
manual uninstall in the future.

* Update Dockerfile
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
…e-assistant#8182)

* Revert "Uninstall enum34 in python3.6 docker image (home-assistant#8103)"

This reverts commit 45f6f44.

* Revert "Add libboost-python1.62-dev (fixes home-assistant#7851) (home-assistant#7868)"

This reverts commit f1290d3.
@home-assistant home-assistant locked and limited conversation to collaborators Oct 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants