Skip to content
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

flashplayer: 29.0.0.113 -> 29.0.0.140 [Critical security fixes] #38765

Merged
merged 1 commit into from Apr 11, 2018

Conversation

taku0
Copy link
Contributor

@taku0 taku0 commented Apr 11, 2018

Motivation for this change

“important bug fixes and security updates”

https://helpx.adobe.com/flash-player/release-note/fp_29_air_29_release_notes.html
https://helpx.adobe.com/security/products/flash-player/apsb18-08.html

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)

  • Built on platform(s)

    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"

  • Tested execution of all binary files (usually in ./result/bin/)

  • Fits CONTRIBUTING.md.

  • Built packages

    • nix-env -i -f . -A pkgs.flashplayer
    • nix-build -I nixpkgs=. -A pkgsi686Linux.flashplayer
    • nix-build -I nixpkgs=. -E "(import <nixpkgs> {}).flashplayer.override { debug = true; }"
    • nix-build -I nixpkgs=. -E "(import <nixpkgs> {}).pkgsi686Linux.flashplayer.override { debug = true; }"
    • nix-build -I nixpkgs=. -A pkgs.flashplayer-standalone
    • nix-build -I nixpkgs=. -A pkgs.flashplayer-standalone-debugger
    • nix-build -I nixpkgs=. -E "(import <nixpkgs> {}).chromium.override { enablePepperFlash = true; }"
  • Played a video with Firefox and Chromium (x86_64).

  • Checked the version with Firefox, Chromium, flashplayer-standalone, and flashplayer-standalone-debugger (x86_64) http://www.adobe.com/software/flash/about/


@7c6f434c 7c6f434c merged commit 896cc08 into NixOS:master Apr 11, 2018
@7c6f434c
Copy link
Member

Are there any committers left who don't know that flashplayer update can only happen because of security issues? I would prefer «needs cherry-picking to 17.09 and 18.03» as a more useful reminder.

(looks at the PR track record) maybe you should also ask for commit access and push such things directly

7c6f434c added a commit that referenced this pull request Apr 11, 2018
flashplayer: 29.0.0.113 -> 29.0.0.140 [Critical security fixes]
(cherry picked from commit 896cc08)
7c6f434c added a commit that referenced this pull request Apr 11, 2018
flashplayer: 29.0.0.113 -> 29.0.0.140 [Critical security fixes]
(cherry picked from commit 896cc08)
@7c6f434c
Copy link
Member

pushed to stable branches

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants