search

mate-desktop / mate-control-center

Utilities to configure the MATE desktop
https://mate-desktop.org
GNU General Public License v2.0
68 stars 59 forks source link

Add time-admin dependency check on systemd #748

Closed zhuyaliang closed 8 months ago

zhuyaliang commented 8 months ago

time-admin depends on systemd. If systemd is not used in the systemd, time-admin is disabled

zhuyaliang commented 8 months ago

@cwendling I am not very familiar with autoconf, can I modify it this way?

lukefromdc commented 8 months ago

Looks to me like we are setting a variable, then setting it to a non-null value only if systemd is detected. I also don't know the build systems that well but this should have the same effect when systemd is not present as the = \ lines as that's what it should expand to in those cases

We also support meson for mate-control-center, though only one actually HAS to work for non-systemd distros to build m-c-c today, the point to any meson work is an eventual transition to it. That is important especially if autotools ever goes unmaintained.

zhuyaliang commented 8 months ago

After this PR is accepted, I will add meson support for checking systemd

zhuyaliang commented 8 months ago

Fix #744