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.

Sign up for GitHub

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

Jump to bottom

scripts/requirements.txt needs updating for west #10241

Closed
galak opened this issue Sep 26, 2018 · 0 comments
Closed

scripts/requirements.txt needs updating for west #10241

galak opened this issue Sep 26, 2018 · 0 comments
Assignees
@mbolivar
Labels
bug The issue is a bug, or the PR is fixing a bug

Comments

@galak
Copy link
Collaborator

galak commented Sep 26, 2018

Tried using west that was in Zephyr, and ran into:

Traceback (most recent call last):
  File "/home/galak/git/zephyr/scripts/meta/west/main.py", line 12, in <module>
    import colorama
ModuleNotFoundError: No module named 'colorama'
@galak galak added the bug The issue is a bug, or the PR is fixing a bug label Sep 26, 2018
mbolivar pushed a commit to mbolivar/zephyr that referenced this issue Sep 26, 2018
scripts: add colorama to requirements.txt
7d59e35 
This got missed in the latest west code import.

Fixes zephyrproject-rtos#10241.

Signed-off-by: Marti Bolivar <marti@foundries.io>
@mbolivar mbolivar mentioned this issue Sep 26, 2018
Merged
nashif pushed a commit that referenced this issue Sep 27, 2018
@nashif
scripts: add colorama to requirements.txt
ddad53a 
This got missed in the latest west code import.

Fixes #10241.

Signed-off-by: Marti Bolivar <marti@foundries.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mbolivar mbolivar

Labels
bug The issue is a bug, or the PR is fixing a bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mbolivar @galak