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
[20.09] brave: 1.15.76 -> 1.17.73 #103658
[20.09] brave: 1.15.76 -> 1.17.73 #103658
Conversation
(cherry picked from commit 13e4f18) Reason: The browser must be kept up-to-date
Looks good to me. Tested by building it and browsing a few websites. 1.17 just hit unstable recently. If you feel like bumping this PR to 1.17 I will re-review it. If not, then it's good as-is. |
I am planning to update my PR to include 1.17 as well. I will poke you once it is done. |
I will wait for another version to come up to fix the CVE. |
@JeffLabonte I wouldn't wait on this... 1.16.72 and below has some memory corruption vulnerabilities in it. CVE-2020-8276 is pretty minor in my view. I would bump this to at least 1.16.76 (I would do 1.17) and push it through now. |
@buckley310 Done 😄 |
Looks good to me. Checked out the 20.09 branch from a 20.09 system, and cherry-picked these commits. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: |
Backported from: #103268
(cherry picked from commit 13e4f18)
Reason: The browser must be kept up-to-date
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)