[StGit] import --mail/--mbox question

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

[StGit] import --mail/--mbox question

From: Shinya Kuribayashi <hidden>
Date: 2016-06-15 22:45:57

Hi,

I've been wondering this for quite some time, now try to sort out.

---

Question: when importing Mozilla Thunderbird mails (eml or mbox), the
imported patch always have committer's date in git log, not submitter's
date.  However, if importing the same mails with git am, we could see
submitter's date in git log.

Is this intentionally-designed log management of StGit?  I would expect
the submitter's date & locale is kept when importing patches from
e-mails.

Reproducible: Always

Steps to Reproduce:

1. Prepare eml or mbox file.

  [Thunderbird eml file]
   - Select a mail,
   - File -> Save as -> File, 
   - Format -> Mail files

  [mbox file]
   - use add-ons, such as ImportExportTools(MboxImport)
   - extract directly from profile/ dir

2. Import eml/mbox file with --mail/--mbox option

  $ stg import --mail [eml file]
  $ stg import --mbox [mbox file]

3. Check git log

Example:

  Mbox header (this mbox contains one patch)
  ------------------------------------------
  From - Wed Jan  7 00:41:57 2009
  X-Account-Key: account3
  X-UIDL: 6d997fc0c53b4541c56eeeeb45732171
  X-Mozilla-Status: 0001
  X-Mozilla-Status2: 00000000
  X-Mozilla-Keys:                                                                                 X-DTI-Virus-Check: checked
  X-DTI-Recipient: [off-list ref]
  Return-Path: [off-list ref]
  Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.188]) by pop.ruby.dti.ne.jp (3.10p) with ESMTP id n06AZr2v009835 for [off-list ref]; Tue, 6 Jan 2009 19:35:53 +0900 (JST)
  Received: from localhost.localdomain (p5B2A461A.dip.t-dialin.net [91.42.70.26])
          by mrelayeu.kundenserver.de (node=mrelayeu1) with ESMTP (Nemesis)
          id 0MKwpI-1LK9Hn0ym1-0002Ja; Tue, 06 Jan 2009 11:35:51 +0100
  From: Stefan Roese [off-list ref]
  To: u-boot@lists.denx.de
  Cc: skuribay@ruby.dti.ne.jp
  Subject: [PATCH 1/3 v2] MIPS: Add VCT board series support (Part 1/3)
  Date: Tue,  6 Jan 2009 11:35:46 +0100
  Message-Id: [off-list ref]
  X-Mailer: git-send-email 1.6.1
  X-Provags-ID: V01U2FsdGVkX19dTpfkGdmBDPbfKKBfEQYFLGbts7teQcJyLTZ
   xUdZWxGujKNfCpOIRv/s7i7nrf13E4zGP6P0nibSP6yFfDLBv3
   0qZDbTBuPI1ViEQrnIGZCASl9Z6H8em

  This patch adds support for the Micronas VCT board series.
  Currently the following platforms are supported:
  :
  :

  Resulting git log
  -----------------
  commit b659910dd0d75220c3cdbd3408a5025340e6a562
  Author: Stefan Roese [off-list ref]
  Date:   Thu Jan 15 00:06:27 2009 +0900 <-- this is my date & locale(JP)

      MIPS: Add VCT board series support (Part 1/3)

      This patch adds support for the Micronas VCT board series.
      Currently the following platforms are supported:

        vct_premium
        vct_premium_small
        vct_premium_onenan
  :
  :

Version:

  Stacked GIT 0.14.3.327.ge4f6.dirty
  git version 1.5.4.3
  Python version 2.5.2 (r252:60911, Jul 31 2008, 17:28:52) 
  [GCC 4.2.3 (Ubuntu 4.2.3-2ubuntu7)]


Thanks in advance,

  Shinya

Re: [StGit] import --mail/--mbox question

From: Karl Hasselström <hidden>
Date: 2016-06-15 22:45:57

On 2009-01-15 00:35:50 +0900, Shinya Kuribayashi wrote:
Question: when importing Mozilla Thunderbird mails (eml or mbox),
the imported patch always have committer's date in git log, not
submitter's date. However, if importing the same mails with git am,
we could see submitter's date in git log.

Is this intentionally-designed log management of StGit? I would
expect the submitter's date & locale is kept when importing patches
from e-mails.
Hmm. I agree with you that the expected behavior when importing
patches from e-mails is that the author date is set to the date in the
e-mail, the way git-am does it.

Looking at the code, it looks like the intention is that it should
work that way, too. So I guess it's simply a bug. Catalin, do you have
a clue?

-- 
Karl Hasselström, kha@treskal.com
      www.treskal.com/kalle
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help