[PATCH v2] move more http -> https

2018-04-29 Thread Daniel Kahn Gillmor
Correct URLs that have crept into the notmuch codebase with http:// when https:// is possible. As part of this conversion, this changeset also indicates the current preferred upstream URLs for both gmime and sup. the new URLs are https-enabled, the old ones are not. This also fixes T310-emacs.sh

Re: non-deterministic behaviour of new.ignore (regexp) test

2018-04-29 Thread Jani Nikula
On Sat, 28 Apr 2018, David Bremner wrote: > For me the following seems to consistently fail after between 30 and 500 > attempts > > export NOTMUCH_TEST_QUIET=yes; count=0; while ./T050-new.sh; do (( > count++ )); echo $count; done I believe this happens because the directory mtime is unchang

Re: Database corruption after clean rebuild

2018-04-29 Thread Gregor Zattler
Hi notmuch developers, I also had this database corruption, I waited for the fix to land in notmuch 0.26.2, build it, moved the xapian directory away, did a notmuch new and restored the tags from a dump. But the problem remains: ~$ xapian-check ~/Mail/.notmuch/xapian docdata: blocksize=8K items=

[PATCH 2/2] test: use --no-mtime-opt in T050-new.sh

2018-04-29 Thread David Bremner
Wherever the test relies on directories being scanned, this option should be used to avoid skipping them due to mtimes on directories matching the database. --- test/T050-new.sh | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/test/T050-new.sh b/test/T050-new.sh i

[PATCH 1/2] CLI/new: add mtime-opt option

2018-04-29 Thread David Bremner
This option, on by default, controls whether mtimes are used to optimize the scanning of directories. The intent is to turn this optimization off for certain tests. --- notmuch-new.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/notmuch-new.c b/notmuch-new.c index c4

Re: Database corruption after clean rebuild

2018-04-29 Thread David Bremner
Gregor Zattler writes: > Hi notmuch developers, > > I also had this database corruption, I waited for the fix to land > in notmuch 0.26.2, build it, moved the xapian directory away, did > a notmuch new and restored the tags from a dump. But the problem > remains: > > ~$ xapian-check ~/Mail/.notm

Re: [PATCH 1/2] CLI/new: add mtime-opt option

2018-04-29 Thread Jani Nikula
On Sun, 29 Apr 2018, David Bremner wrote: > This option, on by default, controls whether mtimes are used to > optimize the scanning of directories. The intent is to turn this > optimization off for certain tests. > --- > notmuch-new.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(

Re: [PATCH 2/2] test: use --no-mtime-opt in T050-new.sh

2018-04-29 Thread Jani Nikula
On Sun, 29 Apr 2018, David Bremner wrote: > Wherever the test relies on directories being scanned, this option > should be used to avoid skipping them due to mtimes on directories > matching the database. I think you could additionally remove a few touch calls in the test. Some of them do actuall

Re: Database corruption after clean rebuild

2018-04-29 Thread Gregor Zattler
Hi David, * David Bremner [2018-04-29; 11:27]: > Gregor Zattler writes: >> I also had this database corruption, I waited for the fix to land >> in notmuch 0.26.2, build it, moved the xapian directory away, did >> a notmuch new and restored the tags from a dump. But the problem >> remains: >> >>

Re: notmuch show --decrypt=stash

2018-04-29 Thread Daniel Kahn Gillmor
On Tue 2018-01-09 18:12:23 -0500, Daniel Kahn Gillmor wrote: > This is a revision of the series initially introduced in > id:20171212025225.11854-1-...@fifthhorseman.net, with minor updates to > accomodate the recent release of notmuch 0.26 (yay!) > > This series allows "notmuch show" to index the

[PATCH 3/4] test: use --full-scan in T050-new.sh

2018-04-29 Thread David Bremner
Wherever the test relies on directories being scanned, this option should be used to avoid skipping them due to mtimes on directories matching the database. --- test/T050-new.sh | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/test/T050-new.sh b/test/T050-new.sh

[PATCH 2/4] test: add tests for notmuch new --full-scan

2018-04-29 Thread David Bremner
Most of these just check that adding the flag does not break existing functionality. The one test that does check the full-scan functionality had to be rewritten to output debugging info. --- test/T050-new.sh | 32 ++-- 1 file changed, 30 insertions(+), 2 deletions(-)

[PATCH 1/4] CLI/new: add full-scan option

2018-04-29 Thread David Bremner
By default notmuch-new uses directory mtimes to optimize the scanning of directories for new mail. This option allows turning that optimization off e.g. for testing or debugging. --- notmuch-new.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/notmuch-new.c b/notmuch-ne

[PATCH 4/4] doc: document notmuch new --full-scan

2018-04-29 Thread David Bremner
--- doc/man1/notmuch-new.rst | 6 ++ 1 file changed, 6 insertions(+) diff --git a/doc/man1/notmuch-new.rst b/doc/man1/notmuch-new.rst index 16af5492..b0016ccc 100644 --- a/doc/man1/notmuch-new.rst +++ b/doc/man1/notmuch-new.rst @@ -59,6 +59,12 @@ Supported options for **new** include S