-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Systemd notify only - no socket activation #100
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
|
I merged everything but CPackRPM workaround to https://github.com/MariaDB/server/tree/bb-systemd. Also I adjusted a couple of things. What are the other options to workaround CPackRPM bug? |
install it as a non @ filename and rename it in a post install script Could install a separate mariadb_bootstrap.service - will be different from Percona and means user customisation will affect the main server service and not the bootstrap. remove the multi instance version (and hence mariadb@bootstrap). probably distributing the cmakeRPM of the earliest supported cmake (2.8) version would of been better. |
|
From the above options anything but distributing CMakeRPM is acceptable. If Percona has such file, how did it workaround this issue? Why mariadb-bootstrap.conf is installed as wsrep-new-cluster.conf in debian scripts and as it's original name otherwise? |
|
Why SYSTEMD_SYSTEM_CONFDIR is unused? |
- unset HAVE_SYSTEMD along with other HAVE_* variables if systemd was not found - s/mariadb-system-convert/mariadb-service-convert/ - mariadb-bootstrap.conf is in source dir (not build dir)
- package mariadb-service-convert only if systemd was requested
- run mariadb-service-convert only if systemd is actually available
- use %{_bindir} instead of hardcoded /usr/bin
|
Percona seems to rely on spec files: I'll get an error to generate if building with a broken cpackRPM and systemd enabled. Will clean up the consistencies with Debian too. |
|
@grooverdan, please note that I continue development of systemd in bb-systemd, it has quite a few fixes already. Also I rebase and adjust commits and do git push --force now and then. |
|
ack. I'll just did a merge. A fairly heavy modification of the debian packaging hit 10.1 that I've merged. Have set up VMs to test packaging more. Will keep pushing here as I fix stuff. |
|
all merged |
@svoj
TODO: more testing on packaging
is fc0e419 too dodgy as work around for http://public.kitware.com/Bug/view.php?id=14782. I could manipulate settings to avoid this using distro version.
changed OPTIONS to EXTRA_ARGS to be closer to compatible to percona systemd script
( https://github.com/percona/percona-xtradb-cluster/blob/5.6/build-ps/rpm/[email protected] )