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

Austin Clements amdragon at MIT.EDU
Fri May 4 11:38:22 PDT 2012


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).

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.


More information about the notmuch mailing list