Re: [notmuch] [PATCH] Build and link against notmuch shared library

2010-01-24 Thread Scott Robinson
Excerpts from bgamari.foss's message of Sat Jan 23 12:58:42 -0600 2010: > True, but I don't think that this means that we need to link the > executable with a C++ compiler. I've tried linking with CC and it seems > to succeed, so I don't think there should be a problem changing it. > Are you usin

Re: [notmuch] [PATCH] Build and link against notmuch shared library

2010-01-23 Thread bgamari . foss
Excerpts from Ingmar Vanhassel's message of Fri Jan 22 21:06:00 -0500 2010: > On Sat, 23 Jan 2010 02:58:53 +0200, Felipe Contreras > wrote: > > Does it need to be CXX? Why not CC instead? > > Xapian and the notmuch wrapper 'parts' are written in C++. > True, but I don't think that this means th

Re: [notmuch] [PATCH] Build and link against notmuch shared library

2010-01-23 Thread Ben Gamari
Excerpts from Felipe Contreras's message of Fri Jan 22 19:58:53 -0500 2010: > Does it need to be CXX? Why not CC instead? > Nope. It's been changed to CC. Thanks! - Ben ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/list

Re: [notmuch] [PATCH] Build and link against notmuch shared library

2010-01-22 Thread Ingmar Vanhassel
On Sat, 23 Jan 2010 02:58:53 +0200, Felipe Contreras wrote: > On Wed, Jan 20, 2010 at 10:35 PM, Ben Gamari wrote: > >  libnotmuch_modules = $(libnotmuch_c_srcs:.c=.o) > > $(libnotmuch_cxx_srcs:.cc=.o) > > -$(dir)/notmuch.a: $(libnotmuch_modules) > > -       $(call quiet,AR) rcs $@ $^ > > +$

Re: [notmuch] [PATCH] Build and link against notmuch shared library

2010-01-22 Thread Felipe Contreras
On Wed, Jan 20, 2010 at 10:35 PM, Ben Gamari wrote: >  libnotmuch_modules = $(libnotmuch_c_srcs:.c=.o) $(libnotmuch_cxx_srcs:.cc=.o) > -$(dir)/notmuch.a: $(libnotmuch_modules) > -       $(call quiet,AR) rcs $@ $^ > +$(dir)/$(SONAME): $(libnotmuch_modules) > +       $(call quiet,CXX,$(LDFLAGS)) $^

Re: [notmuch] [PATCH] Build and link against notmuch shared library

2010-01-20 Thread Mike Hommey
On Wed, Jan 20, 2010 at 03:07:27PM -0500, Ben Gamari wrote: > + install lib/libnotmuch.so $(DESTDIR)$(prefix)/lib/ > +$(dir)/libnotmuch.so: $(libnotmuch_modules) > + $(call quiet,CXX,$(LDFLAGS)) $^ $(FINAL_LDFLAGS) -shared -o $@ If you're going to install that in $(prefix)/lib, you'd bett