This controls where comments and other text wraps. 70 is the default
value, so this simply returns it to the default for people who have
overridden it. Most notmuch code already adheres to this.
---
SO here is the patch (still sorry if I did it wrongly and very badly).
.dir-locals.el | 19 ++
Sorry if it is not correct to act like this (did I ever tell how n00b I
am when it comes to git ?).
So I wanted to /amend/ your patch to add some refactoring to it.
I hope this will not hurt :/
/Xavier
On Fri, 13 Jan 2012 11:50:52 +0200, Tomi Ollila wrote:
> On Fri, 13 Jan 2012 08:22:49 +, David Edmondson wrote:
> > On Thu, 12 Jan 2012 23:07:03 +0100, Xavier Maillard
> > wrote:
> > > On Wed, 11 Jan 2012 17:13:35 +0200, Tomi Ollila
> > > wrote:
> > > > Executed
> > > > for f in emacs
---
notmuch-show.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/notmuch-show.c b/notmuch-show.c
index 87a1c90..d14dac9 100644
--- a/notmuch-show.c
+++ b/notmuch-show.c
@@ -677,7 +677,7 @@ format_part_content_json (GMimeObject *part)
if (g_mime_content_type_is_typ
Hi David,
Have you planned to cook something about this in notmuch wiki ?
That sounds like something I could use to test patches more easily than
I do currently.
Note: I am totally a dummy when I have to use git :/
Regards
/Xavier
On Thu, 12 Jan 2012 18:17:23 -0500, Austin Clements wrote:
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
> ---
pushed.
d
On Fri, 13 Jan 2012 09:44:46 +, David Edmondson wrote:
> Add the charset of text/html parts to the JSON output of 'notmuch
> -show' when it is known. Observe the encoding when rendering such
> parts in emacs.
> ---
pushed
On Fri, 13 Jan 2012 18:07:02 -0500, Austin Clements wrote:
> ---
> notmuch-count.c | 53 +
> 1 files changed, 25 insertions(+), 28 deletions(-)
This seems independent of the rest of the series, and kindof obvious by
this point, so I pushed it
On Fri, 13 Jan 2012 12:52:48 -0800, Jameson Graef Rollins wrote:
>
> Doesn't everything need to be merged into master eventually anyway? It
> seems to me that unless it's a change that very narrowly targeting an
> issue in a release branch that is not an issue in master, every patch
> will ultim
On Fri, 13 Jan 2012 23:27:11 +0100, Xavier Maillard wrote:
>
> What about doing all of this directly in .dir-locals.el (could be
> considered unsafe though to add an EVAL block).
>
I'm not crazy about this ideas, it sounds like it has the potential to
mix whitespace changes in with out changes
ng, and tagging
large collections of email messages in maildir or mh format. It uses
the Xapian library to provide fast, full-text search with a convenient
search syntax.
For more about notmuch, see http://notmuchmail.org
-- next part --
A non-text attachment was scrubbed
---
notmuch-show.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/notmuch-show.c b/notmuch-show.c
index 87a1c90..d14dac9 100644
--- a/notmuch-show.c
+++ b/notmuch-show.c
@@ -677,7 +677,7 @@ format_part_content_json (GMimeObject *part)
if (g_mime_content_type_is_ty
Quoth David Edmondson on Jan 13 at 9:44 am:
> Add the charset of text/html parts to the JSON output of 'notmuch
> -show' when it is known. Observe the encoding when rendering such
> parts in emacs.
> ---
>
> Commentary added.
>
> emacs/notmuch-show.el |3 ++-
> notmuch-show.c| 28
This adds a "search" section to the config file and an
"auto_tag_exclusions" setting in that section. The search and count
commands pass tag tags from the configuration to the library.
---
notmuch-client.h |8
notmuch-config.c | 42 ++
notmuc
This is useful for tags like "deleted" and "spam" that people
generally want to exclude from query results. These exclusions will
be overridden if a tag is explicitly mentioned in a query.
---
lib/notmuch.h |6 ++
lib/query.cc | 35 +++
2 files changed,
---
notmuch-count.c | 53 +
1 files changed, 25 insertions(+), 28 deletions(-)
diff --git a/notmuch-count.c b/notmuch-count.c
index 20ce334..0982f99 100644
--- a/notmuch-count.c
+++ b/notmuch-count.c
@@ -21,6 +21,11 @@
#include "notmuch-clie
This addresses Jani's comments and improves some of the text and code
comments.
Quoth Xavier Maillard on Jan 13 at 11:42 pm:
>
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
> ---
> SO here is the patch (still sorry if I d
On Thu, 12 Jan 2012 18:17:23 -0500, Austin Clements wrote:
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
> ---
pushed.
d
__
On Fri, 13 Jan 2012 09:44:46 +, David Edmondson wrote:
> Add the charset of text/html parts to the JSON output of 'notmuch
> -show' when it is known. Observe the encoding when rendering such
> parts in emacs.
> ---
pushed
___
notmuch mailing list
no
On Fri, 13 Jan 2012 18:07:02 -0500, Austin Clements wrote:
> ---
> notmuch-count.c | 53 +
> 1 files changed, 25 insertions(+), 28 deletions(-)
This seems independent of the rest of the series, and kindof obvious by
this point, so I pushed it
On Fri, 13 Jan 2012 12:52:48 -0800, Jameson Graef Rollins
wrote:
>
> Doesn't everything need to be merged into master eventually anyway? It
> seems to me that unless it's a change that very narrowly targeting an
> issue in a release branch that is not an issue in master, every patch
> will ulti
On Fri, 13 Jan 2012 23:27:11 +0100, Xavier Maillard wrote:
>
> What about doing all of this directly in .dir-locals.el (could be
> considered unsafe though to add an EVAL block).
>
I'm not crazy about this ideas, it sounds like it has the potential to
mix whitespace changes in with out changes
Where to obtain notmuch 0.11
===
http://notmuchmail.org/releases/notmuch-0.11.tar.gz
Which can be verified with:
http://notmuchmail.org/releases/notmuch-0.11.tar.gz.sha1
11eb1d967af089ed36f6816f61ebae308bc19339 notmuch-0.11.tar.gz
http://notmuchmail.org/releases
sired result, and notmuch.el is full of it...
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/96186e0c/attachment.pgp>
up has some explanatory text beside it. Did you omit
> that on purpose?
That should be a requirement, I think.
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/ebde048e/attachment.pgp>
ype: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/b825295a/attachment.pgp>
Quoth David Edmondson on Jan 13 at 9:44 am:
> Add the charset of text/html parts to the JSON output of 'notmuch
> -show' when it is known. Observe the encoding when rendering such
> parts in emacs.
> ---
>
> Commentary added.
>
> emacs/notmuch-show.el |3 ++-
> notmuch-show.c| 28
This adds a "search" section to the config file and an
"auto_tag_exclusions" setting in that section. The search and count
commands pass tag tags from the configuration to the library.
---
notmuch-client.h |8
notmuch-config.c | 42 ++
notmuc
This is useful for tags like "deleted" and "spam" that people
generally want to exclude from query results. These exclusions will
be overridden if a tag is explicitly mentioned in a query.
---
lib/notmuch.h |6 ++
lib/query.cc | 35 +++
2 files changed,
---
notmuch-count.c | 53 +
1 files changed, 25 insertions(+), 28 deletions(-)
diff --git a/notmuch-count.c b/notmuch-count.c
index 20ce334..0982f99 100644
--- a/notmuch-count.c
+++ b/notmuch-count.c
@@ -21,6 +21,11 @@
#include "notmuch-cli
This addresses Jani's comments and improves some of the text and code
comments.
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch
Quoth Xavier Maillard on Jan 13 at 11:42 pm:
>
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
> ---
> SO here is the patch (still sorry if I d
On Fri, 13 Jan 2012 13:42:55 +0100, Xavier Maillard
wrote:
>
> On Fri, 13 Jan 2012 08:23:55 +, David Edmondson wrote:
> > On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet
> > wrote:
> > > Less code, same results, without sacrificing readability.
> >
> > +1, but why not replace non-branch
This controls where comments and other text wraps. 70 is the default
value, so this simply returns it to the default for people who have
overridden it. Most notmuch code already adheres to this.
---
SO here is the patch (still sorry if I did it wrongly and very badly).
.dir-locals.el | 19 ++
Sorry if it is not correct to act like this (did I ever tell how n00b I
am when it comes to git ?).
So I wanted to /amend/ your patch to add some refactoring to it.
I hope this will not hurt :/
/Xavier
___
notmuch mailing list
notmuch@notmuchmail.org
LGTM.
Quoth Tomi Ollila on Jan 13 at 10:17 am:
> emacsclient --eval '(kill-emacs)' makes emacs versions 23.1
> and 23.2 ask user input from running emacs. Redefining
> yes-or-no-p function when kill-emacs is executed for these
> emacs versions in test-lib.el avoids this test problem.
> ---
> Thank
On Fri, 13 Jan 2012 11:50:52 +0200, Tomi Ollila wrote:
> On Fri, 13 Jan 2012 08:22:49 +, David Edmondson wrote:
> > On Thu, 12 Jan 2012 23:07:03 +0100, Xavier Maillard
> > wrote:
> > > On Wed, 11 Jan 2012 17:13:35 +0200, Tomi Ollila
> > > wrote:
> > > > Executed
> > > > for f in emacs
On Fri, 13 Jan 2012 08:23:55 +, David Edmondson wrote:
> On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet wrote:
> > Less code, same results, without sacrificing readability.
>
> +1, but why not replace non-branching `if' with `when' as well?
I tend to use WHEN for case I need to execute m
On Thu, 12 Jan 2012 18:17:23 -0500, Austin Clements wrote:
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
+1
/Xavier
If `notmuch-dev-show-review-patch' is called with a prefix argument,
pull updates for the 'master' branch of the temporary repository.
---
emacs/notmuch-dev.el |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/emacs/notmuch-dev.el b/emacs/notmuch-dev.el
index 465846f.
Remove {, } and ,.
---
emacs/notmuch-dev.el |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/emacs/notmuch-dev.el b/emacs/notmuch-dev.el
index 871ce3b..465846f 100644
--- a/emacs/notmuch-dev.el
+++ b/emacs/notmuch-dev.el
@@ -75,7 +75,7 @@
(defun notmuch-dev-flatten-title
pe: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/37c845e4/attachment.pgp>
On Thu, 12 Jan 2012 18:12:16 +0100, Pieter Praet wrote:
> To allow for expansion whilst keeping everything tidy and organized,
> move all defcustom/defface variables to the following subgroups,
> defined in notmuch-lib.el:
Baring the issue that the Davids brought up about the sub group
descriptio
sub group
descriptions, I think this would be a nice cleanup. +1.
jamie.
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120
On Fri, 13 Jan 2012 05:05:35 -0400, David Bremner wrote:
> I thought about this a bit more, and I agree that at least the release
> candidates (basically anything tagged on branch release) ought to be
> merged back to master. Since any series of bugfix patches seems to be
> cause for a new release
pe: application/pgp-signature
Size: 835 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/e293101b/attachment.pgp>
Hi David,
Have you planned to cook something about this in notmuch wiki ?
That sounds like something I could use to test patches more easily than
I do currently.
Note: I am totally a dummy when I have to use git :/
Regards
/Xavier
___
notmuch mailing
On Fri, 13 Jan 2012 09:44:46 +, David Edmondson wrote:
> Add the charset of text/html parts to the JSON output of 'notmuch
> -show' when it is known. Observe the encoding when rendering such
> parts in emacs.
> ---
>
> Commentary added.
+1
Tomi
On Fri, 13 Jan 2012 08:22:49 +, David Edmondson wrote:
> On Thu, 12 Jan 2012 23:07:03 +0100, Xavier Maillard
> wrote:
> > On Wed, 11 Jan 2012 17:13:35 +0200, Tomi Ollila
> > wrote:
> > > Executed
> > > for f in emacs/*.el
> > > do emacs -q --batch --eval \
> > > "(progn (fi
LGTM.
Quoth Tomi Ollila on Jan 13 at 10:17 am:
> emacsclient --eval '(kill-emacs)' makes emacs versions 23.1
> and 23.2 ask user input from running emacs. Redefining
> yes-or-no-p function when kill-emacs is executed for these
> emacs versions in test-lib.el avoids this test problem.
> ---
> Thank
`magit-delete-branch' runs "git branch -d ", which refuses to
delete branches that are not fully merged. Given that we don't care
and don't want to be stalled, use "git branch -D " directly.
---
This should address the problems removing un-merged branches.
emacs/notmuch-dev.el |4 +++-
1 fil
emacsclient --eval '(kill-emacs)' makes emacs versions 23.1
and 23.2 ask user input from running emacs. Redefining
yes-or-no-p function when kill-emacs is executed for these
emacs versions in test-lib.el avoids this test problem.
---
Thanks Austin for your comments. Including 'activate' into defadv
next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/7e11ae22/attachment.pgp>
Add the charset of text/html parts to the JSON output of 'notmuch
-show' when it is known. Observe the encoding when rendering such
parts in emacs.
---
Commentary added.
emacs/notmuch-show.el |3 ++-
notmuch-show.c| 28 +++-
2 files changed, 25 insertions(+)
On Thu, 12 Jan 2012 18:17:23 -0500, Austin Clements wrote:
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
> ---
+1
Tomi
> .dir-locals.el |
On Fri, 13 Jan 2012 10:25:29 +, David Edmondson wrote:
> `magit-delete-branch' runs "git branch -d ", which refuses to
> delete branches that are not fully merged. Given that we don't care
> and don't want to be stalled, use "git branch -D " directly.
> ---
Yes, that seems to fix it.
Two mo
: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120113/5c61402f/attachment-0001.pgp>
d note something about this as a /coding style/ rule ?
How about "Running `indent-region' over the file should produce no
changes" as well?
-- next part --
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Siz
On Fri, 13 Jan 2012 11:50:52 +0200, Tomi Ollila wrote:
> So, In addition to 'uncrustify.cfg' we need emacs reformatter along lines:
>
> (defun reformat-elisp-buffer ()
> (interactice)
> (emacs-lisp-mode)
> (whitespace-cleanup)
> (indent-region (point-min) (point-max)))
>
> And this locat
On Thu, 12 Jan 2012 23:31:34 -0400, David Bremner wrote:
> On Thu, 12 Jan 2012 18:12:16 +0100, Pieter Praet wrote:
> > To allow for expansion whilst keeping everything tidy and organized,
> > move all defcustom/defface variables to the following subgroups,
> > defined in notmuch-lib.el:
> >
> >
On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet wrote:
> Less code, same results, without sacrificing readability.
Does this change correctly re-indent the line following the if/unless?
pgppn8eS1pTGG.pgp
Description: PGP signature
___
notmuch mailing
If `notmuch-dev-show-review-patch' is called with a prefix argument,
pull updates for the 'master' branch of the temporary repository.
---
emacs/notmuch-dev.el |8 +++-
1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/emacs/notmuch-dev.el b/emacs/notmuch-dev.el
index 465846f.
Remove {, } and ,.
---
emacs/notmuch-dev.el |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/emacs/notmuch-dev.el b/emacs/notmuch-dev.el
index 871ce3b..465846f 100644
--- a/emacs/notmuch-dev.el
+++ b/emacs/notmuch-dev.el
@@ -75,7 +75,7 @@
(defun notmuch-dev-flatten-title
On Thu, 12 Jan 2012 23:46:46 -0400, David Bremner wrote:
> On Thu, 12 Jan 2012 18:25:38 +0100, Pieter Praet wrote:
> > On Sat, 31 Dec 2011 23:22:46 -0400, David Bremner
> > wrote:
>
> > with differing hashes), this has the potential of causing confusion
> > and/or quite some extra work when de
On Fri, 13 Jan 2012 13:42:55 +0100, Xavier Maillard wrote:
> On Fri, 13 Jan 2012 08:23:55 +, David Edmondson wrote:
> > On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet wrote:
> > > Less code, same results, without sacrificing readability.
> >
> > +1, but why not replace non-branching `if'
On Fri, 13 Jan 2012 13:42:55 +0100, Xavier Maillard wrote:
>
> On Fri, 13 Jan 2012 08:23:55 +, David Edmondson wrote:
> > On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet wrote:
> > > Less code, same results, without sacrificing readability.
> >
> > +1, but why not replace non-branching `i
On Fri, 13 Jan 2012 08:23:55 +, David Edmondson wrote:
> On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet wrote:
> > Less code, same results, without sacrificing readability.
>
> +1, but why not replace non-branching `if' with `when' as well?
I tend to use WHEN for case I need to execute m
On Fri, 13 Jan 2012 10:25:29 +, David Edmondson wrote:
> `magit-delete-branch' runs "git branch -d ", which refuses to
> delete branches that are not fully merged. Given that we don't care
> and don't want to be stalled, use "git branch -D " directly.
> ---
Yes, that seems to fix it.
Two mo
On Thu, 12 Jan 2012 18:17:23 -0500, Austin Clements wrote:
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
+1
/Xavier
__
`magit-delete-branch' runs "git branch -d ", which refuses to
delete branches that are not fully merged. Given that we don't care
and don't want to be stalled, use "git branch -D " directly.
---
This should address the problems removing un-merged branches.
emacs/notmuch-dev.el |4 +++-
1 fil
On Fri, 13 Jan 2012 09:44:46 +, David Edmondson wrote:
> Add the charset of text/html parts to the JSON output of 'notmuch
> -show' when it is known. Observe the encoding when rendering such
> parts in emacs.
> ---
>
> Commentary added.
+1
Tomi
__
On Fri, 13 Jan 2012 08:22:49 +, David Edmondson wrote:
> On Thu, 12 Jan 2012 23:07:03 +0100, Xavier Maillard
> wrote:
> > On Wed, 11 Jan 2012 17:13:35 +0200, Tomi Ollila wrote:
> > > Executed
> > > for f in emacs/*.el
> > > do emacs -q --batch --eval \
> > > "(progn (find-fi
On Thu, 12 Jan 2012 18:17:23 -0500, Austin Clements wrote:
> This controls where comments and other text wraps. 70 is the default
> value, so this simply returns it to the default for people who have
> overridden it. Most notmuch code already adheres to this.
Makes sense.
pgp63uDlQWlYX.pgp
De
Add the charset of text/html parts to the JSON output of 'notmuch
-show' when it is known. Observe the encoding when rendering such
parts in emacs.
---
Commentary added.
emacs/notmuch-show.el |3 ++-
notmuch-show.c| 28 +++-
2 files changed, 25 insertions(+)
On Thu, 12 Jan 2012 23:46:46 -0400, David Bremner wrote:
> On Thu, 12 Jan 2012 18:25:38 +0100, Pieter Praet wrote:
> > On Sat, 31 Dec 2011 23:22:46 -0400, David Bremner wrote:
>
> > with differing hashes), this has the potential of causing confusion
> > and/or quite some extra work when debuggi
On Thu, 12 Jan 2012 18:33:06 +0100, Pieter Praet wrote:
> Less code, same results, without sacrificing readability.
+1, but why not replace non-branching `if' with `when' as well?
pgpuhUlvebeLM.pgp
Description: PGP signature
___
notmuch mailing list
n
On Thu, 12 Jan 2012 23:07:03 +0100, Xavier Maillard wrote:
> On Wed, 11 Jan 2012 17:13:35 +0200, Tomi Ollila wrote:
> > Executed
> > for f in emacs/*.el
> > do emacs -q --batch --eval \
> > "(progn (find-file \"$f\") (whitespace-cleanup) (save-buffer))"
> > done
> >
> > which
emacsclient --eval '(kill-emacs)' makes emacs versions 23.1
and 23.2 ask user input from running emacs. Redefining
yes-or-no-p function when kill-emacs is executed for these
emacs versions in test-lib.el avoids this test problem.
---
Thanks Austin for your comments. Including 'activate' into defadv
79 matches
Mail list logo
