Re: [PATCH 1/3] test: add known broken test for conflict with database parameter

2021-12-04 Thread David Bremner
David Bremner writes: > This is arguably user error: having configuration file with bad > settings in it (and/or having a bad NOTMUCH_CONFIG environment > variable). On the other hand returning a different path than was > actually opened is definitely a bug. Series applied to release and

Re: [PATCH 1/3] test: add known broken test for conflict with database parameter

2021-10-27 Thread David Bremner
David Bremner writes: > @@ -933,7 +933,7 @@ NOTMUCH_CONFIG="/nonexistent" > cat c_head3 - c_tail3 <<'EOF' | test_C ${MAIL_DIR} >notmuch_status_t st = notmuch_database_load_config(argv[1], NULL, NULL, > , NULL); > EOF > -NOTMUCH_CONFIG=${old_NOTMUCH_CONFIG} > +export

[PATCH 1/3] test: add known broken test for conflict with database parameter

2021-10-27 Thread David Bremner
This is arguably user error: having configuration file with bad settings in it (and/or having a bad NOTMUCH_CONFIG environment variable). On the other hand returning a different path than was actually opened is definitely a bug. --- test/T590-libconfig.sh | 24 +++- 1 file