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

fix: silent installation script typo #1393

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Yh793
Copy link

@Yh793 Yh793 commented Oct 15, 2024

+2: 跳过当前指令和下一条指令

fixes: 8e6b477

+2: 跳过当前指令和下一条指令

fixes: rime@8e6b477
@fxliang
Copy link
Contributor

fxliang commented Oct 15, 2024

这个是要什么功能,fixes一个pr?

@Yh793
Copy link
Author

Yh793 commented Oct 15, 2024

修复原PR要实现的功能,当前代码

IfSilent +2 ;静默模式这行会跳过
  ExecWait "$INSTDIR\WeaselDeployer.exe /install" ;这行会跳过
  Goto deploy_done ;这里跳转到deploy_done

ExecWait "$INSTDIR\WeaselDeployer.exe /deploy" ;这里无论是否静默都不会执行
deploy_done:

全默認配置/S測試
before: 安裝完成需要重新部署才能打中文
after: 安裝完成直接可以輸出中文

@Yh793 Yh793 closed this Oct 18, 2024
@fxliang
Copy link
Contributor

fxliang commented Oct 18, 2024

@Yh793 Yh793 reopened this Oct 21, 2024
@Yh793
Copy link
Author

Yh793 commented Oct 21, 2024


不确定原PR老哥的设计是不是想静默模式什么都不做的,所以先关闭了。
如果什么都不做的话就有冗余代码。
然后如果是升级安装的话自动重新部署会不会改变原有设置。

IfSilent +3
  ExecWait "$INSTDIR\WeaselDeployer.exe /install" 
  Goto deploy_done

ExecWait "$INSTDIR\WeaselDeployer.exe /deploy"
deploy_done:

@fxliang
Copy link
Contributor

fxliang commented Oct 21, 2024

@mirtlecn 还看看测试下?

@Yh793
Copy link
Author

Yh793 commented Oct 21, 2024

@mirtlecn 还看看测试下?

测试
原有配置文件都在默认用户目录无影响
静默和非静默都对原配置无影响,重新部署会根据原用户目录的user.yaml部署

发现使用静默模式安装的话,托盘右键 用户目录 是无反应的。

@Yh793
Copy link
Author

Yh793 commented Oct 21, 2024

重新安装:无影响
静默安装:无影响
全新安装:符合提示语 可以使【小狼毫】写字了:) 安裝完成直接可以輸出中文

@mirtlecn
Copy link
Contributor

我不太记得了当时是怎么想的,好像是 winget 没法安装才有的这个 pr。那边测试没问题都无所谓

@mirtlecn
Copy link
Contributor

mirtlecn commented Oct 21, 2024

要求是,/S 这个参数给了后,不弹出任何需要确认的弹窗,无人值守可以进行下一个软件安装。这个满足就行

P.S. 这个我好像记得当时测试后,按键盘之后,会自动部署,是能正常用的。是不是有个 pr 把在 server 中部署改掉后,就出现这个 pr 讲的没任何反应了。(好像记错了,当时没在意这个,总之能静默安装就行)

@Yh793
Copy link
Author

Yh793 commented Oct 21, 2024

要求是,/S 这个参数给了后,不弹出任何需要确认的弹窗,无人值守可以进行下一个软件安装。这个满足就行

P.S. 这个我好像记得当时测试后,按键盘之后,会自动部署,是能正常用的。是不是有个 pr 把在 server 中部署改掉后,就出现这个 pr 讲的没任何反应了。(好像记错了,当时没在意这个,总之能静默安装就行)

静默安装没问题,这样改动符合主流输入法安装逻辑。安装完成直接可以打字,不用另外再去点一下部署。

@fxliang
Copy link
Contributor

fxliang commented Oct 22, 2024

LGTM,你们再测试下,没有大问题就考虑合了吧

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.

3 participants