Re: [PATCH v2] lib: make notmuch shared library install_name be full path on Mac OS X

2015-03-06 Thread J. Lewis Muir
On 3/6/15 1:05 AM, David Bremner wrote: J. Lewis Muir jlm...@imca-cat.org writes: The install_name of libnotmuch.dylib on Mac OS X is what is written into a program that links against it. If it is just the name of the shared library file, as opposed to the full path, the program won't be

[Patch v2 2/4] build/ruby: make use of -Wl,--no-undefined configurable

2015-03-06 Thread David Bremner
In particular this is supposed to help build on systems (presumably using a non-gnu ld) where this flag is not available. --- bindings/Makefile.local | 4 +++- bindings/ruby/extconf.rb | 5 +++-- configure| 13 + 3 files changed, 19 insertions(+), 3 deletions(-)

[Patch v2 4/4] test: add initial ruby tests

2015-03-06 Thread David Bremner
This is pretty much a line by line translation of the existing python tests, with two new tests for the count API. --- test/T395-ruby.sh | 86 +++ test/test-lib.sh | 5 2 files changed, 91 insertions(+) create mode 100755

integrate ruby into build and test v2

2015-03-06 Thread David Bremner
This is just a rebase against current master, with one typo fix in bindings/ruby/README ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

[Patch v2 3/4] build/ruby: use notmuch configure script values for shared lib

2015-03-06 Thread David Bremner
This is supposed to help build on systems like MacOS with different conventions for naming shared libraries. We have already computed the relevant names, so doing it again in ruby seems like a bad idea. --- bindings/Makefile.local | 1 + bindings/ruby/extconf.rb | 13 +++-- 2 files

[PATCH] python: replace hardcoding of SONAME version

2015-03-06 Thread David Bremner
Failing to update this string in globals.py causes failures when the SONAME changes. In order to hopefully reduce the number of such errors, automate the process of setting the SONAME in the python bindings. --- Makefile.local | 4 +++- bindings/python/notmuch/globals.py | 5

[Patch v2 1/4] build: integrate building ruby bindings into notmuch build process

2015-03-06 Thread David Bremner
Because ruby generates a Makefile, we have to use recursive make. Because mkmf.rb hardcodes the name Makefile, put our Makefile{.local} in the parent directory. --- Makefile.local | 2 +- bindings/Makefile | 7 +++ bindings/Makefile.local | 18 ++

[PATCH v2] lib: make notmuch shared library install_name be full path on Mac OS X

2015-03-06 Thread David Bremner
"J. Lewis Muir" writes: > The install_name of libnotmuch.dylib on Mac OS X is what is written > into a program that links against it. If it is just the name of the > shared library file, as opposed to the full path, the program won't be > able to find it when it runs and will abort. Instead,

[PATCH] cli: add a tool for starting new message in the emacs ui

2015-03-06 Thread David Bremner
David Bremner writes: > From: Jani Nikula > > Add a tool to start composing an email in the Notmuch Emacs UI with > the specified subject, recipients, and message body. Both Jani and Tomi expressed, if not satisfaction, at least exhaustion with this version, so I pushed to master. d

[PATCH] completion: update list of commands in zsh completion.

2015-03-06 Thread David Bremner
David Bremner writes: > This is actually generated by the script in > > id:1420289900-29717-2-git-send-email-david at tethera.net > pushed this one, and marked the otherone as "wontfix", on a cost/benefit basis. Hopefully the author is not too angry with me. d

[PATCH] go: add binding for notmuch_message_get_date

2015-03-06 Thread David Bremner
Trevor Jim writes: > I'm not sure what the "current troubles" are (happy to hear more), but > the way I've handled the nil case is in keeping with the way it's > handled in the other function bindings for go. Adding an error return > to all relevant functions seems like a big change that

[PATCH v2] lib: make notmuch shared library install_name be full path on Mac OS X

2015-03-06 Thread J. Lewis Muir
On 3/6/15 1:05 AM, David Bremner wrote: > "J. Lewis Muir" writes: > >> The install_name of libnotmuch.dylib on Mac OS X is what is written >> into a program that links against it. If it is just the name of >> the shared library file, as opposed to the full path, the program >> won't be able to

integrate ruby into build and test v2

2015-03-06 Thread David Bremner
This is just a rebase against current master, with one typo fix in bindings/ruby/README

[Patch v2 4/4] test: add initial ruby tests

2015-03-06 Thread David Bremner
This is pretty much a line by line translation of the existing python tests, with two new tests for the count API. --- test/T395-ruby.sh | 86 +++ test/test-lib.sh | 5 2 files changed, 91 insertions(+) create mode 100755

[Patch v2 1/4] build: integrate building ruby bindings into notmuch build process

2015-03-06 Thread David Bremner
Because ruby generates a Makefile, we have to use recursive make. Because mkmf.rb hardcodes the name Makefile, put our Makefile{.local} in the parent directory. --- Makefile.local | 2 +- bindings/Makefile | 7 +++ bindings/Makefile.local | 18 ++

[Patch v2 2/4] build/ruby: make use of -Wl,--no-undefined configurable

2015-03-06 Thread David Bremner
In particular this is supposed to help build on systems (presumably using a non-gnu ld) where this flag is not available. --- bindings/Makefile.local | 4 +++- bindings/ruby/extconf.rb | 5 +++-- configure| 13 + 3 files changed, 19 insertions(+), 3 deletions(-)

[Patch v2 3/4] build/ruby: use notmuch configure script values for shared lib

2015-03-06 Thread David Bremner
This is supposed to help build on systems like MacOS with different conventions for naming shared libraries. We have already computed the relevant names, so doing it again in ruby seems like a bad idea. --- bindings/Makefile.local | 1 + bindings/ruby/extconf.rb | 13 +++-- 2 files