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
nixos/mediawiki: init service to replace httpd subservice #62748
Conversation
a08bd81
to
716110b
Compare
@GrahamcOfBorg test mediawiki |
@redvers Do you use the |
This pull request has been mentioned on Nix community. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/15 |
My contribution to this package was all about fixing a security issue - the task of bringing it into a separate module was above my pay grade. I'm sure I'll learn a lot from reading your PR - Thanks for doing it! There is documentation on nixos.wiki here: https://nixos.wiki/wiki/Mediawiki which we will need to update (and I'm happy to do so). At first glance it looks like the configuration you generate in this new module isn't compatible with existing configurations so I don't think it will be a seamless migration. Are you planning on adding a deprecation message to the old subsystem such that it will throw a warning for existing users like: "subSystem is going away, please move to native mediawiki" or somesuch? |
@redvers My plan is to add an entry to the release notes and write a quick migration guide. If I was able to migrate all Thanks for your offer to write the migration guide! I'll ask again if you are using a Thanks! |
@GrahamcOfBorg test mediawiki |
I'm testing a new installation now - I'll follow up shortly. |
Looks great from here - thanks for doing this integration - will make life easier in the long run. |
779f14c
to
c16d081
Compare
Just one issue (which is probably just something that should be documented...) I don't seem to be able to use the maintenance scripts. Maybe I need to specify more than just php and mediawiki in order for it to resolve the include in the default LocalSettings.php. It contains:
|
So, if we can export MEDIAWIKI_CONFIG into a shell automatically somehow, it will work. I can get it to function like so:
|
@redvers Please test and let me know if any other issues come up. Thanks for reporting! |
@redvers what would be the most common commands used for maintenance (ie. what On a side note there has been some changes going on with |
7cf14a7
to
28e4c8b
Compare
ping @redvers |
b9aa8a4
to
2cff55e
Compare
@GrahamcOfBorg test mediawiki @redvers I have gone back to the way the current |
Merging based on previous input from @redvers and sufficient testing on my own part. |
Motivation for this change
Continue to remove
httpd.extraSubservices
.See part 1 and part 2 for prior work on the topic. I still need to write release notes for 19.09, but waiting for module review before going ahead with that.
A big thanks to @tstarling of the Wikimedia foundation for all his help on this module over IRC.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)