search

fishros / install

一键安装程序，欢迎大家提交代码和小鱼一起一键安装停止浪费生命
https://fishros.github.io/install/
1.6k stars 207 forks source link

[Warning] Wrong code comments #82

Open HRXWEB opened 2 months ago

HRXWEB commented 2 months ago

代码所示 https://github.com/fishros/install/blob/05b0bb0dc2a44502fda3dc9d8ad1b58940d529c3/tools/base.py#L1181-L1198

users列表总是会添加root用户。

没有细看更多逻辑，是不是应该加个选项选择需要修改哪些用户的shell rc

fishros commented 2 months ago

之所以给所有用户都加上，因为有时候ROS用户会使用sudo运行，但是环境变量没有ros导致失败就给加上了，可以考虑去掉root用户，加选项应该不用，大多数都是单用户+root，没必要为可能1%不到的用户让99%的用户多一步选择交互。

HRXWEB commented 2 months ago

emmm那就是1181行的注释问题，优先 两字就让人感觉有普通用户就不会添加root用户

elysia-best commented 2 months ago

emmm那就是1181行的注释问题，优先 两字就让人感觉有普通用户就不会添加root用户

的确，可以修改成合适的描述。