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.

None yet

3 participants