Skip to content

开始打包 #158

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.

Sign up for GitHub

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

Merged
WSXYT merged 2 commits into from
Jan 11, 2026
Merged

开始打包 #158

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c0b6f7e
开始打包
WSXYT Jan 11, 2026
2e20979
Merge remote-tracking branch 'origin/master'
WSXYT Jan 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 0 additions & 25 deletions .github/workflows/build-unified.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -178,31 +178,6 @@ jobs:
Compress-Archive -Path zip_dist/SecRandom/* -DestinationPath $outputZip -Force
echo "目录模式打包完成: $outputZip"

Copy link

Copilot AI Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removal of the Inno Setup packaging step eliminates the ability to generate Windows installer executables (.exe files). After this change, Windows builds will only produce ZIP archives. This is a breaking change that affects the release artifacts available to end users. Consider whether this removal is intentional, and if so, update the documentation and remove the now-unused SRsetup.iss file from the repository. If Inno Setup packaging should be retained, this deletion should be reverted.

Suggested change
# 使用 Inno Setup 生成 Windows 安装程序（.exe）
echo "开始生成 Inno Setup 安装程序..."
# 安装 Inno Setup（使用 Chocolatey，windows-latest 运行器中预装）
choco install innosetup -y
# 尝试定位 ISCC.exe
$possibleIsccPaths = @(
"C:\Program Files (x86)\Inno Setup 6\ISCC.exe",
"C:\Program Files\Inno Setup 6\ISCC.exe"
)
$isccPath = $possibleIsccPaths | Where-Object { Test-Path $_ } | Select-Object -First 1
if (-not $isccPath) {
echo "错误：未找到 Inno Setup 编译器 ISCC.exe"
Get-ChildItem "C:\Program Files" -Recurse -Filter "ISCC.exe" -ErrorAction SilentlyContinue | Select-Object -First 10 | ForEach-Object { $_.FullName }
exit 1
}
echo "使用 Inno Setup 编译器: $isccPath"
if (-not (Test-Path "SRsetup.iss" -PathType Leaf)) {
echo "错误：未找到 Inno Setup 脚本 SRsetup.iss，无法生成安装程序"
exit 1
}
# 可根据需要将版本号传递给 Inno Setup，比如 /DMyAppVersion
$version = "${{ github.ref_name }}"
& "$isccPath" "/DAppVersion=$version" "SRsetup.iss"
if ($LASTEXITCODE -ne 0) {
echo "错误：Inno Setup 编译失败，退出代码：$LASTEXITCODE"
exit $LASTEXITCODE
}
# 将生成的安装程序移动到 zip 目录，便于后续上传
$installerFiles = Get-ChildItem -Path . -Filter "*.exe" -Recurse | Where-Object { $_.Name -like "SecRandom*.exe" }
if (-not $installerFiles) {
echo "警告：未在仓库中找到生成的 SecRandom 安装程序 exe 文件"
} else {
foreach ($file in $installerFiles) {
echo "找到安装程序: $($file.FullName)，移动到 zip 目录"
Copy-Item -Path $file.FullName -Destination "zip/" -Force
}
}

Copilot uses AI. Check for mistakes.
Comment on lines 178 to 180
Copy link

Copilot AI Jan 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR title and description say "开始打包" (start packaging), but the actual change removes the Inno Setup packaging step. This is misleading and contradicts the actual changes made. The PR description should accurately describe that this removes Windows installer generation.

Copilot uses AI. Check for mistakes.
# Inno Setup 打包
- name: Inno Setup 打包
if: matrix.platform == 'windows'
run: |
echo "开始 Inno Setup 打包..."

# 确保构建输出目录存在
if (!(Test-Path "build")) {
mkdir build
}

# 运行 Inno Setup 编译器
& "C:\Program Files (x86)\Inno Setup 6\ISCC.exe" SRsetup.iss

# 检查安装程序是否生成并移动到 zip 目录
if (Test-Path "build/SecRandom setup x64.exe") {
# 修改文件名以包含版本号，方便识别
$setupName = "SecRandom-setup-${{ github.ref_name }}-${{ matrix.arch }}.exe"
Move-Item "build/SecRandom setup x64.exe" "zip/$setupName"
echo "Inno Setup 打包完成: zip/$setupName"
} else {
echo "错误：Inno Setup 安装程序未生成"
exit 1
}

# Linux 打包操作
- name: Linux 打包操作
if: matrix.platform == 'linux'
Expand Down
Loading