Skip to content

Conversation

@supperthomas
Copy link
Member

@supperthomas supperthomas commented Aug 27, 2024

拉取/合并请求描述:(PR description)

[

为什么提交这份PR (why to submit this PR)

添加一建安装
安装install.sh

你的解决方案是什么 (what is your solution)

请提供验证的bsp和config (provide the config and bsp)

  • BSP:
  • .config:
  • action:

]

当前拉取/合并请求的状态 Intent for your PR

必须选择一项 Choose one (Mandatory):

  • 本拉取/合并请求是一个草稿版本 This PR is for a code-review and is intended to get feedback
  • 本拉取/合并请求是一个成熟版本 This PR is mature, and ready to be integrated into the repo

代码质量 Code Quality:

我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:

  • 已经仔细查看过代码改动的对比 Already check the difference between PR and old code
  • 代码风格正确,包括缩进空格,命名及其他风格 Style guide is adhered to, including spacing, naming and other styles
  • 没有垃圾代码,代码尽量精简,不包含#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up
  • 所有变更均有原因及合理的,并且不会影响到其他软件组件代码或BSP All modifications are justified and not affect other components or BSP
  • 对难懂代码均提供对应的注释 I've commented appropriately where code is tricky
  • 代码是高质量的 Code in this PR is of high quality
  • 已经使用formatting 等源码格式化工具确保格式符合RT-Thread代码规范 This PR complies with RT-Thread code specification

@supperthomas supperthomas force-pushed the install branch 2 times, most recently from 0052b18 to fddad4d Compare August 27, 2024 00:44
@BernardXiong
Copy link
Member

是否放到tools目录下更合适?

@supperthomas
Copy link
Member Author

是否放到tools目录下更合适?

这个放根目录下比较明显一些,
很多仓库都有类似的比如esp-idf
惯例一些吧

@supperthomas
Copy link
Member Author

如果觉得不合适也可以放tools

@BernardXiong
Copy link
Member

如果觉得不合适也可以放tools

放tools下,感觉更适合些。例如后续把rt-thread导入到某IDE中,看到根目录下有份install.sh,是不是好奇就去执行下?

@github-actions github-actions bot added action github action yml imporve tools labels Aug 27, 2024
@supperthomas
Copy link
Member Author

如果觉得不合适也可以放tools

放tools下,感觉更适合些。例如后续把rt-thread导入到某IDE中,看到根目录下有份install.sh,是不是好奇就去执行下?

已改

@heyuanjie87
Copy link
Contributor

这个应该是安装env的吧,叫install不能一眼看出它的意图,叫env-install?

@ComerLater
Copy link
Contributor

其实我一直觉得 install_ubuntu.sh,install_windows.ps1等安装环境的脚本应该放在tools目录,而不是放在env仓库,因为随着rtt的升级,install_xxx肯定会变化,如果全部放在了env仓库,那么env修改,那么会影响所有版本,很难做到历史兼容。

env更应该做的是独立的工具链,管理rtt项目等,也就是具备独立rtt版本的工具。

@supperthomas
Copy link
Member Author

这个应该是安装env的吧,叫install不能一眼看出它的意图,叫env-install?

env里面还有scons
install 算所有环境的总和吧。算把所有环境一次性配好。主要方便新手

@BernardXiong
Copy link
Member

其实我一直觉得 install_ubuntu.sh,install_windows.ps1等安装环境的脚本应该放在tools目录,而不是放在env仓库,因为随着rtt的升级,install_xxx肯定会变化,如果全部放在了env仓库,那么env修改,那么会影响所有版本,很难做到历史兼容。

env更应该做的是独立的工具链,管理rtt项目等,也就是具备独立rtt版本的工具。

安装的脚本确实推荐放rt-thread这边,或者有份单独的也可以,比较少的,clone env,然后又去安装env

@BernardXiong
Copy link
Member

install_on_fedora() {
install_on_centos() {
install_on_arch() {

列了这些,是否这些都需要补齐后再进行合并?如果是测试,也许在linux下用docker的模式测试会比较方便?

@supperthomas
Copy link
Member Author

install_on_fedora() {
install_on_centos() {
install_on_arch() {

列了这些,是否这些都需要补齐后再进行合并?如果是测试,也许在linux下用docker的模式测试会比较方便?

其他都是根据env提供的脚本来安装的,env下面还没有,我先删了吧

@mysterywolf
Copy link
Member

@supperthomas 如果就绪请自行合并

@supperthomas supperthomas merged commit 8b4fb52 into RT-Thread:master Aug 28, 2024
@supperthomas supperthomas deleted the install branch September 12, 2024 08:12
rcitach pushed a commit to rcitach/rt-thread that referenced this pull request Nov 1, 2024
* [script] add install.sh for rt-thread

* [install] add gitee chevk
@ThearchyHelios
Copy link
Contributor

install.sh中有几个比较关键的问题,我罗列一下:

  1. macOS在 uname中的结果是 Darwin, 不是 macOS
image

如果不进行修改,会引发报错

image

所以需要进行额外修改

  1. 文件中存在了大量的针对大陆用户 gitee 链接的错误引用
    ex:
# 根据检测结果决定是否使用--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 还请大家尽快修复相关问题

@supperthomas
Copy link
Member Author

install.sh中有几个比较关键的问题,我罗列一下:

  1. macOS在 uname中的结果是 Darwin, 不是 macOS
image 如果不进行修改,会引发报错 image 所以需要进行额外修改
  1. 文件中存在了大量的针对大陆用户 gitee 链接的错误引用
    ex:
# 根据检测结果决定是否使用--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,谢谢。

@ThearchyHelios
Copy link
Contributor

install.sh中有几个比较关键的问题,我罗列一下:

  1. macOS在 uname中的结果是 Darwin, 不是 macOS
image 如果不进行修改,会引发报错 image 所以需要进行额外修改
  1. 文件中存在了大量的针对大陆用户 gitee 链接的错误引用
    ex:
# 根据检测结果决定是否使用--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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action github action yml imporve tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants