-
-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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
iterm2: 3.5.10 -> 3.5.11 #370402
iterm2: 3.5.10 -> 3.5.11 #370402
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The change looks good, I do not have a macOS system to test it, it would be nice if a Darwin user could do a quick smoke test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine for me
nixpkgs-review
result
Generated using nixpkgs-review
.
Command: nixpkgs-review pr 370402
aarch64-darwin
✅ 1 package built:
- iterm2
Thanks for the confirmation! |
Backport failed for Please cherry-pick the changes locally and resolve any conflicts. git fetch origin release-24.11
git worktree add -d .worktree/backport-370402-to-release-24.11 origin/release-24.11
cd .worktree/backport-370402-to-release-24.11
git switch --create backport-370402-to-release-24.11
git cherry-pick -x 2ce46489b3ebc4903355f05ab33f1458ed860b80 |
@DimitarNestorov 3.5.10 thankfully wasn't backported to 24.11. Ideally you'd create a manual backport PR cherry-picking the update commits from both PRs (so the vulnerable version isn't backported in isolation). |
Manual backport: #370837 |
Fixes a critical security issue in iTerm2's ssh integration: https://iterm2.com/downloads/stable/iTerm2-3_5_11.changelog
nixpkgs-24.11 is not affected (has 3.5.4)
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.