Pieter De Gendt

Results 105 comments of Pieter De Gendt

Version 6.24.0 is on pypi, but now 6.24.1 isn't 😄

You didn't import anything from the project, but you could add the `west-commands` entry manually ```yaml manifest: remotes: - name: zephyrproject-rtos url-base: https://github.com/zephyrproject-rtos projects: - name: zephyr remote: zephyrproject-rtos revision:...

Maybe the bug is that if you do something like ```yaml manifest: remotes: - name: zephyrproject-rtos url-base: https://github.com/zephyrproject-rtos projects: - name: zephyr remote: zephyrproject-rtos revision: v4.0.0 import: name-allowlist: - cmsis...

> I think the only reliable and portable way to implement this feature would be: > > * Use `pip --dry-run` or similar to determine whether `west packages pip --install`...

> > This does complicate things a lot, > > Yes of course. I'm not opposed to quick fixes in general but only as long as there's a very clear...

> > But this behavior is how pip works, I'm not sure we can accurately predict this, maybe different versions would work differently. > > If we can't predict what...

> > We can blame designs and have discussions all we want, but it's a fix nonetheless. I want to incrementally improve things, until the next bug pops up we...

> Trying to summarize my current perspective. > > * os.exec() looks good, just add the words "this could downgrade or upgrade west" in the GSG (see [west packages pip...

> > If this option on Windows is to be removed, it should be deprecated first. > > So, should `--install` be deprecated on Windows? Or is it fixable somehow?...

When trying on Windows 11 using Powershell, I get a warning, but it does succeed: ```powershell (.venv) PS C:\Users\pdgendt\zephyrproject> west packages pip --install Installing collected packages: windows-curses, unidiff, t61codec, sortedcontainers,...