[SSSD] [PATCH] two build patches

Sumit Bose sbose at redhat.com
Wed Oct 14 08:14:31 UTC 2009


Hi,

I found the following two patches useful:

0001: distutils on SUSE defaults to /usr/local when installing Python
      modules. This is because they want the same default behaviour as
      autotools. Giving an explicit prefix might be a good idea
      anyways. I'm just wondering is PYTHON_PREFIX is the right variable
      to use.

0002: %defattr is obviously missing

bye,
Sumit
-------------- next part --------------
>From 023337e1bab9bb3fe729c5af3d202a5d53f4dc58 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Tue, 13 Oct 2009 22:21:43 +0200
Subject: [PATCH 1/2] use PYTHON_PREFIX to install SSSDConfig python API

---
 server/Makefile.am |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/server/Makefile.am b/server/Makefile.am
index 611a81e..8c27593 100644
--- a/server/Makefile.am
+++ b/server/Makefile.am
@@ -583,9 +583,9 @@ installsssddirs::
 
 install-exec-hook: installsssddirs
 	if [ "$(DESTDIR)" = "" ]; then \
-		cd $(srcdir)/config; $(PYTHON) setup.py install; \
+		cd $(srcdir)/config; $(PYTHON) setup.py install --prefix=$(PYTHON_PREFIX); \
 	else \
-		cd $(srcdir)/config; $(PYTHON) setup.py install --root=$(DESTDIR); \
+		cd $(srcdir)/config; $(PYTHON) setup.py install  --prefix=$(PYTHON_PREFIX) --root=$(DESTDIR); \
 	fi
 
 clean-local:
-- 
1.6.2.5

-------------- next part --------------
>From 214576b90c305cc7ac144d2fca651c521985d796 Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Tue, 13 Oct 2009 22:40:07 +0200
Subject: [PATCH 2/2] add missing %defattr to the filelist of the client package

---
 contrib/sssd.spec.in |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
index 670f1c9..41818f9 100644
--- a/contrib/sssd.spec.in
+++ b/contrib/sssd.spec.in
@@ -156,6 +156,7 @@ rm -rf $RPM_BUILD_ROOT
 
 
 %files client
+%defattr(-,root,root,-)
 /%{_lib}/libnss_sss.so.2
 /%{_lib}/security/pam_sss.so
 %{_mandir}/man8/pam_sss.8*
-- 
1.6.2.5



More information about the sssd-devel mailing list