Check whether realpath(3) supports the resolved_path == NULL feature,
standardized in POSIX.1-2008.
This is tested by executing the realpath(3) with NULL as second
argument and the program is expected to SIGSEGV in case the
feature is not supported.
If the feature is not supported the compatibili
On Sat, Jan 25 2014, David Bremner wrote:
> Mark Walters writes:
>
>> ido-completing-read is used in notmuch-mua.el without any
>> initialization. This should work but a bug (emacs bug #3247) in some
>> emacs 23 versions (including 23.2.1 from Debian stable) causes emacs
>> to get stuck in the
On Sun, 19 Jan 2014, Mark Walters wrote:
> I have been running this series (with the unread and auto-updating
> described above) for about a month and it seems to work well for my use.
I've been running this for a while now, and I really like it. Haven't
noticed any issues. I'm anticipating the f
On Sat, 25 Jan 2014, Mark Walters wrote:
> What about a config option? Something like
> database_auto_upgrade=true/false? I wouldn't have a strong preference
> which was the default (though I would choose "false" in my own
> config). I guess we would need a command line --upgrade to allow people
>
Tomi Ollila writes:
>
> How about doing it with defadvice with something like:
>
> (from http://comments.gmane.org/gmane.emacs.bugs/27856 one can notice
> this happening with emacs 23.3 -- David can check whether 23.4 if
> affected)
It's fixed in 23.4
>
> (if (and (= emacs-major-version 23) (<
Tomi Ollila writes:
> +#if POSIX_2008_REALPATH
> filename = realpath (config->filename, NULL);
> +#else
> +/* compatibility with minor effort, not elegance, is the ruling factor
> + in these (two) else branches... */
> +char resolved_path[PATH_MAX];
> +filename = realpath (c
On Sat, 25 Jan 2014, David Bremner wrote:
> Jani Nikula writes:
>
>> On Sat, 25 Jan 2014, Jani Nikula wrote:
>>> Perhaps we need to have two prefixes, one of which is the literal
>>> filesystem folder and another which hides the implementation details,
>>> like I mentioned in my mail to Peter [1
On Sun, Jan 19 2014, Jani Nikula wrote:
> Here's an assortment of notmuch new fixes, non-functional cleanups, and
> the oft-requested --quiet option.
>
> These should be pretty straightforward to review.
>
series LGTM.
Tomi
> BR,
> Jani.
>
>
> Jani Nikula (7):
> cli: extract single message a
On Sat, 25 Jan 2014, Jani Nikula wrote:
> Perhaps we need to have two prefixes, one of which is the literal
> filesystem folder and another which hides the implementation details,
> like I mentioned in my mail to Peter [1]. But consider this: my proposed
> implementation does cover *all* use cases
Thanks for posting this. You are quite right about it being orthogonal
to this series so a clear +1 from me for the series.
What about a config option? Something like
database_auto_upgrade=true/false? I wouldn't have a strong preference
which was the default (though I would choose "false" in my o
Tomi Ollila writes:
>
> How about doing it with defadvice with something like:
>
> (from http://comments.gmane.org/gmane.emacs.bugs/27856 one can notice
> this happening with emacs 23.3 -- David can check whether 23.4 if
> affected)
It's fixed in 23.4
>
> (if (and (= emacs-major-version 23) (<
On Sat, 25 Jan 2014, Mark Walters wrote:
> This series LGTM.
Hi Mark, thanks for the review!
> I do now recall there was some discussion on irc about the automatic
> database upgrade: it would be good to have that documented but the
> consensus was to do it, so +1 from me.
Here's some summary,
Tomi Ollila writes:
> +#if POSIX_2008_REALPATH
> filename = realpath (config->filename, NULL);
> +#else
> +/* compatibility with minor effort, not elegance, is the ruling factor
> + in these (two) else branches... */
> +char resolved_path[PATH_MAX];
> +filename = realpath (c
Jani Nikula writes:
> Here's a thought. With boolean prefix folder:, we can devise a scheme
> where the folder: query defines what is to be matched.
I like the idea, but I tried to infer the rules from the examples, and I
failed. It looks like there are two new symbols, "/" and "/." but I
couldn'
Michal Sojka writes:
> Hello,
>
> in emacs, I have a problem displaying an email that contain the
> following part (--format=json):
>
> {
>"content-type" : "image/svg+xml",
>"filename" : "_logo.svg",
>"id" : 4
> }
>
> I see this message "mm-image-fit-p: Invalid
This version file will be as prerequisite to the target files
that use the version info for some purpose, like printing
it to the user to examine. The contents of the VERSION file
is seldom read by the build system itself as the $(VERSION)
variable has the same information.
---
Makefile.local | 11
Mark Walters writes:
> ido-completing-read is used in notmuch-mua.el without any
> initialization. This should work but a bug (emacs bug #3247) in some
> emacs 23 versions (including 23.2.1 from Debian stable) causes emacs
> to get stuck in the mini-buffer. This tests to see if ido-mode has
> be
All implicit rules in notmuch Makefiles are "pattern rules"; Deleting the
default suffixes (to support obsolete, old-fashioned "suffix rules") from
make reduces the output of 'make -d' by 40 to 90 percent, helping e.g.
debugging make problems.
---
output size comparisons:
full-build: script -c 'm
On Sat, Jan 25 2014, Tomi Ollila wrote:
> Check whether realpath(3) supports the resolved_path == NULL feature,
> standardized in POSIX.1-2008.
>
> This is tested by executing the realpath(3) with NULL as second
> argument and the program is expected to SIGSEGV in case the
> feature is not suppor
Check whether realpath(3) supports the resolved_path == NULL feature,
standardized in POSIX.1-2008.
This is tested by executing the realpath(3) with NULL as second
argument and the program is expected to SIGSEGV in case the
feature is not supported.
If the feature is not supported the compatibili
On Sat, Jan 25 2014, Jani Nikula wrote:
> On Sat, 25 Jan 2014, Tomi Ollila wrote:
>> On Sat, Jan 25 2014, Jani Nikula wrote:
>>> Perhaps we need to have two prefixes, one of which is the literal
>>> filesystem folder and another which hides the implementation details,
>>> like I mentioned in my
On Sat, Jan 25 2014, David Bremner wrote:
> Clarify that using the directory after destroying the corresponding
> database is not permitted.
>
> This is implicit in the description of notmuch_database_destroy, but
> it doesn't hurt to be explicit, and we do express similar "ownership"
> relations
Austin Clements writes:
> This is the other reason I disagree with including cur/new. If we strip
> it, people's existing folder: searches will mostly continue to work. If
> we include it, such saved searches, scripted searches, and post-new
> hooks are guaranteed to break.
What about Maildir+
On Sat, Jan 25 2014, David Bremner wrote:
> Mark Walters writes:
>
>> ido-completing-read is used in notmuch-mua.el without any
>> initialization. This should work but a bug (emacs bug #3247) in some
>> emacs 23 versions (including 23.2.1 from Debian stable) causes emacs
>> to get stuck in the
Tomi Ollila writes:
> All implicit rules in notmuch Makefiles are "pattern rules"; Deleting the
> default suffixes (to support obsolete, old-fashioned "suffix rules") from
> make reduces the output of 'make -d' by 40 to 90 percent, helping e.g.
> debugging make problems.
LGTM.
d
On Sat, 25 Jan 2014, Tomi Ollila wrote:
> On Sat, Jan 25 2014, Jani Nikula wrote:
>> Perhaps we need to have two prefixes, one of which is the literal
>> filesystem folder and another which hides the implementation details,
>> like I mentioned in my mail to Peter [1]. But consider this: my propos
Jani Nikula writes:
> On Sat, 25 Jan 2014, Jani Nikula wrote:
>> Perhaps we need to have two prefixes, one of which is the literal
>> filesystem folder and another which hides the implementation details,
>> like I mentioned in my mail to Peter [1]. But consider this: my proposed
>> implementatio
On Sat, Jan 25 2014, Jani Nikula wrote:
> On Fri, 24 Jan 2014, Austin Clements wrote:
>> On Thu, 09 Jan 2014, Jani Nikula wrote:
>>> Hi all, this series makes the folder: search prefix literal, or switches
>>> it from a probabilistic prefix to a boolean prefix. With this, you have
>>> to give t
So that the target is newer than its prerequisites.
---
emacs/Makefile.local | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/emacs/Makefile.local b/emacs/Makefile.local
index 42bfbd9..d5d402e 100644
--- a/emacs/Makefile.local
+++ b/emacs/Makefile.local
@@ -32,8 +32,7 @@ emacs
Austin Clements writes:
> This is the other reason I disagree with including cur/new. If we strip
> it, people's existing folder: searches will mostly continue to work. If
> we include it, such saved searches, scripted searches, and post-new
> hooks are guaranteed to break.
What about Maildir+
On Sun, 19 Jan 2014, Mark Walters wrote:
> I have been running this series (with the unread and auto-updating
> described above) for about a month and it seems to work well for my use.
I've been running this for a while now, and I really like it. Haven't
noticed any issues. I'm anticipating the f
On Fri, 24 Jan 2014, Austin Clements wrote:
> On Thu, 09 Jan 2014, Jani Nikula wrote:
>> Hi all, this series makes the folder: search prefix literal, or switches
>> it from a probabilistic prefix to a boolean prefix. With this, you have
>> to give the path from the maildir root to the folder you
On Sat, 25 Jan 2014, Mark Walters wrote:
> What about a config option? Something like
> database_auto_upgrade=true/false? I wouldn't have a strong preference
> which was the default (though I would choose "false" in my own
> config). I guess we would need a command line --upgrade to allow people
>
Michal Sojka writes:
> Hello,
>
> in emacs, I have a problem displaying an email that contain the
> following part (--format=json):
>
> {
>"content-type" : "image/svg+xml",
>"filename" : "_logo.svg",
>"id" : 4
> }
>
> I see this message "mm-image-fit-p: Invalid
Mark Walters writes:
> ido-completing-read is used in notmuch-mua.el without any
> initialization. This should work but a bug (emacs bug #3247) in some
> emacs 23 versions (including 23.2.1 from Debian stable) causes emacs
> to get stuck in the mini-buffer. This tests to see if ido-mode has
> be
On Sat, 25 Jan 2014, David Bremner wrote:
> Jani Nikula writes:
>
>> On Sat, 25 Jan 2014, Jani Nikula wrote:
>>> Perhaps we need to have two prefixes, one of which is the literal
>>> filesystem folder and another which hides the implementation details,
>>> like I mentioned in my mail to Peter [1
Thanks for posting this. You are quite right about it being orthogonal
to this series so a clear +1 from me for the series.
What about a config option? Something like
database_auto_upgrade=true/false? I wouldn't have a strong preference
which was the default (though I would choose "false" in my o
On Sun, Jan 19 2014, Jani Nikula wrote:
> Here's an assortment of notmuch new fixes, non-functional cleanups, and
> the oft-requested --quiet option.
>
> These should be pretty straightforward to review.
>
series LGTM.
Tomi
> BR,
> Jani.
>
>
> Jani Nikula (7):
> cli: extract single message a
Tomi Ollila writes:
> All implicit rules in notmuch Makefiles are "pattern rules"; Deleting the
> default suffixes (to support obsolete, old-fashioned "suffix rules") from
> make reduces the output of 'make -d' by 40 to 90 percent, helping e.g.
> debugging make problems.
LGTM.
d
___
Jani Nikula writes:
> On Sat, 25 Jan 2014, Jani Nikula wrote:
>> Perhaps we need to have two prefixes, one of which is the literal
>> filesystem folder and another which hides the implementation details,
>> like I mentioned in my mail to Peter [1]. But consider this: my proposed
>> implementatio
This series LGTM.
I do now recall there was some discussion on irc about the automatic
database upgrade: it would be good to have that documented but the
consensus was to do it, so +1 from me.
Best wishes
Mark
On Sun, 19 Jan 2014, Jani Nikula wrote:
> Here's an assortment of notmuch new fi
On Sat, 25 Jan 2014, Jani Nikula wrote:
> Perhaps we need to have two prefixes, one of which is the literal
> filesystem folder and another which hides the implementation details,
> like I mentioned in my mail to Peter [1]. But consider this: my proposed
> implementation does cover *all* use cases
On Sat, 25 Jan 2014, Mark Walters wrote:
> This series LGTM.
Hi Mark, thanks for the review!
> I do now recall there was some discussion on irc about the automatic
> database upgrade: it would be good to have that documented but the
> consensus was to do it, so +1 from me.
Here's some summary,
All implicit rules in notmuch Makefiles are "pattern rules"; Deleting the
default suffixes (to support obsolete, old-fashioned "suffix rules") from
make reduces the output of 'make -d' by 40 to 90 percent, helping e.g.
debugging make problems.
---
output size comparisons:
full-build: script -c 'm
On Sat, Jan 25 2014, Jani Nikula wrote:
> On Sat, 25 Jan 2014, Tomi Ollila wrote:
>> On Sat, Jan 25 2014, Jani Nikula wrote:
>>> Perhaps we need to have two prefixes, one of which is the literal
>>> filesystem folder and another which hides the implementation details,
>>> like I mentioned in my
On Sat, Jan 25 2014, David Bremner wrote:
> Clarify that using the directory after destroying the corresponding
> database is not permitted.
>
> This is implicit in the description of notmuch_database_destroy, but
> it doesn't hurt to be explicit, and we do express similar "ownership"
> relations
On Sat, 25 Jan 2014, Tomi Ollila wrote:
> On Sat, Jan 25 2014, Jani Nikula wrote:
>> Perhaps we need to have two prefixes, one of which is the literal
>> filesystem folder and another which hides the implementation details,
>> like I mentioned in my mail to Peter [1]. But consider this: my propos
On Sat, Jan 25 2014, Jani Nikula wrote:
> On Fri, 24 Jan 2014, Austin Clements wrote:
>> On Thu, 09 Jan 2014, Jani Nikula wrote:
>>> Hi all, this series makes the folder: search prefix literal, or switches
>>> it from a probabilistic prefix to a boolean prefix. With this, you have
>>> to give t
So that the target is newer than its prerequisites.
---
emacs/Makefile.local | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/emacs/Makefile.local b/emacs/Makefile.local
index 42bfbd9..d5d402e 100644
--- a/emacs/Makefile.local
+++ b/emacs/Makefile.local
@@ -32,8 +32,7 @@ emacs
On Fri, 24 Jan 2014, Austin Clements wrote:
> On Thu, 09 Jan 2014, Jani Nikula wrote:
>> Hi all, this series makes the folder: search prefix literal, or switches
>> it from a probabilistic prefix to a boolean prefix. With this, you have
>> to give the path from the maildir root to the folder you
This series LGTM.
I do now recall there was some discussion on irc about the automatic
database upgrade: it would be good to have that documented but the
consensus was to do it, so +1 from me.
Best wishes
Mark
On Sun, 19 Jan 2014, Jani Nikula wrote:
> Here's an assortment of notmuch new fi
51 matches
Mail list logo
