Re: [PATCH 1/2] pull: handle git-fetch's options as well

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Mon, May 18, 2015 at 10:37 PM, Johannes Schindelin
<johannes.schindelin@xxxxxx> wrote:
> On 2015-05-18 15:30, Paul Tan wrote:
>> t5520: failing test for pull --all with no configured upstream
>> t5521: test pull --all --dry-run does not make any changes
>>
>> error_on_no_merge_candidates() does not consider the case where "$#"
>> includes command-line flags that are passed to git-fetch.
>>
>> As such, when the current branch has no configured upstream, and there
>> are no merge candidates because of that, git-pull --all erroneously reports
>> that we are pulling from "--all", as it believes that the first argument
>> is the remote name.
>>
>> Add a failing test that shows this case.
>>
>> Signed-off-by: Paul Tan <pyokagan@xxxxxxxxx>
>> ---
>>  git-pull.sh             | 34 ++++++++++++++++++++++++++++++++--
>>  t/t5520-pull.sh         | 20 ++++++++++++++++++++
>>  t/t5521-pull-options.sh | 14 ++++++++++++++
>>  3 files changed, 66 insertions(+), 2 deletions(-)
>>
>> diff --git a/git-pull.sh b/git-pull.sh
>> index 9ed01fd..28d49ab 100755
>> --- a/git-pull.sh
>> +++ b/git-pull.sh
>> @@ -163,11 +163,39 @@ do
>>       --d|--dr|--dry|--dry-|--dry-r|--dry-ru|--dry-run)
>>               dry_run=--dry-run
>>               ;;
>> +     --all|--no-all)
>> +             all=$1 ;;
>
> I *think* you also want to add a corresponding "all=" line just below the "dry_run=" line, to ensure that "all=blablabla git pull" does not interfere with this command-line setting.

Fixed, thanks.

Regards,
Paul
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]