On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote:
cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
I'm inclined to ack this patch, but I'd like to see the particular error first.
From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list
src/lib/sifp/sss_simpleifp.pc.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 --- a/src/lib/sifp/sss_simpleifp.pc.in +++ b/src/lib/sifp/sss_simpleifp.pc.in @@ -4,8 +4,7 @@ includedir=@includedir@ Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: @VERSION@ -Requires: dbus-1 -Requires: dhash +Requires: dbus-1, dhash Libs: -L@libdir@ -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/ -- 1.7.11.7
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
On (03/07/14 14:35), Jakub Hrozek wrote:
On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote:
cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
Field 'Requires' should contain a list of packages, therwise there is a warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Requires field occurs twice in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
user@a80c17581d5a:/tmp$ pkg-config --version 0.26
I think we can remove dependency "dbus-1". The review process of sss_simpleifp was very long and there were lots of changes.
I tested compilation of simple program (attached in mail). It works without any problem. gcc sifp.c -Wl,--as-needed `pkg-config --libs sss_simpleifp`
Pavel could you test with cmake in openlmi?
I'm inclined to ack this patch, but I'd like to see the particular error first.
From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list
src/lib/sifp/sss_simpleifp.pc.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 --- a/src/lib/sifp/sss_simpleifp.pc.in +++ b/src/lib/sifp/sss_simpleifp.pc.in @@ -4,8 +4,7 @@ includedir=@includedir@ Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: @VERSION@ -Requires: dbus-1 -Requires: dhash +Requires: dbus-1, dhash Libs: -L@libdir@ -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
Could you add next line to this file? prefix=@prefix@ because there is another pkg-config warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Variable 'prefix' not defined in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
LS
On 07/07/2014 02:01 PM, Lukas Slebodnik wrote:
On (03/07/14 14:35), Jakub Hrozek wrote:
On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote:
cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
Field 'Requires' should contain a list of packages, therwise there is a warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Requires field occurs twice in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
user@a80c17581d5a:/tmp$ pkg-config --version 0.26
I think we can remove dependency "dbus-1". The review process of sss_simpleifp was very long and there were lots of changes.
The dbus dependency is there once you include sss_sifp_dbus.h, so it is a question whether we can remove it from pkgconfig.
I tested compilation of simple program (attached in mail). It works without any problem. gcc sifp.c -Wl,--as-needed `pkg-config --libs sss_simpleifp`
Pavel could you test with cmake in openlmi?
I'm inclined to ack this patch, but I'd like to see the particular error first.
From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list
src/lib/sifp/sss_simpleifp.pc.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 --- a/src/lib/sifp/sss_simpleifp.pc.in +++ b/src/lib/sifp/sss_simpleifp.pc.in @@ -4,8 +4,7 @@ includedir=@includedir@ Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: @VERSION@ -Requires: dbus-1 -Requires: dhash +Requires: dbus-1, dhash Libs: -L@libdir@ -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
Could you add next line to this file? prefix=@prefix@ because there is another pkg-config warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Variable 'prefix' not defined in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
Hmm, didn't we remove it during the review process?
LS
sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
On (08/07/14 10:26), Pavel Březina wrote:
On 07/07/2014 02:01 PM, Lukas Slebodnik wrote:
On (03/07/14 14:35), Jakub Hrozek wrote:
On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote:
cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
Field 'Requires' should contain a list of packages, therwise there is a warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Requires field occurs twice in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
user@a80c17581d5a:/tmp$ pkg-config --version 0.26
I think we can remove dependency "dbus-1". The review process of sss_simpleifp was very long and there were lots of changes.
The dbus dependency is there once you include sss_sifp_dbus.h, so it is a question whether we can remove it from pkgconfig.
I see. In this case, you patch was right.
I tested compilation of simple program (attached in mail). It works without any problem. gcc sifp.c -Wl,--as-needed `pkg-config --libs sss_simpleifp`
Pavel could you test with cmake in openlmi?
I'm inclined to ack this patch, but I'd like to see the particular error first.
From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list
src/lib/sifp/sss_simpleifp.pc.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 --- a/src/lib/sifp/sss_simpleifp.pc.in +++ b/src/lib/sifp/sss_simpleifp.pc.in @@ -4,8 +4,7 @@ includedir=@includedir@ Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: @VERSION@ -Requires: dbus-1 -Requires: dhash +Requires: dbus-1, dhash Libs: -L@libdir@ -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
Could you add next line to this file? prefix=@prefix@ because there is another pkg-config warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Variable 'prefix' not defined in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
Hmm, didn't we remove it during the review process?
Yes, it was; but it was not good idea :-) It is not problem with default prefix "/usr". (fedora)
You can try to install into different prefix(exec_prefix) "/usr/local/"
sh-4.2$ cat /usr/local/lib/pkgconfig/sss_simpleifp.pc libdir=${exec_prefix}/lib includedir=${prefix}/include
Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: 1.11.92 Requires: dbus-1 Requires: dhash Libs: -L${exec_prefix}/lib -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
sh-4.2$ PKG_CONFIG_PATH=/usr/local/lib/pkgconfig/ pkg-config --libs sss_simpleifp Variable 'exec_prefix' not defined in '/usr/local/lib/pkgconfig/sss_simpleifp.pc'
You should add both variables prefix and exec_prefix prefix=@prefix@ exec_prefix=@exec_prefix@
LS
On 07/08/2014 12:51 PM, Lukas Slebodnik wrote:
On (08/07/14 10:26), Pavel Březina wrote:
On 07/07/2014 02:01 PM, Lukas Slebodnik wrote:
On (03/07/14 14:35), Jakub Hrozek wrote:
On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote:
cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
Field 'Requires' should contain a list of packages, therwise there is a warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Requires field occurs twice in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
user@a80c17581d5a:/tmp$ pkg-config --version 0.26
I think we can remove dependency "dbus-1". The review process of sss_simpleifp was very long and there were lots of changes.
The dbus dependency is there once you include sss_sifp_dbus.h, so it is a question whether we can remove it from pkgconfig.
I see. In this case, you patch was right.
I tested compilation of simple program (attached in mail). It works without any problem. gcc sifp.c -Wl,--as-needed `pkg-config --libs sss_simpleifp`
Pavel could you test with cmake in openlmi?
I'm inclined to ack this patch, but I'd like to see the particular error first.
From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list
src/lib/sifp/sss_simpleifp.pc.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 --- a/src/lib/sifp/sss_simpleifp.pc.in +++ b/src/lib/sifp/sss_simpleifp.pc.in @@ -4,8 +4,7 @@ includedir=@includedir@ Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: @VERSION@ -Requires: dbus-1 -Requires: dhash +Requires: dbus-1, dhash Libs: -L@libdir@ -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
Could you add next line to this file? prefix=@prefix@ because there is another pkg-config warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Variable 'prefix' not defined in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
Hmm, didn't we remove it during the review process?
Yes, it was; but it was not good idea :-) It is not problem with default prefix "/usr". (fedora)
You can try to install into different prefix(exec_prefix) "/usr/local/"
sh-4.2$ cat /usr/local/lib/pkgconfig/sss_simpleifp.pc libdir=${exec_prefix}/lib includedir=${prefix}/include
Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: 1.11.92 Requires: dbus-1 Requires: dhash Libs: -L${exec_prefix}/lib -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
sh-4.2$ PKG_CONFIG_PATH=/usr/local/lib/pkgconfig/ pkg-config --libs sss_simpleifp Variable 'exec_prefix' not defined in '/usr/local/lib/pkgconfig/sss_simpleifp.pc'
You should add both variables prefix and exec_prefix prefix=@prefix@ exec_prefix=@exec_prefix@
LS
Hi, thanks. Patches are attached.
On (08/07/14 19:37), Pavel Březina wrote:
On 07/08/2014 12:51 PM, Lukas Slebodnik wrote:
On (08/07/14 10:26), Pavel Březina wrote:
On 07/07/2014 02:01 PM, Lukas Slebodnik wrote:
On (03/07/14 14:35), Jakub Hrozek wrote:
On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote:
cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
Field 'Requires' should contain a list of packages, therwise there is a warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Requires field occurs twice in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
user@a80c17581d5a:/tmp$ pkg-config --version 0.26
I think we can remove dependency "dbus-1". The review process of sss_simpleifp was very long and there were lots of changes.
The dbus dependency is there once you include sss_sifp_dbus.h, so it is a question whether we can remove it from pkgconfig.
I see. In this case, you patch was right.
I tested compilation of simple program (attached in mail). It works without any problem. gcc sifp.c -Wl,--as-needed `pkg-config --libs sss_simpleifp`
Pavel could you test with cmake in openlmi?
I'm inclined to ack this patch, but I'd like to see the particular error first.
From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list
src/lib/sifp/sss_simpleifp.pc.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 --- a/src/lib/sifp/sss_simpleifp.pc.in +++ b/src/lib/sifp/sss_simpleifp.pc.in @@ -4,8 +4,7 @@ includedir=@includedir@ Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: @VERSION@ -Requires: dbus-1 -Requires: dhash +Requires: dbus-1, dhash Libs: -L@libdir@ -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
Could you add next line to this file? prefix=@prefix@ because there is another pkg-config warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Variable 'prefix' not defined in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
Hmm, didn't we remove it during the review process?
Yes, it was; but it was not good idea :-) It is not problem with default prefix "/usr". (fedora)
You can try to install into different prefix(exec_prefix) "/usr/local/"
sh-4.2$ cat /usr/local/lib/pkgconfig/sss_simpleifp.pc libdir=${exec_prefix}/lib includedir=${prefix}/include
Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: 1.11.92 Requires: dbus-1 Requires: dhash Libs: -L${exec_prefix}/lib -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
sh-4.2$ PKG_CONFIG_PATH=/usr/local/lib/pkgconfig/ pkg-config --libs sss_simpleifp Variable 'exec_prefix' not defined in '/usr/local/lib/pkgconfig/sss_simpleifp.pc'
You should add both variables prefix and exec_prefix prefix=@prefix@ exec_prefix=@exec_prefix@
LS
Hi, thanks. Patches are attached.
From 2956a495d6b64edd6a6b97de9484665c2c99ecc2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH 1/2] sss_sifp: pkg-config requires is a comma separated list
ACK
From 73615efea61e5697ee7b8f685c3a21a7d43acf74 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Tue, 8 Jul 2014 19:37:04 +0200 Subject: [PATCH 2/2] sss_sifp: add prefix and exec_prefix to pkg-config
ACK
LS
On Wed, Jul 09, 2014 at 10:50:17AM +0200, Lukas Slebodnik wrote:
On (08/07/14 19:37), Pavel Březina wrote:
On 07/08/2014 12:51 PM, Lukas Slebodnik wrote:
On (08/07/14 10:26), Pavel Březina wrote:
On 07/07/2014 02:01 PM, Lukas Slebodnik wrote:
On (03/07/14 14:35), Jakub Hrozek wrote:
On Wed, Jul 02, 2014 at 01:46:49PM +0200, Pavel Březina wrote: >cmake complained about this when building openlmi
What was the error you were seeing?
As a note, I was reading the pkg-config docs again: http://people.freedesktop.org/~dbn/pkg-config-guide.html and I think it's correct we use Requires and not Requires.private since we do expose some libdbus types in our public header.
Field 'Requires' should contain a list of packages, therwise there is a warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Requires field occurs twice in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
user@a80c17581d5a:/tmp$ pkg-config --version 0.26
I think we can remove dependency "dbus-1". The review process of sss_simpleifp was very long and there were lots of changes.
The dbus dependency is there once you include sss_sifp_dbus.h, so it is a question whether we can remove it from pkgconfig.
I see. In this case, you patch was right.
I tested compilation of simple program (attached in mail). It works without any problem. gcc sifp.c -Wl,--as-needed `pkg-config --libs sss_simpleifp`
Pavel could you test with cmake in openlmi?
I'm inclined to ack this patch, but I'd like to see the particular error first.
> From 3b945c3da5b3717752451cac7868b285f158eabd Mon Sep 17 00:00:00 2001 >From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com >Date: Wed, 2 Jul 2014 13:45:53 +0200 >Subject: [PATCH] sss_sifp: pkg-config requires is a comma separated list > >--- > src/lib/sifp/sss_simpleifp.pc.in | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > >diff --git a/src/lib/sifp/sss_simpleifp.pc.in b/src/lib/sifp/sss_simpleifp.pc.in >index a8b625608b0e8451b8fa44e1a927d51a3e11cef1..e57ed8db4174db433d0aad56cbd8179c140e0b17 100644 >--- a/src/lib/sifp/sss_simpleifp.pc.in >+++ b/src/lib/sifp/sss_simpleifp.pc.in >@@ -4,8 +4,7 @@ includedir=@includedir@ > Name: sss_simpleifp > Description: A library that simplifies work with the InfoPipe responder > Version: @VERSION@ >-Requires: dbus-1 >-Requires: dhash >+Requires: dbus-1, dhash > Libs: -L@libdir@ -lsss_simpleifp > Cflags: -I${includedir} > URL: http://fedorahosted.org/sssd/
Could you add next line to this file? prefix=@prefix@ because there is another pkg-config warning
user@a80c17581d5a:/tmp$ pkg-config --libs sss_simpleifp Variable 'prefix' not defined in '/usr/lib/x86_64-linux-gnu/pkgconfig/sss_simpleifp.pc'
Hmm, didn't we remove it during the review process?
Yes, it was; but it was not good idea :-) It is not problem with default prefix "/usr". (fedora)
You can try to install into different prefix(exec_prefix) "/usr/local/"
sh-4.2$ cat /usr/local/lib/pkgconfig/sss_simpleifp.pc libdir=${exec_prefix}/lib includedir=${prefix}/include
Name: sss_simpleifp Description: A library that simplifies work with the InfoPipe responder Version: 1.11.92 Requires: dbus-1 Requires: dhash Libs: -L${exec_prefix}/lib -lsss_simpleifp Cflags: -I${includedir} URL: http://fedorahosted.org/sssd/
sh-4.2$ PKG_CONFIG_PATH=/usr/local/lib/pkgconfig/ pkg-config --libs sss_simpleifp Variable 'exec_prefix' not defined in '/usr/local/lib/pkgconfig/sss_simpleifp.pc'
You should add both variables prefix and exec_prefix prefix=@prefix@ exec_prefix=@exec_prefix@
LS
Hi, thanks. Patches are attached.
From 2956a495d6b64edd6a6b97de9484665c2c99ecc2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Wed, 2 Jul 2014 13:45:53 +0200 Subject: [PATCH 1/2] sss_sifp: pkg-config requires is a comma separated list
ACK
From 73615efea61e5697ee7b8f685c3a21a7d43acf74 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= pbrezina@redhat.com Date: Tue, 8 Jul 2014 19:37:04 +0200 Subject: [PATCH 2/2] sss_sifp: add prefix and exec_prefix to pkg-config
ACK
LS
Pushed to master: 2202090b9e54b566b37361462d5cb4bb34b815a1 da7a0a4622e9bb3232f30d868ed8d3b826c7ccc8
sssd-devel@lists.fedorahosted.org