[PATCH] doc: whitespace cleanup

2016-11-26 Thread Tomi Ollila
In most part, our .rst documents are indented with 8 spaces instead of tabs. Bring the rest of the lines to the same format. Also, on one (supposedly empty) line, trailing spaces were removed. --- This set is something that does not collide with anything that is currently in the review queue --

Re: [PATCH v3] build: install notmuch-emacs-mua.desktop file with emacs

2016-11-26 Thread Tomi Ollila
On Sat, Nov 26 2016, Jani Nikula wrote: > There is really no need to have a separate install target for the > desktop file. Just install the desktop file with emacs, with a > configure option to opt out. > > --- > > v3: check for desktop-file-install in configure. This and the

Re: provide elpa packages

2016-11-26 Thread Tomi Ollila
On Sat, Nov 26 2016, David Bremner wrote: > Tomi Ollila writes: > >> >> (you could amend copying Makefile.global for out of tree builds -- or use >> include $(srcdir)/Makefile.global) >> > > OK, I did that, and pushed to master and release. BTW, the next

Re: [PATCH v2 6/9] notmuch.desktop: move under emacs as notmuch-emacs-mua.desktop

2016-11-26 Thread David Bremner
Jani Nikula writes: > The notmuch.desktop file is part of notmuch-emacs. Move it under > emacs, and rename as notmuch-emacs-mua.desktop to reflect this. Pushed patches 2-6 to master d ___ notmuch mailing list notmuch@notmuchmail.org

Re: provide elpa packages

2016-11-26 Thread David Bremner
Tomi Ollila writes: > > (you could amend copying Makefile.global for out of tree builds -- or use > include $(srcdir)/Makefile.global) > OK, I did that, and pushed to master and release. BTW, the next line still looks problematic for oot builds? Did not test... d

[PATCH v3] build: update the desktop database after installing the desktop file

2016-11-26 Thread Jani Nikula
This makes the option to choose Notmuch as mailto: handler show up in the desktop environment settings. Ignore errors. --- v3: ignore errors on update-desktop-database. I don't want to skip desktop-file-install if update-desktop-database is not available, and if update fails, it's likely a

[PATCH v3] build: install notmuch-emacs-mua.desktop file with emacs

2016-11-26 Thread Jani Nikula
There is really no need to have a separate install target for the desktop file. Just install the desktop file with emacs, with a configure option to opt out. --- v3: check for desktop-file-install in configure. I don't see a reason to differentiate between "with desktop" and "have

[PATCH] test: check the handling of files vanishing between scandir and indexing

2016-11-26 Thread Jani Nikula
Add a file for scandir to find, but use gdb to remove it before it gets indexed. --- The ugly part is that this should require gdb as external dep... but we shouldn't skip all of T050-new.sh if gdb isn't there. I'm in a hurry, any good ideas? --- test/T050-new.sh | 30

Re: [PATCH] cli/new: document new exit code

2016-11-26 Thread Jani Nikula
On Sat, 26 Nov 2016, David Bremner wrote: > It seems important to give the numeric return code for people writing > scripts. Hopefully deviations from this convention are rare. *blush* As a token of my gratitude, a test for the change follows. I'm not sure if it's quite ready