-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[script] add install.sh for rt-thread #9355
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
Conversation
0052b18 to
fddad4d
Compare
|
是否放到tools目录下更合适? |
这个放根目录下比较明显一些, |
|
如果觉得不合适也可以放tools |
放tools下,感觉更适合些。例如后续把rt-thread导入到某IDE中,看到根目录下有份install.sh,是不是好奇就去执行下? |
已改 |
|
这个应该是安装env的吧,叫install不能一眼看出它的意图,叫env-install? |
|
其实我一直觉得 install_ubuntu.sh,install_windows.ps1等安装环境的脚本应该放在tools目录,而不是放在env仓库,因为随着rtt的升级,install_xxx肯定会变化,如果全部放在了env仓库,那么env修改,那么会影响所有版本,很难做到历史兼容。 env更应该做的是独立的工具链,管理rtt项目等,也就是具备独立rtt版本的工具。 |
env里面还有scons |
安装的脚本确实推荐放rt-thread这边,或者有份单独的也可以,比较少的,clone env,然后又去安装env |
列了这些,是否这些都需要补齐后再进行合并?如果是测试,也许在linux下用docker的模式测试会比较方便? |
其他都是根据env提供的脚本来安装的,env下面还没有,我先删了吧 |
|
@supperthomas 如果就绪请自行合并 |
* [script] add install.sh for rt-thread * [install] add gitee chevk
|
如果不进行修改,会引发报错
所以需要进行额外修改
# 根据检测结果决定是否使用--gitee参数
if [ "$use_gitee" = true ]; then
wget https://raw.githubusercontent.com/RT-Thread/env/master/install_macos.sh <---- 此处应为 https://gitee.com/RT-Thread-Mirror/env/raw/master/install_macos.sh
chmod 777 install_macos.sh
echo "Installing on China gitee..."
./install_macos.sh --gitee
else
wget https://raw.githubusercontent.com/RT-Thread/env/master/install_macos.sh
chmod 777 install_macos.sh
echo "Installing on no China..."
./install_macos.sh
fi
rm ./install_macos.sh@supperthomas @BernardXiong 还请大家尽快修复相关问题 |
欢迎提交一个PR,目前我们手上都没有macos的机器,对macos也不熟悉,测起来也不方便,如果有问题,请帮忙提个PR,谢谢。 |
#10593 已递交PR |




拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
添加一建安装
安装install.sh
你的解决方案是什么 (what is your solution)
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up