notmuch/lib/open.cc, branch 0.38.1 thread-based email index, search, and tagging https://git.notmuchmail.org/git/notmuch/atom?h=0.38.1 2023-09-23T11:34:48Z Pass error message from GLib ini parser to CLI 2023-09-23T11:34:48Z David Bremner david@tethera.net 2023-09-15T12:50:04Z urn:sha1:1c10d91d8e4a3e5bc76ca4c6b9939f3759e6ef5e The function _notmuch_config_load_from_file is only called in two places in open.cc. Update internal API to match the idiom in open.cc. Adding a newline is needed for consistency with other status strings. Based in part on a patch [1] from Eric Blake. [1]: id:20230906153402.101471-1-eblake@redhat.com lib: parse index.as_text 2023-04-02T22:22:36Z David Bremner david@tethera.net 2023-01-06T00:02:05Z urn:sha1:3f5809bf28becbddfed9ff33d6f1242346904c23 We pre-parse into a list of compiled regular expressions to avoid calling regexc on the hot (indexing) path. As explained in the code comment, this cannot be done lazily with reasonable error reporting, at least not without touching a lot of the code in index.cc. lib: add field processor for lastmod: prefix 2022-09-03T11:43:33Z David Bremner david@tethera.net 2022-08-14T15:02:59Z urn:sha1:2e5ef69fbf9ce9d67720d5d4abba3026302734e5 By sharing the existing logic used by the sexp query parser, this allows negative lastmod revisions to be interpreted as relative to the most recent revision. lib/open: create database path in some cases 2022-09-03T11:24:43Z David Bremner david@tethera.net 2022-07-29T11:31:37Z urn:sha1:84e4e130e2c920b3dee91901582c4ab6276e2630 There is some duplication of code here, but not all of the locations valid to find a database make sense to create. Furthermore we nead two passes, so the control flow in _choose_database_path would get a bit convoluted. lib/open: return non-SUCCESS on missing database path 2022-09-03T11:24:43Z David Bremner david@tethera.net 2022-07-29T11:23:11Z urn:sha1:8ba3057d01b11fb806581f8dc451a8891a4d4e0e This simplifies the logic of creating the directory path when it doesn't exist. lib/open: refactor call to mkdir into function 2022-09-03T11:24:43Z David Bremner david@tethera.net 2022-07-28T10:41:02Z urn:sha1:25e2790e303b6d9334676e3165a92c6084aa3567 This makes the error handling available for re-use. Using g_mkdir_with_parents also handles the case of a pre-existing directory. This introduces new functionality, namely creating the parent directories, which will be useful for creating directories like '.local/share/notmuch/default'. lib/open: use notmuch->params to track split status 2022-01-15T19:53:31Z David Bremner david@tethera.net 2021-12-25T13:33:03Z urn:sha1:63b4c469835133ac44b7db516fc54b8a4abb90a8 Persisting this status will allow us to use the information in other compilation units, in particular when setting configuration defaults. lib/open: use db struct as talloc ctx for choose_database_path 2022-01-15T19:51:33Z David Bremner david@tethera.net 2021-12-25T13:33:02Z urn:sha1:fd0edeb561e5593d8a834fcc74602b2d8c637123 The extra talloc struct "local" was left over from before the notmuch struct was allocated earlier. Having the notmuch struct available in this function will allow more flexibility to track the configuration variations (e.g. split vs. non-split). Merge branch 'release' 2021-12-29T18:20:49Z David Bremner david@tethera.net 2021-12-29T18:20:49Z urn:sha1:3eb25c94bd8fe4065d6df6d665ee393cb9a0ad6f lib/open: do not consider .notmuch alone as an existing database. 2021-12-29T18:11:21Z David Bremner david@tethera.net 2021-12-25T21:22:56Z urn:sha1:25e0f5e59293ce961549201fdc74f81a3cc1675c It makes perfect sense for users to want to pre-create .notmuch, e.g. to install hooks, so we should handle the case of a .notmuch directory without an actual xapian database more gracefully.