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: Thu, 17 Jul 2014 06:28:55 -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'
>
> if they run 'nmbug status' without an @upstream.
>
> I'm working through the Python translation now ;).

Perhaps the best thing is to revert that patch then?

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]