All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] fix build of unittest_formatter
@ 2012-11-28 15:05 Danny Al-Gaaf
  2012-12-11 12:08 ` Danny Al-Gaaf
  0 siblings, 1 reply; 3+ messages in thread
From: Danny Al-Gaaf @ 2012-11-28 15:05 UTC (permalink / raw)
  To: ceph-devel; +Cc: Sage Weil, Danny Al-Gaaf

Add CRYPTO_CXXFLAGS to unittest_formatter_CXXFLAGS to find pk11pub.h to
be included in src/common/ceph_crypto.h.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
---
 src/Makefile.am | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index be851d6..2f6f486 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -764,7 +764,7 @@ check_PROGRAMS += unittest_heartbeatmap
 unittest_formatter_SOURCES = test/formatter.cc rgw/rgw_formats.cc
 unittest_formatter_LDFLAGS = $(PTHREAD_CFLAGS) ${AM_LDFLAGS}
 unittest_formatter_LDADD = ${UNITTEST_LDADD} $(LIBGLOBAL_LDA)
-unittest_formatter_CXXFLAGS = ${AM_CXXFLAGS} ${UNITTEST_CXXFLAGS}
+unittest_formatter_CXXFLAGS = ${CRYPTO_CXXFLAGS} ${AM_CXXFLAGS} ${UNITTEST_CXXFLAGS}
 check_PROGRAMS += unittest_formatter
 
 unittest_libcephfs_config_SOURCES = test/libcephfs_config.cc
-- 
1.8.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] fix build of unittest_formatter
  2012-11-28 15:05 [PATCH] fix build of unittest_formatter Danny Al-Gaaf
@ 2012-12-11 12:08 ` Danny Al-Gaaf
  2012-12-11 16:44   ` Sage Weil
  0 siblings, 1 reply; 3+ messages in thread
From: Danny Al-Gaaf @ 2012-12-11 12:08 UTC (permalink / raw)
  To: ceph-devel; +Cc: Sage Weil

Hi,

did you have a chance to take a look at this patch?

Danny

Am 28.11.2012 16:05, schrieb Danny Al-Gaaf:
> Add CRYPTO_CXXFLAGS to unittest_formatter_CXXFLAGS to find pk11pub.h to
> be included in src/common/ceph_crypto.h.
> 
> Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
> ---
>  src/Makefile.am | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/Makefile.am b/src/Makefile.am
> index be851d6..2f6f486 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -764,7 +764,7 @@ check_PROGRAMS += unittest_heartbeatmap
>  unittest_formatter_SOURCES = test/formatter.cc rgw/rgw_formats.cc
>  unittest_formatter_LDFLAGS = $(PTHREAD_CFLAGS) ${AM_LDFLAGS}
>  unittest_formatter_LDADD = ${UNITTEST_LDADD} $(LIBGLOBAL_LDA)
> -unittest_formatter_CXXFLAGS = ${AM_CXXFLAGS} ${UNITTEST_CXXFLAGS}
> +unittest_formatter_CXXFLAGS = ${CRYPTO_CXXFLAGS} ${AM_CXXFLAGS} ${UNITTEST_CXXFLAGS}
>  check_PROGRAMS += unittest_formatter
>  
>  unittest_libcephfs_config_SOURCES = test/libcephfs_config.cc
> 


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] fix build of unittest_formatter
  2012-12-11 12:08 ` Danny Al-Gaaf
@ 2012-12-11 16:44   ` Sage Weil
  0 siblings, 0 replies; 3+ messages in thread
From: Sage Weil @ 2012-12-11 16:44 UTC (permalink / raw)
  To: Danny Al-Gaaf; +Cc: ceph-devel

I missed it before; applied.

Thanks!
sage


On Tue, 11 Dec 2012, Danny Al-Gaaf wrote:

> Hi,
> 
> did you have a chance to take a look at this patch?
> 
> Danny
> 
> Am 28.11.2012 16:05, schrieb Danny Al-Gaaf:
> > Add CRYPTO_CXXFLAGS to unittest_formatter_CXXFLAGS to find pk11pub.h to
> > be included in src/common/ceph_crypto.h.
> > 
> > Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
> > ---
> >  src/Makefile.am | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/Makefile.am b/src/Makefile.am
> > index be851d6..2f6f486 100644
> > --- a/src/Makefile.am
> > +++ b/src/Makefile.am
> > @@ -764,7 +764,7 @@ check_PROGRAMS += unittest_heartbeatmap
> >  unittest_formatter_SOURCES = test/formatter.cc rgw/rgw_formats.cc
> >  unittest_formatter_LDFLAGS = $(PTHREAD_CFLAGS) ${AM_LDFLAGS}
> >  unittest_formatter_LDADD = ${UNITTEST_LDADD} $(LIBGLOBAL_LDA)
> > -unittest_formatter_CXXFLAGS = ${AM_CXXFLAGS} ${UNITTEST_CXXFLAGS}
> > +unittest_formatter_CXXFLAGS = ${CRYPTO_CXXFLAGS} ${AM_CXXFLAGS} ${UNITTEST_CXXFLAGS}
> >  check_PROGRAMS += unittest_formatter
> >  
> >  unittest_libcephfs_config_SOURCES = test/libcephfs_config.cc
> > 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2012-12-11 16:44 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-28 15:05 [PATCH] fix build of unittest_formatter Danny Al-Gaaf
2012-12-11 12:08 ` Danny Al-Gaaf
2012-12-11 16:44   ` Sage Weil

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.