Following up.

On Jun 4, 2014, at 9:53 AM, Chuck Lever <[email protected]> wrote:

> Hi Catalin-
> 
> On Jun 4, 2014, at 5:50 AM, Catalin Marinas <[email protected]> wrote:
> 
>> On 4 June 2014 00:04, Chuck Lever <[email protected]> wrote:
>>> I recently noticed that after "stg rebase", the commit date and author
>>> date in all patches in the rebased branch are now the same. How can
>>> I retain the original author date when performing stgit operations?
>> 
>> I think this needs fixing in stgit, there is no option. Basically
>> refresh assumes that the author date needs to be updated to the latest
>> and this also has implications on rebase. Something like below,
>> untested, but we need to assess what other cases are affected:
>> 
>> diff --git a/stgit/stack.py b/stgit/stack.py
>> index 9cd43a3e8809..15ab1879ef6e 100644
>> --- a/stgit/stack.py
>> +++ b/stgit/stack.py
>> @@ -706,6 +706,8 @@ class Series(PatchSet):
>>            author_name = patch.get_authname()
>>        if not author_email:
>>            author_email = patch.get_authemail()
>> +        if not author_date:
>> +            author_date = patch.get_authdate()
>>        if not committer_name:
>>            committer_name = patch.get_commname()
>>        if not committer_email:
> 
> I did a little comparison, fwiw.
> 
>  git://git.linux-nfs.org/projects/cel/fedfs-utils.git
> 
> I manage this repo from a newer system:
> 
> [cel@seurat ~]$ stg --version
> Stacked GIT 0.16
> git version 1.8.3.1
> Python version 2.7.5 (default, Nov 12 2013, 16:18:42) 
> [GCC 4.8.2 20131017 (Red Hat 4.8.2-1)]
> [cel@seurat ~]$
> 
> It appears to retain unique author dates.
> 
> This repo:
> 
>  git://git.linux-nfs.org/projects/cel/cel-2.6.git
> 
> I’m managing currently from OL6 using the stgit packaged
> in EPEL:
> 
> [cel@manet linux-2.6]$ stg --version
> Stacked GIT 0.14.3
> git version 1.7.1
> Python version 2.6.6 (r266:84292, Jan 22 2014, 01:49:05) 
> [GCC 4.4.7 20120313 (Red Hat 4.4.7-4)]
> [cel@manet linux-2.6]$
> 
> A cursory check against www.kernel.org shows the author dates
> in my “master” branch match master upstream. But in the
> nfs-rdma-client branch, which I maintain, authdates all appear
> to be the same as the commit date.
> 
> The “for-3.12” branch in this repo was managed with a version
> of git/stgit similar to the fedfs-utils repo above, and the
> authdates appear to be preserved.

I updated git and stgit on my current OL6 test system to match what
I have on the Fedora 19 system. stgit is now version 0.16.

These subcommands now preserve authdate: refresh, push, pop

These subcommands do not preserve authdate: pick -B, rebase

stg import is not working for other reasons, so I couldn’t test it.
It seems to get crabby about importing raw e-mail from Mail.app.
Ideally it should pick up the authdate from the e-mail’s Date: header.

I applied the patch you suggested above. No change for my test cases.

--
Chuck Lever
chuck[dot]lever[at]oracle[dot]com


_______________________________________________
stgit-users mailing list
[email protected]
https://mail.gna.org/listinfo/stgit-users

Reply via email to