[PATCH] [mq:qrefresh] update help for --current-date

Peter Arrenbrecht peter.arrenbrecht at gmail.com
Tue Aug 25 00:43:02 CDT 2009


On Mon, Aug 24, 2009 at 9:30 PM, Yann E.
MORIN<yann.morin.1998 at anciens.enib.fr> wrote:
> Peter, Martin,
> All,
>
> On Thursday 20 August 2009 15:25:10 Peter Arrenbrecht wrote:
>> On Mon, Aug 3, 2009 at 10:57 AM, Martin Geisler<mg at lazybytes.net> wrote:
> [--SNIP code snipet--]
>> > It's clear that setdate is missing out on something :-)
>> Which is (was?) deliberate. The idea was that [defaults] qrefresh=-D
>> should not add dates to patches that didn't have them.
>
> I can understand that. But following this idea, why does:
>  [defaults]
>  qrefresh = -U
> add the user (From:) to patches that don't have it?
>
> What was odd is that -D and -U do not behave the same. It seemed there was
> some consensus that, if adding the user was appropriate, then adding the
> date was appropriate, too. Thus the patch <series.

The thinking was that when revising a patch, you typically want the
embedded date auto-updated, if there is one. On the other hand,
updating the user should be a manual operation because a minor tweak
to a patch does not warrant changing the user. Hence [defaults]
qrefresh=-D but not -U.
-parren



More information about the Mercurial-devel mailing list