[Pkg-sssd-devel] [Git][sssd-team/sssd][ubuntu-bionic] GPO: Allow customization of GPO_CROND per OS

[Victor Tapia]

Victor Tapia pushed to branch ubuntu-bionic at Debian SSSD packaging / sssd


Commits:
07648987 by Victor Tapia at 2019-02-28T08:26:25Z
GPO: Allow customization of GPO_CROND per OS

- - - - -


3 changed files:

- debian/changelog
- + debian/patches/GPO_CROND-customization.diff
- debian/patches/series


Changes:

=====================================
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 <victor.tapia at canonical.com>  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


=====================================
debian/patches/GPO_CROND-customization.diff
=====================================
@@ -0,0 +1,54 @@
+From bc65ba9a07a924a58b13a0d5a935114ab72b7524 Mon Sep 17 00:00:00 2001
+From: Victor Tapia <victor.tapia at canonical.com>
+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 <mzidek at redhat.com>
+---
+ 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 <sys/socket.h>]])
+ 
+--- 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"


=====================================
debian/patches/series
=====================================
@@ -1,2 +1,3 @@
 #placeholder
 fix-id-out-of-range-lookup.patch
+GPO_CROND-customization.diff



View it on GitLab: https://salsa.debian.org/sssd-team/sssd/commit/07648987ec0e8d943ff9ef5f6686a12d6ed1fad8

-- 
View it on GitLab: https://salsa.debian.org/sssd-team/sssd/commit/07648987ec0e8d943ff9ef5f6686a12d6ed1fad8
You're receiving this email because of your account on salsa.debian.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-sssd-devel/attachments/20190228/6db7b773/attachment-0001.html>