]> git.notmuchmail.org Git - notmuch/commit
nmbug: Only error for invalid diff lines in tags/
authorW. Trevor King <wking@tremily.us>
Mon, 16 Oct 2017 18:01:47 +0000 (11:01 -0700)
committerDavid Bremner <david@tethera.net>
Sat, 16 Dec 2017 12:20:13 +0000 (08:20 -0400)
commit572259885af4d5858c3be5c2119ec7019e1ca617
tree2c22bf60c13f6f853ba3a4c26e2db5b52315e105
parent7354d6d8f6fe413097e4fe3dc450594f016334a9
nmbug: Only error for invalid diff lines in tags/

Avoid:

  Traceback (most recent call last):
    File "/home/nmbug/bin/nmbug", line 834, in <module>
      args.func(**kwargs)
    File "/home/nmbug/bin/nmbug", line 385, in checkout
      status = get_status()
    File "/home/nmbug/bin/nmbug", line 580, in get_status
      maybe_deleted = _diff_index(index=index, filter='D')
    File "/home/nmbug/bin/nmbug", line 658, in _diff_index
      for id, tag in _unpack_diff_lines(stream=p.stdout):
    File "/home/nmbug/bin/nmbug", line 678, in _unpack_diff_lines
      'Invalid line in diff: {!r}'.format(line.strip()))
  ValueError: Invalid line in diff: u'.mailmap'

With this commit, folks can commit READMEs, .mailmap, etc. to their
nmbug repositories, and 'nmbug diff' and 'status' won't choke on them.
If you want to check for this sort of thing, you can set --log-level
to info or greater.  nmbug will still error if the unrecognized path
is under tags/, since that's more likely to be a user error.
devel/nmbug/nmbug