Skip to content

Check box version is the latest or not when run box self-update#36

Merged
huangzhhui merged 7 commits into
hyperf:masterfrom
huangdijia:check-box-version
Oct 4, 2022
Merged

Check box version is the latest or not when run box self-update#36
huangzhhui merged 7 commits into
hyperf:masterfrom
huangdijia:check-box-version

Conversation

@huangdijia
Copy link
Copy Markdown
Member

No description provided.

@huangdijia huangdijia requested a review from huangzhhui October 4, 2022 16:08
@huangzhhui
Copy link
Copy Markdown
Member

感觉统一处理版本比较这事会更好,单独做 box 的抽象性不够

@huangzhhui
Copy link
Copy Markdown
Member

而且还要考虑强制重新下载的方式,增加版本比对后如果下载的有损坏,暂时是没法重新下载的

@huangdijia
Copy link
Copy Markdown
Member Author

感觉统一处理版本比较这事会更好,单独做 box 的抽象性不够

我看并不是每个 pkg 都走 GitHub 获取 versions,在 pkgs.json 里也并不是都有 versions 信息,所以只是针对 box 做了处理。

@huangdijia
Copy link
Copy Markdown
Member Author

而且还要考虑强制重新下载的方式,增加版本比对后如果下载的有损坏,暂时是没法重新下载的

这层确实没考虑到

@huangzhhui
Copy link
Copy Markdown
Member

感觉统一处理版本比较这事会更好,单独做 box 的抽象性不够

我看并不是每个 pkg 都走 GitHub 获取 versions,在 pkgs.json 里也并不是都有 versions 信息,所以只是针对 box 做了处理。

pkgs.json 里面有一个 key 是 latest_fetch_type,计划就是用来明确 latest version 获取的方式和来源

@huangdijia
Copy link
Copy Markdown
Member Author

pkgs.json 里面有一个 key 是 latest_fetch_type,计划就是用来明确 latest version 获取的方式和来源

每个pkg的当前安装版本能获取到?像PHP、phpunit在pkgs.json里都是大版本

@huangzhhui
Copy link
Copy Markdown
Member

pkgs.json 里面有一个 key 是 latest_fetch_type,计划就是用来明确 latest version 获取的方式和来源

每个pkg的当前安装版本能获取到?像PHP、phpunit在pkgs.json里都是大版本

不是所有都能,目前也仅能解决 Github 获取的形式,所以有 latest 和 versions 这两个字段来辅助,这块就得靠人工维护了

@huangzhhui
Copy link
Copy Markdown
Member

pkgs.json 里面有一个 key 是 latest_fetch_type,计划就是用来明确 latest version 获取的方式和来源

每个pkg的当前安装版本能获取到?像PHP、phpunit在pkgs.json里都是大版本

不是所有都能,目前也仅能解决 Github 获取的形式,所以有 latest 和 versions 这两个字段来辅助,这块就得靠人工维护了

composer 有 https://getcomposer.org/versions 这个 API 可以获取到少量版本,接口我还没搞明白参数是啥,然后 phpunit 还没找到地方,但是 box 是通过 release 来明确版本的,所以可以把这个当做大多数吧

@huangzhhui huangzhhui merged commit 65ee4e3 into hyperf:master Oct 4, 2022
@huangdijia huangdijia deleted the check-box-version branch October 5, 2022 00:28
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.

2 participants