Re: [RFC] fast-import: note deletion command

From: Sverre Rabbelier <srabbelier@gmail.com>
Date: Sep 18, 2011, 20:39 UTC
Message-ID: <CAGdFq_hpA95Kj4eMr4e1duuXTpr+OkkwF4K5bTapXEi9UjWcSA@mail.gmail.com>
In-Reply-To: <20110918203506.GG2308@elie>

Heya,

On Sun, Sep 18, 2011 at 22:35, Jonathan Nieder <jrnieder@gmail.com> wrote:

Show 14 quoted lines

>> ND commit_sha1 # One may think there are also NC, NR, NM
>> N delete commit_sha1
>> # Only :mark, full-40-byte-sha1 and 'inline' are allowed currently.
>> # So no clashes arise, but then one may also want M 'delete' path
>> # command to work too.
>
> If I were doing it:
>
>  - advertise "N 0{40} <commit>" as the historical way to delete a
>   note, and make sure it keeps working
>
>  - introduce 'ND' or 'notedelete <commit>' as a human-friendly
>   synonym, keeping in mind that NC and NR could be introduced in the
>   future if there is demand for them.

Is this perhaps a good moment to also think about branch deletion? That came up earlier as well, and thinking about that might give us some insights in how to deal with deletions uniformly.

-- 
Cheers,

Sverre Rabbelier