Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread David Bremner
Tomi Ollila writes: > > --ignore-whitespace could help... > > for the --no-keep-cr (the default) not to work, in thread containing this > >https://lkml.org/lkml/2019/12/1/69 > > may give some hint why the situation is as unclear as it is now... --ignore-whitespace adds CRs to the files,

Re: [PATCH v5] Emacs: Ensure left-to-right display for message headers

2020-08-09 Thread =?utf-8?Q?T=E2=80=AEomi?= Ollila
How about this =D (Sorry how the headers might look...) From: contains U+202E (LEFT-TO-RIGHT OVERRIDE) (in =?utf-8?Q?T=E2=80=AEomi?=) https://www.fileformat.info/info/unicode/char/202e/index.htm Tomi --- (top-posting on purpose, all rest is for reference only) On Fri, Aug 07 2020, Teemu

[PATCH] Emacs: Indent first header line only when indentation is turned on

2020-08-09 Thread Teemu Likonen
Previously in message-show mode message's first header line (From header) was always indented, even if user had turned thread indentation off with "<" (notmuch-show-toggle-thread-indentation) command. This change modifies notmuch-show-insert-headerline function so that it doesn't indent the first

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread Tomi Ollila
On Sun, Aug 09 2020, David Bremner wrote: > Jonas Bernoulli writes: >> >> Jonas Bernoulli (34): >> emacs: Shorten long lines >> emacs: Remove excess empty lines > > > These patches (and Đoàn Trần Công Danh's that I just applied by hand) > both seem to have problem with base64 encoded patches

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread Jonas Bernoulli
David Bremner writes: > A short term fix would be to fetch a ref from somewhere. You can fetch it with: git fetch https://github.com/tarsiiformes/notmuch.git cleanup Jonas ___ notmuch mailing list -- notmuch@notmuchmail.org To unsubscribe send

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread Jonas Bernoulli
Đoàn Trần Công Danh writes: > On 2020-08-09 09:09:39-0300, David Bremner wrote: >> Jonas Bernoulli writes: >> > >> > Jonas Bernoulli (34): >> > emacs: Shorten long lines >> > emacs: Remove excess empty lines >> >> >> These patches (and Đoàn Trần Công Danh's that I just applied by hand)

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread Đoàn Trần Công Danh
On 2020-08-09 09:09:39-0300, David Bremner wrote: > Jonas Bernoulli writes: > > > > Jonas Bernoulli (34): > > emacs: Shorten long lines > > emacs: Remove excess empty lines > > > These patches (and Đoàn Trần Công Danh's that I just applied by hand) > both seem to have problem with base64

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread David Bremner
Jonas Bernoulli writes: > > Jonas Bernoulli (34): > emacs: Shorten long lines > emacs: Remove excess empty lines These patches (and Đoàn Trần Công Danh's that I just applied by hand) both seem to have problem with base64 encoded patches documented at.

Re: [PATCH 02/19] lib: add stub for notmuch_database_open_with_config

2020-08-09 Thread David Bremner
Reto writes: >> + * Mode to open database. Use one of #NOTMUCH_DATABASE_MODE_READ_WRITE >> + * or #NOTMUCH_DATABASE_MODE_READ_WRITE > > I think you want to have NOTMUCH_DATABASE_MODE_READ_ONLY here? > > Greetings, > Reto yes, you are correct. d ___

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread Jonas Bernoulli
Tomi Ollila writes: > I made a quick check of release dates of emacs 25.1, gmime 3 and xapian 1.4 > -- gmime 3 released early 2017, xapian 1.4 middle of 2016 and emacs 25.1 > sept 2016 -- so requiring emacs 25.1 is pretty much aligned to those > releases and therefore dropping emacs 24 support

Re: [PATCH 02/19] lib: add stub for notmuch_database_open_with_config

2020-08-09 Thread Reto
On Sat, Aug 08, 2020 at 11:16:36AM -0300, David Bremner wrote: > +/* NOTMUCH_DEPRECATED(5, 4) */ > +notmuch_status_t > +notmuch_database_open_verbose (const char *path, > +notmuch_database_mode_t mode, > +notmuch_database_t **database, > +

Re: [PATCH v3 00/34] A great number of cosmetic changes

2020-08-09 Thread Tomi Ollila
On Sat, Aug 08 2020, Jonas Bernoulli wrote: > David Bremner writes: > >> Tomi Ollila writes: > >>> So we have 3 options: >>> >>> 1) apply this patch and now drop support for emacs 24 >>> >>> 2) apply this patch and somehow infor emacs 24 users to install cl-lib from >>> ELPA >> >> I'm fine with