-
Notifications
You must be signed in to change notification settings - Fork 30
packaging: rpms: Add systemd support for openSUSE #31
Conversation
Thank you. Looks like we'll need to produce two separate packages for SUSE like we do for CentOS. |
@hwoarang how recent is recent? We can't drop support for versions below 42 in |
@michaelklishin Well I was mostly referring to 42.X indeed. The rest of the openSUSE versions have reached EOL. In fact, 42.1 has also reached EOL. https://en.opensuse.org/Lifetime So it's up to you I suppose. If you need to build two variants for pre- and post- 42.X then I can modify this commit to use |
@michaelklishin @dumbbell @hwoarang Thank you for your PR. I tried the setup and it raised this error:
( |
4db0752
to
8f8633c
Compare
Thank you for the report. I believe the updated PR fixes that. |
thanks @hwoarang but there are still problems:
I created this docker image to test the build, you could use it to test your script:
Then:
|
Recent openSUSE releases have been migrated to systemd so use the %{suse_version} macro to detect if we are building on openSUSE so we can include the necessary systemd files and also perform the needed systemd steps. This also defines the 'suse_version' to 1315 which is what openSUSE Leap 42.x use. This also drops the buggy rhel7 conditional for the sysvinit script and use another 'find' command instead. The previous condition was wrong since it would also try to include the sysvinit file on openSUSE and Fedora which do not define the %rhel macro and as such the 0 < 7 conditional was always true.
8f8633c
to
2211e5f
Compare
@Gsantomaggio thank you that was very helpful. I fixed the problems and it should be OK now. |
@hwoarang ok now the builds work correctly ! :)! Tested the package on
and:
|
@hwoarang @Gsantomaggio @dumbbell @gerhard @hairyhum @kjnilsson @dcorbacho @lukebakken I'm still not 100% sure where we are with the number of packages we should support for OpenSUSE. Can we have just one or there are reasonable arguments for supporting two (like we do for CentOS: 6 and 7)? |
We decided to not create a separate package for 42.1 and older releases. This will be in Thank you, @hwoarang! |
Recent openSUSE releases have been migrated to systemd so use the
%{suse_version} macro to detect if we are building on openSUSE
so we can include the necessary systemd files and also perform the
needed systemd steps. This also defines the 'suse_version' to 1315
which is what openSUSE Leap 42.x use.