diff -u sssd-1.16.1/debian/changelog sssd-1.16.1/debian/changelog --- sssd-1.16.1/debian/changelog +++ sssd-1.16.1/debian/changelog @@ -1,3 +1,10 @@ +sssd (1.16.1-1ubuntu1.2) bionic; urgency=medium + + * d/p/GPO_CROND-customization.diff: Set GPO_CROND to cron instead of + crond for Debian and Ubuntu (LP: #1572908) + + -- Victor Tapia Wed, 27 Feb 2019 14:53:11 +0100 + sssd (1.16.1-1ubuntu1.1) bionic; urgency=medium * d/p/fix-id-out-of-range-lookup.patch: CACHE_REQ: Do not fail the domain diff -u sssd-1.16.1/debian/patches/series sssd-1.16.1/debian/patches/series --- sssd-1.16.1/debian/patches/series +++ sssd-1.16.1/debian/patches/series @@ -2,0 +3 @@ +GPO_CROND-customization.diff only in patch2: unchanged: --- sssd-1.16.1.orig/debian/patches/GPO_CROND-customization.diff +++ sssd-1.16.1/debian/patches/GPO_CROND-customization.diff @@ -0,0 +1,54 @@ +From bc65ba9a07a924a58b13a0d5a935114ab72b7524 Mon Sep 17 00:00:00 2001 +From: Victor Tapia +Date: Fri, 22 Feb 2019 14:54:13 +0100 +Subject: [PATCH] GPO: Allow customization of GPO_CROND per OS +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +For both Debian and Ubuntu, the cron PAM service is named "cron" instead +of "crond", denying the use of the service by default. + +This patch enables the HAVE_$OS (HAVE_DEBIAN/HAVE_FEDORA...) macros to select +the service name during build, allowing further customization if required. + +Resolves: +https://bugs.launchpad.net/ubuntu/+source/sssd/+bug/1572908 + +Reviewed-by: Michal Židek +--- + src/external/platform.m4 | 8 ++++++++ + src/providers/ad/ad_gpo.c | 4 ++++ + 2 files changed, 12 insertions(+) + +--- a/src/external/platform.m4 ++++ b/src/external/platform.m4 +@@ -36,6 +36,14 @@ + AM_CONDITIONAL([HAVE_DEBIAN], [test x"$osname" = xdebian]) + AM_CONDITIONAL([HAVE_GENTOO], [test x"$osname" = xgentoo]) + ++AS_CASE([$osname], ++ [redhat], [AC_DEFINE_UNQUOTED([HAVE_REDHAT], 1, [Build with redhat config])], ++ [fedora], [AC_DEFINE_UNQUOTED([HAVE_FEDORA], 1, [Build with fedora config])], ++ [suse], [AC_DEFINE_UNQUOTED([HAVE_SUSE], 1, [Build with suse config])], ++ [gentoo], [AC_DEFINE_UNQUOTED([HAVE_GENTOO], 1, [Build with gentoo config])], ++ [debian], [AC_DEFINE_UNQUOTED([HAVE_DEBIAN], 1, [Build with debian config])], ++ [AC_MSG_NOTICE([Build with $osname config])]) ++ + AC_CHECK_MEMBERS([struct ucred.pid, struct ucred.uid, struct ucred.gid], , , + [[#include ]]) + +--- a/src/providers/ad/ad_gpo.c ++++ b/src/providers/ad/ad_gpo.c +@@ -199,7 +199,11 @@ + #define GPO_SSHD "sshd" + #define GPO_FTP "ftp" + #define GPO_SAMBA "samba" ++#ifdef HAVE_DEBIAN ++#define GPO_CROND "cron" ++#else + #define GPO_CROND "crond" ++#endif + #define GPO_POLKIT "polkit-1" + #define GPO_SUDO "sudo" + #define GPO_SUDO_I "sudo-i"