Stefan Hagen <[email protected]> writes:

> Hello,
>
> This is a major update of mail/mu. There are too many changes to list
> and explain here.
>
> Please read the official NEWS document:
> https://github.com/djcb/mu/blob/c4388b78b6d8902c927fe334fda4a6fd9e130db6/NEWS.org
>
> I'm personally only using the "mu" part.

i've moved to mblaze + mu, but i still have my mu4e config around.  It
seems to work just fine, fyi i'm typing this mail in emacs after having
found the thread with a query :)

The *mu4e-headers* buffer seems to render faster than what I recall,
just as the NEWS said.

> Can someone please test the mu4e parts and the guile FLAVOR?

never used the guile flavor, sorry :/

> A note on guile (from the NEWS document):
>     The current guile support has been deprecated. It may be revamped at
>     some point, but will be different from the current one, which is to
>     be removed after 1.8
>
> Comments? OK?

just two nitpics:

 - update the comment about the C++ version to C++17
 - drop some CONFIGURE_FLAGS (configure complained that
   --disable-{gtk,webkit} are unrecognized)

I'd also include a patch to fix the time_t in g_snprintf:

patches/patch-lib_mu-store_cc:

Index: lib/mu-store.cc
--- lib/mu-store.cc.orig
+++ lib/mu-store.cc
@@ -599,7 +599,7 @@ Store::set_dirstamp(const std::string& path, time_t ts
 {
        std::array<char, 2 * sizeof(tstamp) + 1> data{};
        const auto len = static_cast<size_t>(
-           g_snprintf(data.data(), data.size(), "%zx", tstamp));
+           g_snprintf(data.data(), data.size(), "%lld", (long long)tstamp));
 
        set_metadata(path, std::string{data.data(), len});
 }


anyway, seems to work just as fine as usual.  ok op@

Reply via email to