-
-
Notifications
You must be signed in to change notification settings - Fork 15.4k
[20.09] mediawiki: update 1.34.2 -> 1.34.4 #102706
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
Conversation
Is bumping the mediawiki version the only supported way to fix those issues? |
They are also fixed in 1.34.4 which could be backported. |
Then let's stay the 1.34.x release cycle. |
Sounds good - will do. |
6268e6b
to
0b9702b
Compare
@GrahamcOfBorg test mediawiki |
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.
Please recheck that you're following https://github.com/NixOS/nixpkgs/blob/master/.github/CONTRIBUTING.md. The commit message, for example, is off.
0b9702b
to
9b63830
Compare
Okay - dealt with that. Do I need to worry about the now mis-named branchnames? If so, I have no idea how to rename the branches locally, remotely, and to assure that it doesn't automatically close the PR (which is what happened last time I tried to rename them) |
The commit message is still off: - mediawiki: update 1.34.2 -> 1.34.4
+ mediawiki: 1.34.2 -> 1.34.4 Your branch name is immaterial after merging this, don't worry about it. |
9b63830
to
60544f9
Compare
Thanks for taking the time to sweat the details. I'm working my way through all the rest of the PRs I've opened which have similar issues. Appreciate it. |
Motivation for this change
Bringing this up to current - same version that is in current master.
Fixes Security Issues:
#100310
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)