-
Notifications
You must be signed in to change notification settings - Fork 45
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
Linux/MacOS support #185
Linux/MacOS support #185
Conversation
感谢提交PR, @Guozhanxin 郭老师请review一下 |
wsl2测试通过,windows下测试通过 |
def get_rtt_root(bsp_root): | ||
rtt_root = os.getenv("RTT_ROOT") | ||
if rtt_root is None: | ||
rtt_root = os.path.join(bsp_root, '..', '..') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这块可能不一定是向上反两级是rt-thread根目录,有些BSP有可能是三级。靠这种方法可能不可靠。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这块可以问问熊大或者郭老师有没有什么更好的方法
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
RTT_ROOT 在什么情况下会缺失?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我在macos跑一直是缺失的
先合并 有问题再提😄 |
menuconfig在Linux/MacOS可以使用了