emacs: Autoload notmuch-jump-search only once
authorJonas Bernoulli <jonas@bernoul.li>
Sat, 8 Aug 2020 11:49:51 +0000 (13:49 +0200)
committerDavid Bremner <david@tethera.net>
Mon, 10 Aug 2020 00:14:36 +0000 (21:14 -0300)
commitc2e9ec17fd46e67c7cf1f06d4c6da2ed59da34d9
treeefe962f12841ac3d27318f900e303edff9232b3b
parente63f37a4a90c5d2a399a99a0566b6e1dfea263aa
emacs: Autoload notmuch-jump-search only once

This function is being autoloaded using an autoload cookie, so it
shouldn't additionally be autoloaded using an `autoload' form.

When building libraries we don't actually load the autoloads file and
dropping the `autoload' form results in an error, which reveals a so
far unspecified dependency: `notmuch-tree' needs `notmuch-jump'.

Before this commit compiling (or even just loading) `notmuch-tree'
resulted in `notmuch-jump' being loaded because the former requires
`notmuch-lib', which autoloaded `notmuch-jump-search'.

The bug was that this dependency was not explicitly specified, which
we fix by adding the respective `require' form.
emacs/notmuch-lib.el
emacs/notmuch-tree.el