Skip to content
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

关于脚本使用的疑惑 #2

Open
jiuyueshenhua opened this issue Dec 5, 2024 · 5 comments
Open

关于脚本使用的疑惑 #2

jiuyueshenhua opened this issue Dec 5, 2024 · 5 comments

Comments

@jiuyueshenhua
Copy link

有一点我不太清楚。
我遇见的所有仓库的脚本,它们的使用方法都是类似./shxx aa bb cc 。
为什么你要求用户先加载这个脚本的符号,再调用这个函数呢?我不太明白这与前者相比有什么好处,还多了需多敲一步命令和污染用户符号环境的问题。

get_avaliable_board我认为通过读取prebuild文件目录更新更好。否则贡献新板子prebuild的人必须先理解脚本。而且这样做,现在你shell显示的可支持板子和prebuild下的目录是不对应的。

@koikky
Copy link
Owner

koikky commented Dec 5, 2024

你说的是哪个函数

@koikky
Copy link
Owner

koikky commented Dec 5, 2024

还需要改,有什么问题可以提一下 :D

@jiuyueshenhua
Copy link
Author

你说的是哪个函数

image
这里我觉得改为通过读取prebuild下的子目录来得知所有可支持的板子会更好点。后续支持其他板子的人不需动脚本,只需提交工具链。
我上周写过类似功能的辅助函数。,或许你能用到。

我觉得通过传递参数来执行用户指定的功能好一点,可以一次敲完。其他仓库也是这样的。让用户加载文件符号,再调用他自己关心的函数。我之前是没见过有人这样做。更何况后面还要支持更新prebuild。这个可以是用户去更新,也可以是维护者更新。指定板子的功能也没有。这如果全让用户去执行函数的话,要敲命令的次数会很多。

我认为脚本就三个部分,参数解析,打包镜像,配置环境。参数解析和支持新板子的工具链都是需要考虑未来拓展的。

help函数必须改名!因为有些shell的默认文档说明命令就是help,比如zsh,fish。
image

安装后执行命令应该是mkimage,而不是./mkimage。反正我这是只能执行前者。
image
image

@koikky
Copy link
Owner

koikky commented Dec 6, 2024

ok,提议可以,我试着再看看

@koikky
Copy link
Owner

koikky commented Dec 6, 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

No branches or pull requests

2 participants