On Wed, Aug 17, 2016 at 09:47:36AM +0200, Jakub Hrozek wrote:
On Wed, Aug 17, 2016 at 09:40:38AM +0200, Lukas Slebodnik wrote:
> On (16/08/16 17:01), Jakub Hrozek wrote:
> >To test, run:
> > rpm -qf /var/lib/sss/secrets/
> >the directory was unowned before and should be owned by sssd-common now.
>
> >From a398c91ef196be680f6c2d5b2d5251e060005ec0 Mon Sep 17 00:00:00 2001
> >From: Jakub Hrozek <jhrozek(a)redhat.com>
> >Date: Tue, 16 Aug 2016 16:45:36 +0200
> >Subject: [PATCH] SPEC: Own the secrets DB path
> >
> >---
> > contrib/sssd.spec.in | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> >diff --git a/contrib/sssd.spec.in b/contrib/sssd.spec.in
> >index
b58eebba54a82a041f72507b430ceca708a34632..05bb83b57015da5e730dd9b0d9eae5d760596597 100644
> >--- a/contrib/sssd.spec.in
> >+++ b/contrib/sssd.spec.in
> >@@ -106,6 +106,7 @@ Requires: python-sssdconfig = %{version}-%{release}
> > %global mcpath %{sssdstatedir}/mc
> > %global pubconfpath %{sssdstatedir}/pubconf
> > %global gpocachepath %{sssdstatedir}/gpo_cache
> >+%global secdbpath %{sssdstatedir}/secrets
> >
> > ### Build Dependencies ###
> >
> >@@ -619,6 +620,7 @@ autoreconf -ivf
> > --with-pubconf-path=%{pubconfpath} \
> > --with-gpo-cache-path=%{gpocachepath} \
> > --with-init-dir=%{_initrddir} \
> >+ --with-secrets-db-path=%{secdbpath} \
> Is there a reason to call configure with this option?
> "%{secdbpath}" is already a default.
No, I just wanted to be consistent with how we set the other paths, like
gpo cache or even the db, that's all.
I'm fine with dropping this line if you prefer.
OK, attached is a patch w/o the explicit configure line.