[PATCH v4 0/6] Config-related patches
Mark Walters
markwalters1009 at gmail.com
Wed Apr 25 01:06:15 PDT 2012
On Sat, 14 Apr 2012, Mark Walters <markwalters1009 at gmail.com> wrote:
> On Sat, 14 Apr 2012, Peter Wang <novalazy at gmail.com> wrote:
>> Changes from v3:
>> - rephrase part of the 'list' implementation as a separate patch
>> - test 'set' on an extant key
>> - test removing keys
>
> This looks good to me. +1
>
> Two minor comments which you might like to consider (but definitely are
> not required):
> 1) You could check that there are no further arguments when the user
> calls `notmuch config list'
> 2) In the man page you could explicitly say what the output is for a
> configuration item which has not been set.
>
> Best wishes
>
> Mark
Just to make it completely clear the above was not meant to delay
acceptance of this series: I am quite happy with it as is.
Best wishes
Mark
More information about the notmuch
mailing list