Skip to content

Conversation

magnmaeh
Copy link
Member

@magnmaeh magnmaeh commented Mar 6, 2024

This update goes hand in hand with:

lf-lang/lf-west-template#7

@lhstrh Do you want me to update the docs for previous versions of docs as well? I assume this update will apply for all versions of the docs as well?

Copy link
Collaborator

@erlingrj erlingrj left a comment

Choose a reason for hiding this comment

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

LGTM!

Regarding updating the earlier versions of the docs, I don't think it is that important. The docs already say that you should only have a single version of Zephyr installed, so technically they arent wrong.

@magnmaeh
Copy link
Member Author

magnmaeh commented Mar 6, 2024

Oh haha, so the entire reason it failed initially was because I didn't properly read the docs 😅

But I guess that might be a common user mistake having more than one installation of Zephyr - and sourcing the zephyr-env.sh remedies that mistake

@erlingrj
Copy link
Collaborator

erlingrj commented Mar 7, 2024

Oh haha, so the entire reason it failed initially was because I didn't properly read the docs 😅

But I guess that might be a common user mistake having more than one installation of Zephyr - and sourcing the zephyr-env.sh remedies that mistake

It wasn't exactly well documented and this is the right solution to that problem that will likely be common among potential users!

@erlingrj erlingrj merged commit 242e569 into lf-lang:main Mar 7, 2024
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.

2 participants