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, which is pretty ick.

I ended up writing a python script to massage the patches, and added
that to my patch application alias.

The series is now applied to master.




email-to-8bit
Description: Binary data
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


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 documented at.
>
>  https://lore.kernel.org/git/20191218194144.gl17...@pobox.com/
>
> It seems that the patches have CRLF line endings inside the base64
> encoding, and git fails to strip the CRs.
>
> I'm not sure why this happens for some patches and not for others;
> I hope there is not re-encoding happening on the mailing list. If there
> is we should fix that.
>
> The practical upshot is that I can't really apply such a long series by
> hand. Ideas welcome. A short term fix would be to fetch a ref from
> somewhere. OTOH, I hope this isn't going to be a recurring problem.

--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...

Tomi

> d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


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 an email to notmuch-le...@notmuchmail.org


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)
>> both seem to have problem with base64 encoded patches documented at.
>> 
>>  https://lore.kernel.org/git/20191218194144.gl17...@pobox.com/
>> 
>> It seems that the patches have CRLF line endings inside the base64
>> encoding, and git fails to strip the CRs.
>> 
>> I'm not sure why this happens for some patches and not for others;
>> I hope there is not re-encoding happening on the mailing list. If there
>> is we should fix that.
>
> It seems like the mailing list base64-encodes utf-8 email.
> My email will be sent in utf-8 for obvious reason.
>
> I don't know about Jonas's, some of them are in utf-8 and some other
> is in plain ascii.

That seems to be it.  I don't remember the exact question but when
sending I was asked about what encoding to use and I confirmed the
default, which was utf-8.  Usually I am not asked this question.

 Jonas
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


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 encoded patches documented at.
> 
>  https://lore.kernel.org/git/20191218194144.gl17...@pobox.com/
> 
> It seems that the patches have CRLF line endings inside the base64
> encoding, and git fails to strip the CRs.
> 
> I'm not sure why this happens for some patches and not for others;
> I hope there is not re-encoding happening on the mailing list. If there
> is we should fix that.

It seems like the mailing list base64-encodes utf-8 email.
My email will be sent in utf-8 for obvious reason.

I don't know about Jonas's, some of them are in utf-8 and some other
is in plain ascii.


-- 
Danh
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


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.

 https://lore.kernel.org/git/20191218194144.gl17...@pobox.com/

It seems that the patches have CRLF line endings inside the base64
encoding, and git fails to strip the CRs.

I'm not sure why this happens for some patches and not for others;
I hope there is not re-encoding happening on the mailing list. If there
is we should fix that.

The practical upshot is that I can't really apply such a long series by
hand. Ideas welcome. A short term fix would be to fetch a ref from
somewhere. OTOH, I hope this isn't going to be a recurring problem.

d
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


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 looks good to me.

Great!

>> That resulted in a few new commits of course, marked with * below.
>> And since I had to add some commit anyway, I also added three other
>> minor commits, marked with ^ below.
>
> I checked the new commits, looks good to me. I also was preparing dropping
> emacs 23 support code and the changes aligned what I tried. Interestinglyy
> this change in notmuch-mua.el:
>   ;; There is a bug in emacs 23's message.el that results in a newline
>   ;; not being inserted after the References header, so the next header
>   ;; is concatenated to the end of it. 
> Is still effective in emacs 26.3 -- I wonder whether there is something
> wrong notmuch-emacs handles the case `notmuch-mua-insert-references` is
> needed.

I'll have a look.

> I trust the earlier changes are the same as in v2 and v1

Almost. In "emacs: Increase consistency of library headers" I did bring
the headers of "notmuch-compat.el" and "emacs/notmuch-version.el.tmpl"
more in line.  But that's it.

 Jonas
___
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-le...@notmuchmail.org


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 either of these options. I'd hope we can specify what
>> versions we need in emacs/notmuch-pkg.el.templ
>
> I went with option 1: drop support for emacs 24.  Instead of just
> adding a NEWS entry and bump a number, I also removed some old cruft
> that was only necessary for emacs 24 (and in some cases emacs 23!).

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 looks good to me.

> That resulted in a few new commits of course, marked with * below.
> And since I had to add some commit anyway, I also added three other
> minor commits, marked with ^ below.

