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

"version 子命令新增工具链和 quic-go 版本信息" --- 可以把这个信息放在别的命令里面,而不是改动 version 命令的输出格式 #1271

Open
cary-sas opened this issue Dec 11, 2024 · 7 comments · May be fixed by #1272

Comments

@cary-sas
Copy link

cary-sas commented Dec 11, 2024

你的功能请求是否与某个问题有关?
由于我程序需要提取 version 命令的输出, 但现在新版本里面,这个命令的输出由于多了其他的内容,导致格式变了,我的程序受到了影响,而且还不大好调整。

描述你希望的解决方案
希望可以把 libVersion: 信息放在别的命令里面 (类似于 多一个命令 “libVersion”),而不是改动 version 命令的输出格式。

或者可以教我如何在源码里面自己去修改这个命令的输出,然后再自己编译二进制。

@cary-sas cary-sas added the enhancement New feature or request label Dec 11, 2024
@haruue
Copy link
Collaborator

haruue commented Dec 11, 2024

您好,

您可以参照这个提交来 parse hysteria version 命令的输出: d4a1c2b

如果您希望自己编译 hysteria-core 以改变 hysteria version 的输出的话, 您可以更改下面的代码:

hysteria/app/cmd/root.go

Lines 41 to 49 in 15e31d4

appVersionLong = fmt.Sprintf("Version:\t%s\n"+
"BuildDate:\t%s\n"+
"BuildType:\t%s\n"+
"Toolchain:\t%s\n"+
"CommitHash:\t%s\n"+
"Platform:\t%s\n"+
"Architecture:\t%s\n"+
"LibVersion:\t%s",
appVersion, appDate, appType, appToolchain, appCommit, appPlatform, appArch, libVersion)

@haruue haruue removed the enhancement New feature or request label Dec 11, 2024
@cary-sas
Copy link
Author

最好还是上游能够 把 libVersion: 信息放在别的命令里面啊, 我并不是不会改code 来提取我需要的信息,而是因为别的原因无法改动code 来做调整

@haruue
Copy link
Collaborator

haruue commented Dec 11, 2024

您前面说是您自己的程序, 但是又无法改 code 来调整。 可否提供更多细节以便我们为您提供帮助?

@Smallthing
Copy link

您前面说是您自己的程序, 但是又无法改 code 来调整。 可否提供更多细节以便我们为您提供帮助?

估计卖出去给机场了。别管他

@cary-sas
Copy link
Author

cary-sas commented Dec 11, 2024

您前面说是您自己的程序, 但是又无法改 code 来调整。 可否提供更多细节以便我们为您提供帮助?

我是做的这个项目, 如果改程序的话,使用者必须要先更新我的插件版本,才能再更新 hysteria2 的二进制, 但是实际情况是,可能用户在不知情的情况下直接更新了 hysteria2 ,然后就会导致 一些程序出错,连web界面都无法正确显示,很麻烦的

@cary-sas
Copy link
Author

我在想能否这样,如果你们真的没法保持原来的输出格式。
我fork 你们项目,然后在哪里修改一下 workflow 里面的代码,定期跑workflow action,得到修改过的 hysteria 二进制build。
但这个办法看起来比较麻烦

haruue added a commit that referenced this issue Dec 11, 2024
close: #1271

A key that also contains "Version" broke the version parsing of some
third-party clients.
@haruue haruue linked a pull request Dec 11, 2024 that will close this issue
@cary-sas
Copy link
Author

Thank you for your solution, that's indeed can solve my problem. Looking forward!

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 a pull request may close this issue.

3 participants