[WIP] Fix bug in install.sh for temporary file cleanup

[Copilot]

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.

Original prompt

问题描述

当前 install.sh 脚本的第 101 行存在一个 bug，导致脚本退出后临时文件无法被正确清理。

trap 'rm -rf "$tmp_dir"' EXIT INT TERM HUP

问题原因

在单引号 '...' 中，Shell 不会展开变量。这意味着：

• 当 trap 被触发时，它会尝试删除字面字符串 "$tmp_dir"（包括引号和美元符号）
• 实际的临时目录路径不会被替换，导致 rm -rf 命令删除错误的路径
• 结果是临时文件残留在系统中

解决方案

将第 101 行修改为：

trap "rm -rf '$tmp_dir'" EXIT INT TERM HUP

或者：

trap "rm -rf \"$tmp_dir\"" EXIT INT TERM HUP

使用双引号包裹命令字符串，这样 $tmp_dir 会在 trap 设置时被展开为实际的临时目录路径��

测试方法

修改前：

# 运行脚本后检查临时目录
ls /tmp/ | grep tmp.
# 会看到残留的临时目录

修改后：

# 运行脚本后检查临时目录
ls /tmp/ | grep tmp.
# 临时目录应该被正确清理

相关文件

• install.sh 第 101 行

引用

原始代码位置：

get.pnpm.io/install.sh

Line 101 in da9123b

trap 'rm -rf "$tmp_dir"' EXIT INT TERM HUP

This pull request was created as a result of the following prompt from Copilot chat.

问题描述

当前 install.sh 脚本的第 101 行存在一个 bug，导致脚本退出后临时文件无法被正确清理。

trap 'rm -rf "$tmp_dir"' EXIT INT TERM HUP

问题原因

在单引号 '...' 中，Shell 不会展开变量。这意味着：

• 当 trap 被触发时，它会尝试删除字面字符串 "$tmp_dir"（包括引号和美元符号）
• 实际的临时目录路径不会被替换，导致 rm -rf 命令删除错误的路径
• 结果是临时文件残留在系统中

解决方案

将第 101 行修改为：

trap "rm -rf '$tmp_dir'" EXIT INT TERM HUP

或者：

trap "rm -rf \"$tmp_dir\"" EXIT INT TERM HUP

使用双引号包裹命令字符串，这样 $tmp_dir 会在 trap 设置时被展开为实际的临时目录路径��

测试方法

修改前：

# 运行脚本后检查临时目录
ls /tmp/ | grep tmp.
# 会看到残留的临时目录

修改后：

# 运行脚本后检查临时目录
ls /tmp/ | grep tmp.
# 临时目录应该被正确清理

相关文件

• install.sh 第 101 行

引用

原始代码位置：

get.pnpm.io/install.sh

Line 101 in da9123b

trap 'rm -rf "$tmp_dir"' EXIT INT TERM HUP

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
get-pnpm-io	Ready	Preview	Comment	Nov 15, 2025 0:07am

[netlify]

✅ Deploy Preview for getpnpm ready!

Name	Link
🔨 Latest commit	581fd05
🔍 Latest deploy log	https://app.netlify.com/projects/getpnpm/deploys/69186ce4232a80000838c90c
😎 Deploy Preview	https://deploy-preview-42--getpnpm.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.