I checked the new commits, looks good to me. I also was preparing dropping
emacs 23 support code and the changes aligned what I tried. Interestinglyy
this change in notmuch-mua.el:
  ;; There is a bug in emacs 23's message.el that results in a newline
  ;; not being inserted after the References header, so the next header
  ;; is concatenated to the end of it. 
Is still effective in emacs 26.3 -- I wonder whether there is something
wrong notmuch-emacs handles the case `notmuch-mua-insert-references` is
needed.

I trust the earlier changes are the same as in v2 and v1

Tomi


>
>  Cheers,
>  Jonas
>
> Jonas Bernoulli (34):
>   emacs: Shorten long lines
>   emacs: Remove excess empty lines
>   emacs: Fix indentation
>   emacs: Closing parenthesis go on the same line
>   emacs: Only set one variable per setq form
>   emacs: Use 'and' instead of 'when' when the return value matters
>   emacs: Use 'unless' instead of 'when' and 'not'
>   emacs: Use 'when' instead of 'if' when there is no ELSE part
>   emacs: Use one or three lines for 'if' forms
>   emacs: Extend face to window edge again
>   emacs: Fix some function declarations
>   emacs: No longer define notmuch-hello-mode-map as a function
>   emacs: notmuch-poll: Let the user know we are polling
>   emacs: Use makefile-gmake-mode in Makefile*s
>   emacs: Improve doc-strings
>   emacs: Autoload notmuch-jump-search only once
>   emacs: Autoload notmuch-jump using an autoload cookie
>   emacs: Various cosmetic changes
>   emacs: Increase consistency of library headers
>   Fix typos
>   .dir-locals.el: Set variables for correct "shell" mode
>   test: Fix indentation
> ^ .gitignore: Sort using sort-lines
> ^ emacs: Provide 'rstdoc' feature at end of file
> ^ emacs: Add end-of-file line to libraries that lack it
> * NEWS: Add stub for 0.31
> * NEWS: At least Emacs 25.1 is required now
> * emacs: Use cl-incf where appropriate
> * emacs: Remove notmuch-setq-local
> * emacs: Remove notmuch-read-char-choice
> * emacs: Drop old advices that were only need for Emacs 23
> * emacs: Do not abuse advice to monkey patch while testing
> * emacs: Use new advice mechanism do advice mm-shr
> * try-emacs-mua: Trim `require' advice for Emacs 25
>
>  .dir-locals.el |   2 +-
>  .gitignore |  28 +-
>  Makefile.global|   1 +
>  Makefile.local |   2 +-
>  NEWS   |  10 +-
>  bindings/Makefile.local|   2 +-
>  bindings/python-cffi/notmuch2/__init__.py  |   2 +-
>  bindings/python-cffi/notmuch2/_base.py |   6 +-
>  bindings/python-cffi/notmuch2/_database.py |   8 +-
>  bindings/python-cffi/notmuch2/_message.py  |   4 +-
>  bindings/python-cffi/notmuch2/_tags.py |   8 +-
>  bindings/python-cffi/tests/conftest.py |   2 +-
>  bindings/python/notmuch/database.py|  12 +-
>  bindings/python/notmuch/query.py   |   2 +-
>  compat/Makefile.local  |   2 +-
>  completion/Makefile.local  |   2 +-
>  devel/try-emacs-mua|  26 +-
>  doc/Makefile.local |   2 +-
>  emacs/Makefile.local   |   2 +-
>  emacs/coolj.el |  24 +-
>  emacs/make-deps.el |   7 +-
>  emacs/notmuch-address.el   | 188 -
>  emacs/notmuch-company.el   |  56 +--
>  emacs/notmuch-compat.el|  88 +
>  emacs/notmuch-crypto.el|  49 ++-
>  emacs/notmuch-draft.el |  24 +-
>  emacs/notmuch-hello.el | 102 +++--
>  emacs/notmuch-jump.el  |  61 ++-
>  emacs/notmuch-lib.el   | 202 +-
>  emacs/notmuch-maildir-fcc.el   | 122 +++---
>  emacs/notmuch-messa

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

2020-08-08 Thread Jonas Bernoulli
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 either of these options. I'd hope we can specify what
> versions we need in emacs/notmuch-pkg.el.templ

I went with option 1: drop support for emacs 24.  Instead of just
adding a NEWS entry and bump a number, I also removed some old cruft
that was only necessary for emacs 24 (and in some cases emacs 23!).

That resulted in a few new commits of course, marked with * below.
And since I had to add some commit anyway, I also added three other
minor commits, marked with ^ below.

 Cheers,
 Jonas

Jonas Bernoulli (34):
  emacs: Shorten long lines
  emacs: Remove excess empty lines
  emacs: Fix indentation
  emacs: Closing parenthesis go on the same line
  emacs: Only set one variable per setq form
  emacs: Use 'and' instead of 'when' when the return value matters
  emacs: Use 'unless' instead of 'when' and 'not'
  emacs: Use 'when' instead of 'if' when there is no ELSE part
  emacs: Use one or three lines for 'if' forms
  emacs: Extend face to window edge again
  emacs: Fix some function declarations
  emacs: No longer define notmuch-hello-mode-map as a function
  emacs: notmuch-poll: Let the user know we are polling
  emacs: Use makefile-gmake-mode in Makefile*s
  emacs: Improve doc-strings
  emacs: Autoload notmuch-jump-search only once
  emacs: Autoload notmuch-jump using an autoload cookie
  emacs: Various cosmetic changes
  emacs: Increase consistency of library headers
  Fix typos
  .dir-locals.el: Set variables for correct "shell" mode
  test: Fix indentation
^ .gitignore: Sort using sort-lines
^ emacs: Provide 'rstdoc' feature at end of file
^ emacs: Add end-of-file line to libraries that lack it
* NEWS: Add stub for 0.31
* NEWS: At least Emacs 25.1 is required now
* emacs: Use cl-incf where appropriate
* emacs: Remove notmuch-setq-local
* emacs: Remove notmuch-read-char-choice
* emacs: Drop old advices that were only need for Emacs 23
* emacs: Do not abuse advice to monkey patch while testing
* emacs: Use new advice mechanism do advice mm-shr
* try-emacs-mua: Trim `require' advice for Emacs 25

 .dir-locals.el |   2 +-
 .gitignore |  28 +-
 Makefile.global|   1 +
 Makefile.local |   2 +-
 NEWS   |  10 +-
 bindings/Makefile.local|   2 +-
 bindings/python-cffi/notmuch2/__init__.py  |   2 +-
 bindings/python-cffi/notmuch2/_base.py |   6 +-
 bindings/python-cffi/notmuch2/_database.py |   8 +-
 bindings/python-cffi/notmuch2/_message.py  |   4 +-
 bindings/python-cffi/notmuch2/_tags.py |   8 +-
 bindings/python-cffi/tests/conftest.py |   2 +-
 bindings/python/notmuch/database.py|  12 +-
 bindings/python/notmuch/query.py   |   2 +-
 compat/Makefile.local  |   2 +-
 completion/Makefile.local  |   2 +-
 devel/try-emacs-mua|  26 +-
 doc/Makefile.local |   2 +-
 emacs/Makefile.local   |   2 +-
 emacs/coolj.el |  24 +-
 emacs/make-deps.el |   7 +-
 emacs/notmuch-address.el   | 188 -
 emacs/notmuch-company.el   |  56 +--
 emacs/notmuch-compat.el|  88 +
 emacs/notmuch-crypto.el|  49 ++-
 emacs/notmuch-draft.el |  24 +-
 emacs/notmuch-hello.el | 102 +++--
 emacs/notmuch-jump.el  |  61 ++-
 emacs/notmuch-lib.el   | 202 +-
 emacs/notmuch-maildir-fcc.el   | 122 +++---
 emacs/notmuch-message.el   |   3 +-
 emacs/notmuch-mua.el   | 185 -
 emacs/notmuch-parser.el|  16 +-
 emacs/notmuch-pkg.el.tmpl  |   3 +-
 emacs/notmuch-print.el |   2 +-
 emacs/notmuch-query.el |  29 +-
 emacs/notmuch-show.el  | 439 ++---
 emacs/notmuch-tag.el   |  43 +-
 emacs/notmuch-tree.el  | 189 -
 emacs/notmuch-version.el.tmpl  |   3 +-
 emacs/notmuch-wash.el  | 109 +++--
 emacs/notmuch.el   | 278 ++---
 emacs/rstdoc.el|  22 +-
 lib/Makefile.local |   2 +-
 lib/notmuch.h  |   4 +-
 parse-time-string/Makefile.local   |   2 +
 performance-test/Makefile.local|   2 +-
 tag-util.c |   2 +-
 tag-util.h |   2 +-
 test/Makefile.local|