Introduce support for upgrading dependencies; Change mbedtls_url to the Github source; fix SC2162; fix SC2181#118
Introduce support for upgrading dependencies; Change mbedtls_url to the Github source; fix SC2162; fix SC2181#118IceCodeNew wants to merge 8 commits into
mbedtls_url to the Github source; fix SC2162; fix SC2181#118Conversation
我特意在整个 repo 范围内搜寻了一下 `--no-check-certificate` 或者其他可能的关键词,没有看到有人提 issue 说 wget 检查证书导致了错误,所以才加入这个参数来兼容。 那么我的意见是没有必要在一开始就加入这个参数,这降低了安全性,很多源代码都用到了 `wget` 拉下来,有必要通过验证证书确保拉下来的东西是未经篡改过的。 另外以我自己为例,我环境上 `wget` 还是一个 alias: `wget2 --progress=bar --secure-protocol=PFS --https-enforce=soft`,所以我打算一点一点地把脚本中的二进制改为通过绝对路径调用,避免 alias 和 脚本中参数冲突导致问题的可能性。
0. fix SC2068 1. fix SC2196 2. use `[[ … ]]` rather than `[ … ]` (refer: https://google.github.io/styleguide/shellguide.html#s6.3-tests)
Upgrading dependencies while reinstalling(which is actually means upgrading) shadowsocks.
0. Change `mbedtls_url` to the Github source 1. fix SC2181 2. use `[[ … ]]` rather than `[ … ]` (refer: https://google.github.io/styleguide/shellguide.html#s6.3-tests) 3. Minor fix
mbedtls_url to the Github source; fix SC2162; fix SC2181
|
已经实机跑了一遍确认没有问题了。我觉得下一个目标就是 CI test 了( |
|
@teddysun 可以的话希望能给些意见,这样我好跟着做对应的修改。 |
|
#117 已经关闭。 |
|
另: |
CI test 是说笑的,wget 那个修改我可以 drop 掉,但我觉得你 miss 了这个 PR 的主要目的——让脚本安装的 shadowsocks 可以更新,要做到这一点不只是简单重新运行一遍脚本就行了,依赖项也非常有必要升级。 此外对于最近的一个 commit 我也不太理解你的用意,我在那个 commit #1e25c70 下面做了评论,希望能得到你的解答。 |
|
|
所以综上,这个 PR 里一些试图规范化部分 shell 代码的部分我们可以再议,但是核心的对依赖项更新的支持功能还没有得到讨论,所以我认为这个 PR 不应该关闭。 |
就这点来说,欢迎提交 PR :) |
我看要不然您还是把这个 PR 重新打开,我把分支变基一下,只保留与这个功能相关的功能。这样您就可以 merge 了。 对 wget 所作的一切改进已经丢弃掉了。包括由 |
|
请重新提交 PR 吧,这个已经无法 reopen 了 |
请对新的 PR #119 提出修改意见,或合并。 |



Introduce support for upgrading dependencies
Upgrading dependencies while reinstalling(which is actually means upgrading) shadowsocks.
fix SC2162
Always use
-rwithreadunless you have a good reason not to.Be aware that
readwithout-rwill mangle backslashes!这个 PR 我的考虑是要么等 PR #117 被通过以后,我变基重新 force-push 一遍这个分支;要么是直接请 @teddysun 关闭 PR #117 ,直接合并这个 PR。
之所以这么做是考虑到我新增的这个功能改动有点大,万一不能一次性过呢?所以把一些无关紧要的修改分到了另一个分支
4pr_icn上面去,但是为了避免合并 PR 时遇到冲突,所以这个分支是在4pr_icn的基础上再添加 commits 的,这样如果最后变成两个 PR 都分开合并的情况,我 git am 还有 git rebase 时方便一些。