Re: [PATCH 2/4] nmbug: Handle missing @upstream in is

Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged

Subject: Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged

Date: Sat, 19 Jul 2014 09:34:31 -0300

"W. Trevor King" <wking@tremily.us> writes:

> On Wed, Jul 16, 2014 at 07:36:10PM -0300, David Bremner wrote:
>> W. Trevor King writes:
>> > If we don't have an upstream, there is nothing to merge, so
>> > nothing is unmerged.  This avoids errors like:
>> 
>> pushed this one patch.
>
> Without the stderr-catching of something like patch 3, this means
> folks without an upstream are going to see a distracting:
>
>   error: No upstream configured for branch 'master'
>

On reflection, I don't think that patch made things worse for this use
case. So I don't think it really hurts to leave it there for now.

d

Previous message (by thread): [PATCH] potential fix for nmbug merge problems

Thread:

W. Trevor King—[PATCH 0/4] nmbug without an upstream repository (and init) [inbox, unread]
- W. Trevor King—[PATCH 1/4] nmbug: Add a git_with_status helper function [inbox, notmuch::patch, notmuch::pushed, unread]
  - David Bremner—Re: [PATCH 1/4] nmbug: Add a git_with_status helper function [inbox, unread]
- W. Trevor King—[PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, notmuch::patch, notmuch::pushed, unread]
  - David Bremner—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, unread]
    - W. Trevor King—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, signed, unread]
  - David Bremner—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, unread]
    - W. Trevor King—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, signed, unread]
      - David Bremner—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, unread]
        W. Trevor King—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, signed, unread]
      - David Bremner—Re: [PATCH 2/4] nmbug: Handle missing @upstream in is_unmerged [inbox, unread]
- W. Trevor King—[PATCH 3/4] nmbug: Catch stderr in is_unmerged [inbox, notmuch::obsolete, notmuch::patch, unread]
  - David Bremner—Re: [PATCH 3/4] nmbug: Catch stderr in is_unmerged [inbox, unread]
    - W. Trevor King—Re: [PATCH 3/4] nmbug: Catch stderr in is_unmerged [inbox, signed, unread]
      - David Bremner—Re: [PATCH 3/4] nmbug: Catch stderr in is_unmerged [inbox, unread]
- W. Trevor King—[PATCH 4/4] nmbug: Add an 'init' command [inbox, notmuch::obsolete, notmuch::patch, unread]
  - David Bremner—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, unread]
    - W. Trevor King—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, signed, unread]
  - David Edmondson—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, unread]
    - W. Trevor King—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, signed, unread]
      - David Edmondson—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, unread]
        W. Trevor King—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, signed, unread]
        David Bremner—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, unread]
        W. Trevor King—Re: [PATCH 4/4] nmbug: Add an 'init' command [inbox, signed, unread]
        David Bremner—[PATCH] potential fix for nmbug merge problems [inbox, unread]