On Tue, 13 Dec 2011 15:56:47 -0400, David Bremner wrote:
> From: David Bremner
>
> The idea is that $test_count could be used in tests to label
> intermediate files. The output enabled by this patch (and --debug)
> helps figure out which OUTPUT.nn file belongs to which test in case
> several sub
On Tue, 13 Dec 2011 20:18:48 +0200, tomi.ollila at iki.fi wrote:
> From: Tomi Ollila
>
> In my test case added g_object_unref(charset_filter) reduces memory
> consumption over 90% when 'notmuch show --format=text "*"' is
> executed (~11000 messages, RES ~330M -> ~25M).
> ---
> Thanks Dmitry. I di
On Tue, 13 Dec 2011 18:32:12 +0100, Thomas Jost
wrote:
> This hook is called every time the notmuch-hello buffer is updated.
> ---
> emacs/notmuch-hello.el |9 -
> 1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el
> ind
Hi Daniel.
I have finished reviewing this patch at last. Sorry, it is a bit messy.
Overall, I like the patch. It is a very nice improvement.
I am sure I have missed some important points, but I guess this is the
best I can do right now. Perhaps I will find more comments for the next
version of
On Tue, 13 Dec 2011 14:03:33 +0200, Tomi Ollila wrote:
> In my use case g_object_unref(charset_filter) reduces memory
> consumption over 90% when 'notmuch show --format=text "*"' is
> executed (~11000 messages, RES ~330M -> ~25M).
> ---
> notmuch-show.c |4 +++-
> 1 files changed, 3 insertion
On Tue, 13 Dec 2011 15:35:53 -0800, Jameson Graef Rollins
wrote:
> On Wed, 14 Dec 2011 03:24:23 +0400, Dmitry Kurochkin
> wrote:
> > BTW I have some plans to introduce optional explicit test ids that can
> > be used for inter-test dependencies. E.g.:
> >
> > t
On Tue, 13 Dec 2011 19:18:16 -0400, David Bremner wrote:
> On Tue, 13 Dec 2011 14:22:21 -0800, Jameson Graef Rollins
> wrote:
>
> > I've only been vaguely following this "test count" stuff, but I'm not
> > sure I understand what's the point of giving tests a number that is
> > ultimately mutabl
On Tue, 13 Dec 2011 22:55:18 +0200, Tomi Ollila wrote:
> On Wed, 14 Dec 2011 00:15:43 +0400, Dmitry Kurochkin
> wrote:
> > On Tue, 13 Dec 2011 15:56:47 -0400, David Bremner wrote:
> > > From: David Bremner
> > >
> > > The idea is that $tes
On Tue, 13 Dec 2011 15:56:47 -0400, David Bremner wrote:
> From: David Bremner
>
> The idea is that $test_count could be used in tests to label
> intermediate files. The output enabled by this patch (and --debug)
> helps figure out which OUTPUT.nn file belongs to which test in case
> several sub
On Tue, 13 Dec 2011 20:18:48 +0200, tomi.oll...@iki.fi wrote:
> From: Tomi Ollila
>
> In my test case added g_object_unref(charset_filter) reduces memory
> consumption over 90% when 'notmuch show --format=text "*"' is
> executed (~11000 messages, RES ~330M -> ~25M).
> ---
> Thanks Dmitry. I did n
On Tue, 13 Dec 2011 18:32:12 +0100, Thomas Jost wrote:
> This hook is called every time the notmuch-hello buffer is updated.
> ---
> emacs/notmuch-hello.el |9 -
> 1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el
> inde
On Tue, 13 Dec 2011 14:03:33 +0200, Tomi Ollila wrote:
> In my use case g_object_unref(charset_filter) reduces memory
> consumption over 90% when 'notmuch show --format=text "*"' is
> executed (~11000 messages, RES ~330M -> ~25M).
> ---
> notmuch-show.c |4 +++-
> 1 files changed, 3 insertion
On Mon, 12 Dec 2011 16:24:51 -0500, Austin Clements wrote:
> Quoth Jani Nikula on Dec 12 at 11:13 pm:
> > On Tue, 13 Dec 2011 00:53:05 +0400, Dmitry Kurochkin > gmail.com> wrote:
> > > On Mon, 12 Dec 2011 22:50:04 +0200, Jani Nikula
> > > wrote:
> > >
On Mon, 12 Dec 2011 22:50:04 +0200, Jani Nikula wrote:
> Support nil value for notmuch-poll-script to run "notmuch new" instead of
> an external script, and make this the new default. "notmuch new" is run
> using the configured notmuch-command.
>
> This allows taking better advantage of the "notm
Hi Jani.
On Mon, 12 Dec 2011 21:57:28 +0200, Jani Nikula wrote:
> Support nil value for notmuch-poll-script to run "notmuch new" instead of
> an external script, and make this the new default. "notmuch new" is run
> using the configured notmuch-command.
>
> This allows taking better advantage of
On Mon, 12 Dec 2011 12:15:44 +0200, Tomi Ollila wrote:
> On Sun, 11 Dec 2011 19:31:03 -0500, Austin Clements
> wrote:
> >
> > So here's another idea, prefaced with a rant.
> >
> > It's bothered me for a long time that notmuch-emacs didn't just know
> > by default how to check for new mail. Wh
On Mon, 12 Dec 2011 16:24:51 -0500, Austin Clements wrote:
> Quoth Jani Nikula on Dec 12 at 11:13 pm:
> > On Tue, 13 Dec 2011 00:53:05 +0400, Dmitry Kurochkin
> > wrote:
> > > On Mon, 12 Dec 2011 22:50:04 +0200, Jani Nikula wrote:
> > > > +If set to nil (th
On Mon, 12 Dec 2011 22:50:04 +0200, Jani Nikula wrote:
> Support nil value for notmuch-poll-script to run "notmuch new" instead of
> an external script, and make this the new default. "notmuch new" is run
> using the configured notmuch-command.
>
> This allows taking better advantage of the "notm
Hi Jani.
On Mon, 12 Dec 2011 21:57:28 +0200, Jani Nikula wrote:
> Support nil value for notmuch-poll-script to run "notmuch new" instead of
> an external script, and make this the new default. "notmuch new" is run
> using the configured notmuch-command.
>
> This allows taking better advantage of
On Sun, 11 Dec 2011 19:31:03 -0500, Austin Clements wrote:
> Quoth Jani Nikula on Dec 12 at 1:10 am:
> >On Dec 12, 2011 12:56 AM, "Austin Clements" <[1]amdragon at mit.edu>
> > wrote:
> >>
> >> Quoth Dmitry Ku
On Mon, 12 Dec 2011 00:19:36 +0200, Jani Nikula wrote:
>
> Hi Dmitry -
>
> On Mon, 12 Dec 2011 02:00:45 +0400, Dmitry Kurochkin gmail.com> wrote:
> > On Sun, 11 Dec 2011 23:48:20 +0200, Jani Nikula wrote:
> > > Let notmuch-poll-script be a function as
On Mon, 12 Dec 2011 12:15:44 +0200, Tomi Ollila wrote:
> On Sun, 11 Dec 2011 19:31:03 -0500, Austin Clements wrote:
> >
> > So here's another idea, prefaced with a rant.
> >
> > It's bothered me for a long time that notmuch-emacs didn't just know
> > by default how to check for new mail. What
Hi Jani.
On Sun, 11 Dec 2011 23:48:20 +0200, Jani Nikula wrote:
> Let notmuch-poll-script be a function as well as a string. Make default
> value nil instead of an empty string, but allow "" for backwards
> compatibility. Add a notmuch poll function to call "notmuch new" using the
> configured no
On Sun, 11 Dec 2011 12:19:44 -0400, David Bremner wrote:
> From: David Bremner
>
> The character set is chosen to be suitable for pathnames, and the same
> as that used by contrib/nmbug. The new encoded/decoded strings are
> allocated using talloc.
> ---
> This isn't urgent, but it is useful for
Hi David.
On Sun, 11 Dec 2011 12:07:51 -0400, David Bremner wrote:
> From: David Bremner
>
> Two new internal routines are created _config_get_list and
> _config_set_list; the notmuch_config_get_* functions that deal with
> lists are simply wrappers for these functions.
Looks good to me. Some
Hi Thomas.
On Wed, 7 Dec 2011 10:46:17 +0100, Thomas Jost
wrote:
> The new test_python() function makes writing Python tests a little easier:
> - it sets the environment variables as needed
> - it redirects stdout to the OUTPUT file (like test_emacs()).
>
> This commit also declares python as
On Sun, 11 Dec 2011 19:31:03 -0500, Austin Clements wrote:
> Quoth Jani Nikula on Dec 12 at 1:10 am:
> >On Dec 12, 2011 12:56 AM, "Austin Clements" <[1]amdra...@mit.edu> wrote:
> >>
> >> Quoth Dmitry Kurochkin on Dec 12 at 2:00 am:
>
Austin, good job! :) Thanks for this work. (and continue, please :))
Most comments in my review are minor and/or concern the old code
(i.e. the new code does not make it worse). Please feel free to ignore
them.
I vote for pushing this series as soon as Austin finds it appropriate.
If Austin mak
On Mon, 12 Dec 2011 00:19:36 +0200, Jani Nikula wrote:
>
> Hi Dmitry -
>
> On Mon, 12 Dec 2011 02:00:45 +0400, Dmitry Kurochkin
> wrote:
> > On Sun, 11 Dec 2011 23:48:20 +0200, Jani Nikula wrote:
> > > Let notmuch-poll-script be a function as well as a string
Hi Austin.
I enjoyed reviewing this patch. It is a pleasure to see how complex and
confusing code becomes much smaller and cleaner.
I still have some questions with the new code. It seems confusing to me
that part_content is called first and then go envelope headers. But I
this is just the fir
Hi Jani.
On Sun, 11 Dec 2011 23:48:20 +0200, Jani Nikula wrote:
> Let notmuch-poll-script be a function as well as a string. Make default
> value nil instead of an empty string, but allow "" for backwards
> compatibility. Add a notmuch poll function to call "notmuch new" using the
> configured no
On Sun, 11 Dec 2011 12:19:44 -0400, David Bremner wrote:
> From: David Bremner
>
> The character set is chosen to be suitable for pathnames, and the same
> as that used by contrib/nmbug. The new encoded/decoded strings are
> allocated using talloc.
> ---
> This isn't urgent, but it is useful for
Hi David.
On Sun, 11 Dec 2011 12:07:51 -0400, David Bremner wrote:
> From: David Bremner
>
> Two new internal routines are created _config_get_list and
> _config_set_list; the notmuch_config_get_* functions that deal with
> lists are simply wrappers for these functions.
Looks good to me. Some
Hi Thomas.
On Wed, 7 Dec 2011 10:46:17 +0100, Thomas Jost wrote:
> The new test_python() function makes writing Python tests a little easier:
> - it sets the environment variables as needed
> - it redirects stdout to the OUTPUT file (like test_emacs()).
>
> This commit also declares python as a
Austin, good job! :) Thanks for this work. (and continue, please :))
Most comments in my review are minor and/or concern the old code
(i.e. the new code does not make it worse). Please feel free to ignore
them.
I vote for pushing this series as soon as Austin finds it appropriate.
If Austin mak
Hi Austin.
I enjoyed reviewing this patch. It is a pleasure to see how complex and
confusing code becomes much smaller and cleaner.
I still have some questions with the new code. It seems confusing to me
that part_content is called first and then go envelope headers. But I
this is just the fir
On Fri, 9 Dec 2011 14:54:27 -0500, Austin Clements wrote:
> This function matches how we number parts for the --part argument to
> show. It will allow us to jump directly to the desired part, rather
> than traversing the entire tree and carefully tracking whether or not
> we're "in the zone".
>
On Fri, 9 Dec 2011 14:54:27 -0500, Austin Clements wrote:
> This function matches how we number parts for the --part argument to
> show. It will allow us to jump directly to the desired part, rather
> than traversing the entire tree and carefully tracking whether or not
> we're "in the zone".
>
Hi Austin.
+/* The number of children of this part. */
+int children;
Consider renaming to children_count or similar to make it clear that it
is a counter and not the actual children.
+notmuch_bool_t decrypt_success;
Perhaps s/decrypt_success/is_decrypted/ for consistency with
is_en
On Fri, 9 Dec 2011 14:54:26 -0500, Austin Clements wrote:
> Quoth Dmitry Kurochkin on Dec 09 at 11:05 pm:
> > On Sun, 4 Dec 2011 14:31:37 -0500, Austin Clements
> > wrote:
> > > }
> > >
> > > notmuch_status_t
> > > -show_message_b
On Sun, 4 Dec 2011 14:31:37 -0500, Austin Clements wrote:
> In addition to simplifying the code, we'll need the notmuch_message_t*
> in show_message_body shortly.
> ---
> notmuch-client.h |2 +-
> notmuch-reply.c |3 +--
> notmuch-show.c |3 +--
> show-message.c |3 ++-
> 4
Hi Austin.
On Fri, 9 Dec 2011 12:39:13 -0500, Austin Clements wrote:
> Just a reminder that this series awaits review and is the first step
> in a better, simpler, and cleaner show that will make possible things
> like improvements to the JSON format, better encoding handling, proper
> raw suppor
Hi Austin.
+/* The number of children of this part. */
+int children;
Consider renaming to children_count or similar to make it clear that it
is a counter and not the actual children.
+notmuch_bool_t decrypt_success;
Perhaps s/decrypt_success/is_decrypted/ for consistency with
is_en
On Fri, 9 Dec 2011 14:54:26 -0500, Austin Clements wrote:
> Quoth Dmitry Kurochkin on Dec 09 at 11:05 pm:
> > On Sun, 4 Dec 2011 14:31:37 -0500, Austin Clements
> > wrote:
> > > }
> > >
> > > notmuch_status_t
> > > -show_message_b
On Sun, 4 Dec 2011 14:31:37 -0500, Austin Clements wrote:
> In addition to simplifying the code, we'll need the notmuch_message_t*
> in show_message_body shortly.
> ---
> notmuch-client.h |2 +-
> notmuch-reply.c |3 +--
> notmuch-show.c |3 +--
> show-message.c |3 ++-
> 4
Hi Austin.
On Fri, 9 Dec 2011 12:39:13 -0500, Austin Clements wrote:
> Just a reminder that this series awaits review and is the first step
> in a better, simpler, and cleaner show that will make possible things
> like improvements to the JSON format, better encoding handling, proper
> raw suppor
On Thu, 08 Dec 2011 08:32:12 -0400, David Bremner wrote:
> On Thu, 08 Dec 2011 13:33:09 +0400, Dmitry Kurochkin gmail.com> wrote:
>
> > I am not sure recommending w3m-el is a good idea. It has very annoying
> > bugs with scrolling (C-n): at some point the cursor jumps ba
Hi David.
I am not sure recommending w3m-el is a good idea. It has very annoying
bugs with scrolling (C-n): at some point the cursor jumps back above,
i.e. it goes in a loop. I did not investigate the issue though.
Regards,
Dmitry
On Thu, 08 Dec 2011 08:32:12 -0400, David Bremner wrote:
> On Thu, 08 Dec 2011 13:33:09 +0400, Dmitry Kurochkin
> wrote:
>
> > I am not sure recommending w3m-el is a good idea. It has very annoying
> > bugs with scrolling (C-n): at some point the cursor jumps back above,
Hi David.
I am not sure recommending w3m-el is a good idea. It has very annoying
bugs with scrolling (C-n): at some point the cursor jumps back above,
i.e. it goes in a loop. I did not investigate the issue though.
Regards,
Dmitry
___
notmuch mailin
David, this seems ready for push as well.
Regards,
Dmitry
Hi David.
How about pushing this series? Just a humble ping :)
Regards,
Dmitry
David, this seems ready for push as well.
Regards,
Dmitry
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch
Hi David.
How about pushing this series? Just a humble ping :)
Regards,
Dmitry
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch
Change atomicity tests to use the new external binary dependencies.
This simplifies the code and makes output consistent.
---
test/atomicity | 12
1 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/test/atomicity b/test/atomicity
index ad7d4a3..6df0a00 100755
--- a/test/
Change atomicity tests to use the new external binary dependencies.
This simplifies the code and makes output consistent.
---
test/atomicity | 12
1 files changed, 4 insertions(+), 8 deletions(-)
diff --git a/test/atomicity b/test/atomicity
index ad7d4a3..6df0a00 100755
--- a/test/
+1
Regards,
Dmitry
Before the change, there was a workaround to avoid notmuch show calls
for parts with application/* Content-Type. But non-inlinable parts
are not limited to this Content-Type (e.g. mp3 files have audio/mpeg
Content-Type and are not inlinable). For such parts
`notmuch-show-insert-part-*/*' handler
The patch adds two new test cases:
* Do not call notmuch for non-inlinable application/mpeg parts
* Do not call notmuch for non-inlinable audio/mpeg parts
The application/mpeg test passes thanks to a workaround for
application/* Content-Types. The audio/mpeg is currently broken.
---
test/emacs
The patch adds two auxiliary functions and a variable:
notmuch_counter_reset
$notmuch_counter_command
notmuch_counter_value
They allow to count how many times notmuch binary is called.
notmuch_counter_reset() function generates a script that counts how
many times it is called and resets the
Changes:
v4 since v3:
* Use $(()) instead of expr(1) for math.
v3 since v2:
* Use read function instead of cat(1) to read counter value.
* Add a newline after the count value in state file and
notmuch_counter_value() output.
v2 since v1:
* Rename $notmuch_counter variable to $notmuch_counte
Hi Tomi.
On Tue, 29 Nov 2011 14:58:00 +0200, Tomi Ollila wrote:
> Hi Dmitry.
>
> On Tue, 29 Nov 2011 01:26:39 +0400, Dmitry Kurochkin gmail.com> wrote:
> > Hi Tomi.
> >
> > On Mon, 28 Nov 2011 22:42:50 +0200, Tomi Ollila
> > wrote:
> > > On Mon,
+1
Regards,
Dmitry
___
notmuch mailing list
notmuch@notmuchmail.org
http://notmuchmail.org/mailman/listinfo/notmuch
The patch adds two auxiliary functions and a variable:
notmuch_counter_reset
$notmuch_counter_command
notmuch_counter_value
They allow to count how many times notmuch binary is called.
notmuch_counter_reset() function generates a script that counts how
many times it is called and resets the
Before the change, there was a workaround to avoid notmuch show calls
for parts with application/* Content-Type. But non-inlinable parts
are not limited to this Content-Type (e.g. mp3 files have audio/mpeg
Content-Type and are not inlinable). For such parts
`notmuch-show-insert-part-*/*' handler
The patch adds two new test cases:
* Do not call notmuch for non-inlinable application/mpeg parts
* Do not call notmuch for non-inlinable audio/mpeg parts
The application/mpeg test passes thanks to a workaround for
application/* Content-Types. The audio/mpeg is currently broken.
---
test/emacs
Changes:
v4 since v3:
* Use $(()) instead of expr(1) for math.
v3 since v2:
* Use read function instead of cat(1) to read counter value.
* Add a newline after the count value in state file and
notmuch_counter_value() output.
v2 since v1:
* Rename $notmuch_counter variable to $notmuch_counte
Hi Tomi.
On Tue, 29 Nov 2011 14:58:00 +0200, Tomi Ollila wrote:
> Hi Dmitry.
>
> On Tue, 29 Nov 2011 01:26:39 +0400, Dmitry Kurochkin
> wrote:
> > Hi Tomi.
> >
> > On Mon, 28 Nov 2011 22:42:50 +0200, Tomi Ollila wrote:
> > > On Mon, 28 Nov 2011 07:28:1
On Tue, 29 Nov 2011 02:13:53 +0400, Dmitry Kurochkin
wrote:
> On Tue, 29 Nov 2011 01:53:49 +0400, Dmitry Kurochkin
> wrote:
> > On Mon, 28 Nov 2011 23:16:27 +0200, Tomi Ollila wrote:
> > > On Thu, 17 Nov 2011 05:56:21 +0400, Dmitry Kurochkin
> > > wrote:
>
Hi Jamie.
On Mon, 28 Nov 2011 06:24:19 -0800, Jameson Graef Rollins
wrote:
> On Sat, 26 Nov 2011 02:23:30 +0400, Dmitry Kurochkin
> wrote:
> > -(defun notmuch-show-get-header (header)
> > +(defun notmuch-show-get-header (header &optional props)
> >"Retur
The fake missing binary functions check if the binary has already be
added to the diagnostic message to avoid duplicates. Unfortunately,
this check was buggy because the message string does not have the
trailing space.
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(
The grep(1) command used in the fake binary functions was missing the
quiet option.
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/test/test-lib.sh b/test/test-lib.sh
index 9dcb2d2..2861d88 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -556
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/test/test-lib.sh b/test/test-lib.sh
index d7282ff..9dcb2d2 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -557,7 +557,7 @@ test_declare_external_prereq () {
test_missing_external_prereq_
test_missing_external_prereq_${binary}_ variable indicates that the
binary is missing. It must be set in test_declare_external_prereq()
outside of the fake $binary() function.
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/test/test-lib.sh b/test/te
Hello.
This is a series of trivial but important fixes for the recently added
binary dependencies.
Special thanks goes to Tomi Ollila who did review of the original
patches.
Regards,
Dmitry
___
notmuch mailing list
notmuch@notmuchmail.org
http://not
On Tue, 29 Nov 2011 01:53:49 +0400, Dmitry Kurochkin
wrote:
> On Mon, 28 Nov 2011 23:16:27 +0200, Tomi Ollila wrote:
> > On Thu, 17 Nov 2011 05:56:21 +0400, Dmitry Kurochkin
> > wrote:
> > > There is existing support for general prerequisites in the test suite.
On Mon, 28 Nov 2011 23:16:27 +0200, Tomi Ollila wrote:
> On Thu, 17 Nov 2011 05:56:21 +0400, Dmitry Kurochkin
> wrote:
> > There is existing support for general prerequisites in the test suite.
> > But it is not very convenient to use: every test case has to keep
> > tra
Hi Tomi.
On Mon, 28 Nov 2011 22:42:50 +0200, Tomi Ollila wrote:
> On Mon, 28 Nov 2011 07:28:13 +0400, Dmitry Kurochkin
> wrote:
>
> [...]
> > +
> > +These allow to count how many times notmuch binary is called.
> > +notmuch_counter_reset() function
Before the change, there was a workaround to avoid notmuch show calls
for parts with application/* Content-Type. But non-inlinable parts
are not limited to this Content-Type (e.g. mp3 files have audio/mpeg
Content-Type and are not inlinable). For such parts
`notmuch-show-insert-part-*/*' handler
The patch adds two new test cases:
* Do not call notmuch for non-inlinable application/mpeg parts
* Do not call notmuch for non-inlinable audio/mpeg parts
The application/mpeg test passes thanks to a workaround for
application/* Content-Types. The audio/mpeg is currently broken.
---
test/emacs
The patch adds two auxiliary functions and a variable:
notmuch_counter_reset
$notmuch_counter_command
notmuch_counter_value
They allow to count how many times notmuch binary is called.
notmuch_counter_reset() function generates a script that counts how
many times it is called and resets the
Make some changes suggested by Tomi Ollila [1].
Changes:
v3 since v2:
* Use read function instead of cat(1) to read counter value.
* Add a newline after the count value in state file and
notmuch_counter_value() output.
v2 since v1:
* Rename $notmuch_counter variable to $notmuch_counter_comma
On Tue, 29 Nov 2011 02:13:53 +0400, Dmitry Kurochkin wrote:
> On Tue, 29 Nov 2011 01:53:49 +0400, Dmitry Kurochkin gmail.com> wrote:
> > On Mon, 28 Nov 2011 23:16:27 +0200, Tomi Ollila
> > wrote:
> > > On Thu, 17 Nov 2011 05:56:21 +0400, Dmitry Kurochkin > > g
The fake missing binary functions check if the binary has already be
added to the diagnostic message to avoid duplicates. Unfortunately,
this check was buggy because the message string does not have the
trailing space.
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(
The grep(1) command used in the fake binary functions was missing the
quiet option.
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/test/test-lib.sh b/test/test-lib.sh
index 9dcb2d2..2861d88 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -556
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/test/test-lib.sh b/test/test-lib.sh
index d7282ff..9dcb2d2 100644
--- a/test/test-lib.sh
+++ b/test/test-lib.sh
@@ -557,7 +557,7 @@ test_declare_external_prereq () {
test_missing_external_prereq_
test_missing_external_prereq_${binary}_ variable indicates that the
binary is missing. It must be set in test_declare_external_prereq()
outside of the fake $binary() function.
---
test/test-lib.sh |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/test/test-lib.sh b/test/te
Hello.
This is a series of trivial but important fixes for the recently added
binary dependencies.
Special thanks goes to Tomi Ollila who did review of the original
patches.
Regards,
Dmitry
On Tue, 29 Nov 2011 01:53:49 +0400, Dmitry Kurochkin wrote:
> On Mon, 28 Nov 2011 23:16:27 +0200, Tomi Ollila wrote:
> > On Thu, 17 Nov 2011 05:56:21 +0400, Dmitry Kurochkin > gmail.com> wrote:
> > > There is existing support for general prerequisites in the test suite.
On Mon, 28 Nov 2011 23:16:27 +0200, Tomi Ollila wrote:
> On Thu, 17 Nov 2011 05:56:21 +0400, Dmitry Kurochkin gmail.com> wrote:
> > There is existing support for general prerequisites in the test suite.
> > But it is not very convenient to use: every test case has to keep
Hi Tomi.
On Mon, 28 Nov 2011 22:42:50 +0200, Tomi Ollila wrote:
> On Mon, 28 Nov 2011 07:28:13 +0400, Dmitry Kurochkin gmail.com> wrote:
>
> [...]
> > +
> > +These allow to count how many times notmuch binary is called.
> > +notmuch_counter_reset() fu
Before the change, there was a workaround to avoid notmuch show calls
for parts with application/* Content-Type. But non-inlinable parts
are not limited to this Content-Type (e.g. mp3 files have audio/mpeg
Content-Type and are not inlinable). For such parts
`notmuch-show-insert-part-*/*' handler
The patch adds two new test cases:
* Do not call notmuch for non-inlinable application/mpeg parts
* Do not call notmuch for non-inlinable audio/mpeg parts
The application/mpeg test passes thanks to a workaround for
application/* Content-Types. The audio/mpeg is currently broken.
---
test/emacs
The patch adds two auxiliary functions and a variable:
notmuch_counter_reset
$notmuch_counter_command
notmuch_counter_value
They allow to count how many times notmuch binary is called.
notmuch_counter_reset() function generates a script that counts how
many times it is called and resets the
Make some changes suggested by Tomi Ollila [1].
Changes:
v3 since v2:
* Use read function instead of cat(1) to read counter value.
* Add a newline after the count value in state file and
notmuch_counter_value() output.
v2 since v1:
* Rename $notmuch_counter variable to $notmuch_counter_comma
Hi Jamie.
On Mon, 28 Nov 2011 06:24:19 -0800, Jameson Graef Rollins wrote:
> On Sat, 26 Nov 2011 02:23:30 +0400, Dmitry Kurochkin gmail.com> wrote:
> > -(defun notmuch-show-get-header (header)
> > +(defun notmuch-show-get-header (header &optional props)
> >&qu
Hi Daniel.
On Thu, 24 Nov 2011 15:01:01 +0100, Daniel Schoepe
wrote:
> On Thu, 24 Nov 2011 09:54:50 -0400, David Bremner
> wrote:
> > On Mon, 10 Oct 2011 15:39:41 +0200, Daniel Schoepe
> > wrote:
> > > From: Daniel Schoepe
> > >
> > > This patch makes the notmuch-hello screen fully customi
Before the change, there was a workaround to avoid notmuch show calls
for parts with application/* Content-Type. But non-inlinable parts
are not limited to this Content-Type (e.g. mp3 files have audio/mpeg
Content-Type and are not inlinable). For such parts
`notmuch-show-insert-part-*/*' handler
The patch adds two new test cases:
* Do not call notmuch for non-inlinable application/mpeg parts
* Do not call notmuch for non-inlinable audio/mpeg parts
The application/mpeg test passes thanks to a workaround for
application/* Content-Types. The audio/mpeg is currently broken.
---
test/emacs
The patch adds two auxiliary functions and a variable:
notmuch_counter_reset
$notmuch_counter_command
notmuch_counter_value
They allow to count how many times notmuch binary is called.
notmuch_counter_reset() function generates a script that counts how
many times it is called and resets the
701 - 800 of 1248 matches
Mail list logo
