[PATCH 1/5] go: reorganize the go bindings

Justus Winter 4winter at informatik.uni-hamburg.de
Sat May 5 04:15:35 PDT 2012


Quoting Austin Clements (2012-05-04 20:43:20)
> Quoth myself on May 04 at  2:38 pm:
> > Quoth Justus Winter on Apr 30 at  9:55 pm:
> > > Signed-off-by: Justus Winter <4winter at informatik.uni-hamburg.de>
> > 
> > Whole series LGTM (and yay gofmt!).  I have only two minor comments.
> > 
> > It would be nice to have a little more log message for some of these.
> > Mostly the first one could use some explanation of why you're
> > rearranging the source tree and a mention that there are no changes to
> > the files (since this is hard to get from the diff).

Good point.

> > It would also be nice if this could optionally be built using the
> > libnotmuch and notmuch.h directly from the source tree containing the
> > Go bindings, rather than requiring them to be installed system-wide.
> > I don't know Go 1's build system well enough to know if there's an
> > easy way to do this.

Yes, I've been pondering that myself. There are CGO_CFLAGS and
CGO_LDFLAGS that could be used to point the c compiler to the right
paths. I'll experiment with this and propose a patch.

> Oh, also a news patch would be good, though that can obviously be
> separate.

Sure.

I'll send an updated patch series. Thanks for the review.

Justus


More information about the notmuch mailing list