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

Update install-release.sh #1705

Merged
merged 2 commits into from
May 28, 2019
Merged

Update install-release.sh #1705

merged 2 commits into from
May 28, 2019

Conversation

sunshineplan
Copy link
Contributor

提示已安装新版本应该显示本地版本而不是网上获取的新版本号。否则会出现本地版本为4.18.0,却提示已经安装了最新版本4.18.2

@kslr
Copy link
Contributor

kslr commented May 28, 2019

你好 @sunshineplan

这个方案实际上没有解决这个问题,并且会对其他用户造成干扰。

在下一个版本将会同步到4.19.1

@sunshineplan
Copy link
Contributor Author

@kslr 可能是我没表达清楚。我现在本地版本4.18.0,如果运行脚本,会显示:

Latest version v4.18.2 is already installed.

这是个错误的提示,因为按照原本的设计,大版本号不变是不更新的,所以并没有升级到4.18.2,但是却显示成4.18.2已安装。

本次修改后会正常显示:

Latest version v4.18.0 is already installed.

@kslr
Copy link
Contributor

kslr commented May 28, 2019

但是Latest version是4.18.0 就不太合理了

所以这还是一个因为使用内部版本判断,导致超出的问题

@sunshineplan
Copy link
Contributor Author

毕竟当初版本号第3位是测试使用,需要手动打入完整版本号进行安装,检测版本升级只是用前2位做比对。
以前的测试版本存在时间也比较短,推荐普通用户安装的只有.0大版本号。不过,现在软件很稳定了,升级版本速度已经慢了许多,升级策略的确可以重新考虑。但是这里的改动是正确的, 因为这里是需要显示本地的版本号。出现这个问题的原因应该是以前new ver和cur ver都是截取前2位更改了变量值后做比较,所以2个值是一样的,用哪个都无所谓,但是后来改过一版不更改变量值而是比对时进行临时截取前2位,所以那句提示不能再用原来的new ver而是使用cur ver了。

@kslr
Copy link
Contributor

kslr commented May 28, 2019

我们应该该升级脚本,支持小版本号的升级和检测。
你可以来做这个工作吗

@sunshineplan
Copy link
Contributor Author

代码我是改完了,只要去掉只判断前2位限制即可。但是我发现下载来的4.18.2程序,运行v2ray -version得到的结果是4.19,这个版本号以后还是需要加以规范的。

@kslr kslr merged commit 46ba7cd into v2ray:master May 28, 2019
@kslr
Copy link
Contributor

kslr commented Jun 12, 2019

hi @sunshineplan
如果你有兴趣可以加到开发者讨论群中 https://t.me/joinchat/JiDjBFceHgtzk2gxXRnyRg

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

Successfully merging this pull request may close these issues.

None yet

2 participants