DragonFly BSD
DragonFly bugs List (threaded) for 2010-10
[Date Prev][Date Next]  [Thread Prev][Thread Next]  [Date Index][Thread Index]

Re: [issue1863] Implement 'hammer volume-list' subcommand


From: Aggelos Economopoulos <aoiko@xxxxxxxxxxxxxx>
Date: Wed, 20 Oct 2010 08:39:07 +0200

On 10/20/2010 02:00 AM, Sascha Wildner wrote:
> On 10/19/2010 23:22, Aggelos Economopoulos wrote:
>> On 10/19/2010 10:59 PM, Stathis Kamperis (via DragonFly issue tracker)
>> wrote:
>>>
>>> Stathis Kamperis<ekamperi@gmail.com>  added the comment:
>>>
>>> New patch, same url:
>>> http://stathisk.ath.cx/0001-HAMMER-Implement-volume-list-command.patch
>>
>> Can you explain why you use the 0001- prefix if you're going to use the
>> same url over and over again? It's just extra pain for us to rename your
>> patches, which is something I did, so now I can do the following, in
>> order to only review your changes relative to the last patch.
>> [...]
> 
> fetch -o ...

Yes, that is the renaming I was refering to. The fact that it is  easy
to do does not address the problems I pointed out, namely that it is
pointless to have a serial counter prefix and not increment it.
Addditionally, rewriting the patch means that it is the reviewers who
need to bother with keeping history (which, as demonstrated, *is*
useful) and of course I can no longer refer unambiguously to version 1
of the patch (My local numbering after the renames is 0-2. What's yours?).

Aggelos



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