[PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

2013-04-07 Thread Karl Fogel
This patch fixes a trivial missing-paren problem in notmuch-address.el (and reindents the following defun accordingly). I'm not subscribed to this list, so please keep me CC'd on any followups. Best, -Karl From 4c74ad313f608f0834961c63c70d1f811ef103b7 Mon Sep 17 00:00:00 2001 From: Karl Fogel

Re: [PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

2013-04-07 Thread David Bremner
Karl Fogel kfo...@red-bean.com writes: This patch fixes a trivial missing-paren problem in notmuch-address.el (and reindents the following defun accordingly). I'm not subscribed to this list, so please keep me CC'd on any followups. Best, -Karl Dear Karl; Thanks very much for the patch.

[PATCH 2/3] cli: config: do not overwrite symlinks when saving config file

2013-04-07 Thread Jani Nikula
Use realpath to canonicalize the config path before writing. Previously 'notmuch setup' and 'notmuch config set' overwrote the config file even if it was a symbolic link. --- notmuch-config.c | 24 test/config |1 - 2 files changed, 20 insertions(+), 5

[PATCH 1/3] test: add some config file tests

2013-04-07 Thread Jani Nikula
Test the --config=FILE option, and add a broken test for writing config file through a symbolic link. --- test/config | 24 1 file changed, 24 insertions(+) diff --git a/test/config b/test/config index cfa1f32..344eced 100755 --- a/test/config +++ b/test/config @@

[PATCH 3/3] TODO: remove some completed todo items from the list

2013-04-07 Thread Jani Nikula
Bash completion has been rewritten. Configuration file saves follow symlinks. There is --config=FILE top level option to specify configuration file. --- devel/TODO | 14 -- 1 file changed, 14 deletions(-) diff --git a/devel/TODO b/devel/TODO index e4f36c2..f63385d 100644 ---

Re: [PATCH 0/3] config: follow symlinks when saving

2013-04-07 Thread Tomi Ollila
On Sun, Apr 07 2013, Jani Nikula j...@nikula.org wrote: This is v4 of [1], adding tests (and some semi-related TODO file updates). The actual code is the same as before. This makes my symlink based dotfiles setup happy. Looks good and PASS Read config file through a symlink PASS Write

Re: [PATCH] cli: mime node: fix compiler warning when building against gmime 2.4

2013-04-07 Thread Tomi Ollila
On Sat, Apr 06 2013, Jani Nikula j...@nikula.org wrote: commit d487ef9e58bcd193118f19f771d5ef3984616be5 Author: Jani Nikula j...@nikula.org Date: Sat Mar 30 15:53:16 2013 +0200 cli: mime node: abstract decryption and signature verification introduced a compiler warning, reported by

[PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

2013-04-07 Thread David Bremner
Karl Fogel writes: > This patch fixes a trivial missing-paren problem in notmuch-address.el > (and reindents the following defun accordingly). I'm not subscribed > to this list, so please keep me CC'd on any followups. > > Best, > -Karl Dear Karl; Thanks very much for the patch. Since the

[PATCH 0/3] config: follow symlinks when saving

2013-04-07 Thread Jani Nikula
This is v4 of [1], adding tests (and some semi-related TODO file updates). The actual code is the same as before. This makes my symlink based dotfiles setup happy. BR, Jani. [1] id:1362320310-10930-1-git-send-email-jani at nikula.org Jani Nikula (3): test: add some config file tests cli:

[PATCH 2/3] cli: config: do not overwrite symlinks when saving config file

2013-04-07 Thread Jani Nikula
Use realpath to canonicalize the config path before writing. Previously 'notmuch setup' and 'notmuch config set' overwrote the config file even if it was a symbolic link. --- notmuch-config.c | 24 test/config |1 - 2 files changed, 20 insertions(+), 5

[PATCH 1/3] test: add some config file tests

2013-04-07 Thread Jani Nikula
Test the --config=FILE option, and add a broken test for writing config file through a symbolic link. --- test/config | 24 1 file changed, 24 insertions(+) diff --git a/test/config b/test/config index cfa1f32..344eced 100755 --- a/test/config +++ b/test/config @@

[PATCH 3/3] TODO: remove some completed todo items from the list

2013-04-07 Thread Jani Nikula
Bash completion has been rewritten. Configuration file saves follow symlinks. There is --config=FILE top level option to specify configuration file. --- devel/TODO | 14 -- 1 file changed, 14 deletions(-) diff --git a/devel/TODO b/devel/TODO index e4f36c2..f63385d 100644 ---

[PATCH 0/3] config: follow symlinks when saving

2013-04-07 Thread Tomi Ollila
On Sun, Apr 07 2013, Jani Nikula wrote: > This is v4 of [1], adding tests (and some semi-related TODO file > updates). The actual code is the same as before. > > This makes my symlink based dotfiles setup happy. Looks good and PASS Read config file through a symlink PASS Write config file

[PATCH] cli: mime node: fix compiler warning when building against gmime 2.4

2013-04-07 Thread Tomi Ollila
On Sat, Apr 06 2013, Jani Nikula wrote: > commit d487ef9e58bcd193118f19f771d5ef3984616be5 > Author: Jani Nikula > Date: Sat Mar 30 15:53:16 2013 +0200 > > cli: mime node: abstract decryption and signature verification > > introduced a compiler warning, reported by Mark Walters, when

[PATCH] emacs: add missing paren to fix defun in notmuch-address.el.

2013-04-07 Thread Karl Fogel
David Bremner writes: >Karl Fogel writes: >> This patch fixes a trivial missing-paren problem in notmuch-address.el >> (and reindents the following defun accordingly). I'm not subscribed >> to this list, so please keep me CC'd on any followups. > >Dear Karl; > >Thanks very much for the patch. >