Skip to content
This repository has been archived by the owner on Oct 30, 2019. It is now read-only.

Appdaemon: Use newest python version when installing. #157

Merged
merged 1 commit into from
Apr 6, 2018
Merged

Appdaemon: Use newest python version when installing. #157

merged 1 commit into from
Apr 6, 2018

Conversation

ludeeus
Copy link
Member

@ludeeus ludeeus commented Apr 5, 2018

Description:

This will use the newest version of python installed on the unit when creating venv.

Related issue (if applicable): Fixes #154

Checklist (Required):

If pertinent:

  • Script has validation check of the job.
  • Created/Updated documentation at /docs

@ludeeus ludeeus added this to the v0.9.0 milestone Apr 5, 2018
Copy link
Collaborator

@Landrash Landrash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe your overcomplicating this one. python3 will symlink to whatever version is installed ;)

pi@hassbian:~ $ ls -la /usr/bin/python3
lrwxrwxrwx 1 root root 9 Jan 20  2017 /usr/bin/python3 -> python3.5

@ludeeus
Copy link
Member Author

ludeeus commented Apr 5, 2018

That depends on how (if) you upgraded python.
I have 3.6 installed, and it still points to 3.5

pi@hassbian:~$ which python3.6
/usr/local/bin/python3.6
pi@hassbian:~$ ls -la /usr/bin/python3
lrwxrwxrwx 1 root root 9 Jan 20  2017 /usr/bin/python3 -> python3.5

Copy link
Collaborator

@Landrash Landrash left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like a odd workaround but well include it since for some reason building newer python versions are not updating symlinks properly.

@Landrash Landrash merged commit 7963b1b into home-assistant:dev Apr 6, 2018
@ludeeus ludeeus deleted the appdaemon-newewst-python branch April 6, 2018 21:53
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.

3 participants