Re: [PATCHv2 6/7] builtin/describe.c: describe a blob

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

Subject: Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
From: Andreas Schwab <schwab@xxxxxxxxxxxxxx>
Date: Thu, 02 Nov 2017 08:23:49 +0100
Cc: Stefan Beller <sbeller@xxxxxxxxxx>, Jacob Keller <jacob.keller@xxxxxxxxx>, Junio C Hamano <gitster@xxxxxxxxx>, git <git@xxxxxxxxxxxxxxx>, Kevin Daudt <me@xxxxxxxxx>
In-reply-to: <alpine.DEB.2.21.1.1711012340290.6482@virtualbox> (Johannes Schindelin's message of "Wed, 1 Nov 2017 23:41:45 +0100 (CET)")
References: <20171031003351.22341-1-sbeller@google.com> <20171031211852.13001-1-sbeller@google.com> <20171031211852.13001-7-sbeller@google.com> <xmqqbmkmvdrq.fsf@gitster.mtv.corp.google.com> <alpine.DEB.2.21.1.1711011329300.6482@virtualbox> <CAGZ79kZPNiNnSJd6CNYb7XkTVsT2ONLQLhwAQxt6_SPFTSwMcw@mail.gmail.com> <EF573E5E-EA72-4DEE-822C-B44265FD581B@gmail.com> <alpine.DEB.2.21.1.1711012310250.6482@virtualbox> <CAGZ79kbOEM_W65Rym4yiDNHpFGTNWMYdh=aVPjThNWjEHPQong@mail.gmail.com> <alpine.DEB.2.21.1.1711012340290.6482@virtualbox>
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.3 (gnu/linux)

On Nov 01 2017, Johannes Schindelin <Johannes.Schindelin@xxxxxx> wrote:

> Sure, but it is still a tricky thing. Imagine
>
> - A1 - B1 - A2 - B2 - B3
>
> where all the B* commits have the blob. Do you really want to report B1
> rather than B2 as the commit introducing the blob? (I would prefer B2...)

What if B3 renames or copies the blob?

Andreas.

-- 
Andreas Schwab, schwab@xxxxxxxxxxxxxx
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."

Follow-Ups:
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Stefan Beller

References:
- [PATCH 0/7] git-describe <blob>
  - From: Stefan Beller
- [PATCHv2 0/7] git describe blob
  - From: Stefan Beller
- [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Stefan Beller
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Junio C Hamano
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Johannes Schindelin
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Stefan Beller
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Jacob Keller
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Johannes Schindelin
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Stefan Beller
- Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
  - From: Johannes Schindelin

Prev by Date: Re: On the nature of cover letters [WAS Re: [PATCH 0/6] Create Git/Packet.pm]
Next by Date: Re: On the nature of cover letters [WAS Re: [PATCH 0/6] Create Git/Packet.pm]
Previous by thread: Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
Next by thread: Re: [PATCHv2 6/7] builtin/describe.c: describe a blob
Index(es):
- Date
- Thread

[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]