Re: [PATCH v5 5/6] rebase -i: support --ignore-date

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

 



Junio C Hamano <gitster@xxxxxxxxx> writes:

> Rohit Ashiwal <rohit.ashiwal265@xxxxxxxxx> writes:
>
>>  --ignore-date::
>> +	Instead of using the given author date, reset it to the
>> +	current time. This implies --force-rebase.
>
> The first sentence is unclear and puzzles me with "for what?".  IOW,
> you are saying that by default the command uses the given (I presume
> that you meant what is recorded in the original commits being
> rebased---but we should find a way to explain it more clearly)
> author date, but with this option what is used is reset to the
> current time.  It is left unspecified what the command is using that
> time for.
>
> Perhaps (I am writing this paragraph after writing the rest of the
> message, i.e. after reading the patch through):

Oops, too much re-editing without final proofreading.  Sorry.

> 	By default, the author date recorded in the commit being
> 	rebased as the author date is used as the author date of the
> 	rebased commits.  The option tells Git to use the current
> 	timestamp as the author date of the rebased commits instead.
> 	This implies `--force-rebase`.

	By default, the author date of the original commit is used
	as the author date for the resulting commit.  This option
	tells Git to use the current timestamp instead and implies
	`--force-rebase`.

would be much less redundant without losing clarity.



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

  Powered by Linux