Skip to content

fix(install): verify_install 优先校验实际安装路径 - #27

Open
ntxf31415 wants to merge 1 commit into
wps365-open:mainfrom
ntxf31415:fix/install-verify-install-dir
Open

fix(install): verify_install 优先校验实际安装路径#27
ntxf31415 wants to merge 1 commit into
wps365-open:mainfrom
ntxf31415:fix/install-verify-install-dir

Conversation

@ntxf31415

Copy link
Copy Markdown

摘要

install.sh 在自定义 WPS365_INSTALL_DIR 时,最终校验用 command -v wps365-cli 判断安装结果:如果 PATH 里已存在旧版本,会优先返回旧路径,导致「明明装到了自定义目录,最后却提示装到了旧路径」的误导(见 #8)。

变更

verify_install() 改为:

  1. 优先校验 ${INSTALL_DIR}/${BINARY_NAME} 是否存在且可执行 → 报告实际安装路径;
  2. 否则 fallback 到 command -v
  3. 都没有时维持原「不在 PATH」提示。

验证

  • 场景 A:PATH 无旧版本 + 自定义目录 → 提示显示自定义目录。
  • 场景 B:PATH 有旧版本 + 自定义目录 → 提示仍显示自定义目录(修复前会显示旧路径)。

Closes #8

自定义 WPS365_INSTALL_DIR 安装时,若 PATH 已存在旧版本二进制,
command -v 会返回旧路径,导致最终提示与实际安装目录不符(wps365-open#8)。
先校验 ${INSTALL_DIR}/${BINARY_NAME},再 fallback 到 command -v。
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.

install.sh 在自定义 WPS365_INSTALL_DIR 时最终校验路径可能报错

1 participant