Skip to content

Conversation

@AndreMiras
Copy link
Member

Travis error was:

pyenv: pip3.7: command not found

Also bumps actions/setup-python to latest tag.
Hopefully fixes recent set-env and add-path commands depreciation.
The error is:

Error: Unable to process command
'##[set-env name=pythonLocation;]/opt/hostedtoolcache/Python/3.7.9/x64' successfully.
Error: The `set-env` command is disabled.
Please upgrade to using Environment Files or opt into unsecure command execution by
setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`.
For more information see:
https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

Travis error was:
```
pyenv: pip3.7: command not found
```
Also bumps `actions/setup-python` to latest tag.
Hopefully fixes recent `set-env` and `add-path` commands depreciation.
The error is:
```
Error: Unable to process command
'##[set-env name=pythonLocation;]/opt/hostedtoolcache/Python/3.7.9/x64' successfully.
Error: The `set-env` command is disabled.
Please upgrade to using Environment Files or opt into unsecure command execution by
setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`.
For more information see:
https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
```
Copy link
Member

@opacam opacam left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks!

Note: I see that we still have an error with our MacOS build with travis, but this is already an improvement 😄

@AndreMiras AndreMiras merged commit 73a4c52 into kivy:develop Dec 28, 2020
@AndreMiras AndreMiras deleted the feature/fix_travis branch December 28, 2020 15:03
@obfusk
Copy link
Contributor

obfusk commented Dec 28, 2020

Also bumps actions/setup-python to latest tag.
Hopefully fixes recent set-env and add-path commands depreciation.

Wasn't that already fixed by #2368? (not that it really matters)

@AndreMiras
Copy link
Member Author

Indeed, I got mislead by this PR #2380 that actually was targeting master branch hence was failing with this so I thought the issue was there.
Thanks for the heads up

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 this pull request may close these issues.

3 participants