Fix artifacts uploads in Windows workflows#1030
Merged
BYVoid merged 2 commits intoBYVoid:masterfrom Jan 24, 2026
Merged
Conversation
## 修改内容 ### MSVC Workflow - 添加文件列表调试步骤，显示构建和安装目录的内容 - 扩展 artifacts 上传路径，包含: - build/Release/*.exe (多配置生成器产生的可执行文件) - build/Release/*.dll (运行时库) - build/src/tools/Release/*.exe (工具可执行文件) ### MinGW Workflow - 添加 `cmake --install build` 步骤，将产物安装到 install 目录 - 修改 CMAKE_INSTALL_PREFIX 从 /usr 改为 $PWD/install - 添加文件列表调试步骤 - 更新 artifacts 上传路径为: - install/** (安装后的所有文件，包含 bin/, lib/, share/ 等) - build/src/tools/*.exe (备用，直接从构建目录上传可执行文件) - 测试相关文件 (Testing/**, ctest.log) ## 问题分析 原问题：opencc.exe、opencc_dict.exe 等关键产物未被上传 ### MSVC - Windows 多配置生成器将产物放在 build/Release/ 而不是根目录 bin/ - 虽然 build.cmd 执行了 install，但需要验证实际安装路径 ### MinGW - 缺少 install 步骤，产物留在 build/src/tools/ 深层目录 - 上传 build/** 包含大量不必要的中间文件 ## 验证方式 下次 workflow 运行时，查看 "list installed files" 步骤的输出， 确认 opencc.exe 等文件的实际位置，如需进一步调整可根据日志修正路径。 https://claude.ai/code/session_DH6D6
## 修改内容 ### MSVC Workflow 根据安装日志，文件实际安装在 build/ 子目录下： - `build/bin/` - 可执行文件和 DLL - `build/lib/` - 静态库和 CMake 配置 - `build/share/opencc/` - 所有 .ocd2 字典和 .json 配置文件 - `build/include/` - 头文件 更新 artifacts 上传路径： - 从根目录 `bin/**`, `lib/**`, `share/**`, `include/**` - 改为 `build/bin/**`, `build/lib/**`, `build/share/**`, `build/include/**` - 移除冗余的 `build/Release/*.exe` 等备用路径 更新调试步骤： - 列出 build/bin, build/lib, build/share, build/include 的内容 ### MinGW Workflow 已通过 `cmake --install` 安装到 install/ 目录，上传 `install/**` 包含所有必需文件。 更新调试步骤： - 单独列出 install/bin/ 和 install/share/opencc/ 以便快速验证 - 保留完整文件列表供详细检查 ## 重要说明 **share/opencc/** 目录包含运行时必需的文件： - *.ocd2 - 字典文件（STCharacters, STPhrases, TWVariants 等） - *.json - 配置文件（s2t.json, t2s.json, s2tw.json 等） 没有这些文件，opencc.exe 无法正常工作。 https://claude.ai/code/session_DH6D6
This was referenced Jan 24, 2026
|
miss MSVCP140D.dll,VCRUNTIME140D.dll,VCRUNTIME140_1D.dll,ucrtbased.dll |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Example outputs from Windows-related workflows: