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.

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

Remove support for using ninja flash/debug on hardware without installing west with pip #53497

Open
carlescufi opened this issue Jan 4, 2023 · 2 comments
Assignees
Labels
area: Build System area: West West utility Enhancement Changes/Updates/Additions to existing features

Comments

@carlescufi
Copy link
Member

carlescufi commented Jan 4, 2023

As described in the documentation, today it is still supported to be able to execute the flash and debug build targets when west is not installed in the system with pip, via the use of the WEST_DIR optional variable. This added complexity is not deemed worthwhile, and so we should remove the support for it.

@carlescufi carlescufi added the Enhancement Changes/Updates/Additions to existing features label Jan 4, 2023
@carlescufi carlescufi changed the title Remove support for using ninja flash/debug on hardware without installing west Remove support for using ninja flash/debug on hardware without installing west with pip Jan 4, 2023
@marc-hb
Copy link
Collaborator

marc-hb commented Jan 4, 2023

I understand the desire to remove WEST_DIR and simplify but why does pip matter? As long as west can be found in the PATH then it's enough and OK, correct? I hope it does not matter where west came from.

@zephyrbot
Copy link
Collaborator

Hi @tejlmand,

This issue, marked as an Enhancement, was opened a while ago and did not get any traction. Please confirm the issue is correctly assigned and re-assign it otherwise.

Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.

@carlescufi you are also encouraged to help moving this issue forward by providing additional information and confirming this request/issue is still relevant to you.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System area: West West utility Enhancement Changes/Updates/Additions to existing features
Projects
Status: To do
Development

No branches or pull requests

4 participants