Hello, Am Donnerstag, 29. Juni 2017, 11:31:32 CEST schrieb Theo Chatzimichos: ...
I took a look, in general they are fine and production ready imho.
:-)
I took the following notes, but they are mostly minor:
elasticsearch - why not linkpac from security:logging:elma, take changes from :devel, andrew and his and send them all as SR to security:logging:elma
The SR to security:logging:elma probably makes sense - I'll do that in the next days. Do I need to follow the link chain (which would mean to ask Andrew to do the SR), or can I send the SR directly to security:logging:elma without breaking the linked packages in between?
- why it has old distro / non-systemd support? - why is the Source1 URL commented out? - what is %setup -a1? - what is %build true? maybe just remove the "true"? - getent group || groupadd - on package removal, user and group should be preserved - instead of systemctl reload, it should do the same %service_*_* as it did for the main package
Let me try a summary answer ;-) I took the package from Andrew's home repo because it was clearly the best updated I could get. I only did a minor version update which means I only touched the version number in the spec. For everything else, please blame the previous packagers ;-) As already mentioned elsewhere, elasticsearch 1.x is EOL (but MediaWiki 1.27.x unfortunately still requires this version), so I'd prefer not to waste time on this package ;-) For newer elasticsearch versions, there are already separate packages, so improvements in the 1.x package won't improve the newer packages automatically. That said - if you still think I should spend time on this package and/ or answer your comments in detail, please tell me ;-)
mediawiki-1_27 - instead of linkpac to home:cboltz:infra, I would suggest to remove the _link and osc branch openSUSE:infrastructure:wiki/$pkg
Indeed, that makes sense. I just did osc detachbranch all packages (and made the packages in home:cboltz:infra links instead). If the elasticsearch SR gets accepted, we can in theory linkpac elasticsearch to security:... - but since there won't be any new 1.x releases, that won't be really useful.
- remove the commented out Requires and Conflicts
I assume you are talking about the mediawiki_1_27 package here. The commented out Requires and Conflicts basically mean differences to the package in server:php:applications (which I used as a base, but it was too different from my needs to just use it). (Actually, thanks for the reminder - I had to add a few missing dependencies which I should send to the package in server:php:applications) The ImageMagick dependency depends on the MediaWiki config - for historical reasons, the openSUSE wiki doesn't use ImageMagick. php-gettext really seems to be superfluous, so I'll remove that. For the versioned vs. unversioned php5 conflict - that's also something I'll remove.
- mediawiki gets installed in /usr/share/mediawiki-1_27, how are you handling it in production? copying it somewhere or using it from there directly?
Does salt/profile/wiki/docroot.sls answer your question? ;-) TL;DR: the document root is in /srv/www/$wiki/public/, and I let salt create symlinks to /usr/share/mediawiki_1_27/ there. Regards, Christian Boltz -- Linux just isn't user-friendly when it comes to viruses. You have to work to find and run them. It doesn't happen automatically as it does with Windows. The GNU/Linux folks really should improve this glaring discrepancy. [http://os.newsforge.com/article.pl?sid=05/01/25/1430222&from=rss] -- To unsubscribe, e-mail: heroes+unsubscribe@opensuse.org To contact the owner, e-mail: heroes+owner@opensuse.org