From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f67.google.com (mail-wr1-f67.google.com [209.85.221.67]) by mx.groups.io with SMTP id smtpd.web12.3656.1579210485029336033 for ; Thu, 16 Jan 2020 13:34:45 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=Fm31A8gE; spf=pass (domain: linaro.org, ip: 209.85.221.67, mailfrom: leif.lindholm@linaro.org) Received: by mail-wr1-f67.google.com with SMTP id z7so20619535wrl.13 for ; Thu, 16 Jan 2020 13:34:44 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=OxB6wwnvPO59tjJEu+FnJPXM1FhvTq8+dqXCY7CFZdQ=; b=Fm31A8gEpQOGOW7npWhsgrVpCEdfxfx713PidK8j2WrsRky50L+ewY+61ZxAY5mPMo 5YwD9nhhsbXEoQcANCk9scJPzg5UMI+6Ay3/ku1jgV1f2awCJLAGvdaIMBetcAOSBRee UZIAgJ5iS9mjGuwJ2DI2GuoD6Up5Q17GuUPiQwYxjJFf/q86JG4kD1kIXPClVVsL1Hqo /Cj11P9eO6k8nnMAmU3F7x4K1CbBNhbYEUPnxapB/kh93u+M7ZeiiUbRuCNT0zleV5ME IE9gwl2EGNO2hJnHddVQJP+Mm6kwqP8yZl7cI+xc/9GF5CDzcEFFbDqsQ4En59l4Aazw 58BQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=OxB6wwnvPO59tjJEu+FnJPXM1FhvTq8+dqXCY7CFZdQ=; b=XRwnBMeV6B2GEplZg6nEDJlKGTjjvnuUULh70oEUChI0Px2S9D1lE/s2YpwBfJNY+6 AF+211hpWL6oqG6gAGecO1fpY4ZRVvLmDLGY29yjci3rS8OEfuRL+a1NaC+/mj4nGpiJ +uMyV07sMb0W6x2jL+wDF654BRCxYTrezD9SzmuQwcUQxH/nvRiPRiaDT86RsycQegG+ JtX3pnfUNoZ93PeiUItzxK1Ly6PBYDYKO6ZpALrcqJfKCT+dLbrmXG4BcF+QEPzahgFj hfjkIejf2Z5GAhdB1I/4ncXrnsubFzo2Gl3KXRoi99nGGNGM/acGpKmv0l+wo2t5lnlY 7AIg== X-Gm-Message-State: APjAAAWlqjDLruq99GZn+KE8QNhDwHZgyrC+/GxchFKZT16jtVXSMDAD TcTLsaZnWpvzvFJ3wGmacvdK8A== X-Google-Smtp-Source: APXvYqxhOxuEifc+Vo5/w32B0qz620RcsE/HZaHgYi0qAcyqOVa7T504DPBTiTW/C+dJ9rYpQ1yNzQ== X-Received: by 2002:a5d:6a02:: with SMTP id m2mr5313223wru.52.1579210483451; Thu, 16 Jan 2020 13:34:43 -0800 (PST) Return-Path: Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id m3sm29985122wrs.53.2020.01.16.13.34.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 16 Jan 2020 13:34:42 -0800 (PST) Date: Thu, 16 Jan 2020 21:34:41 +0000 From: Leif Lindholm To: Philippe =?iso-8859-1?Q?Mathieu-Daud=E9?= Cc: Laszlo Ersek , devel@edk2.groups.io, Andrew Fish , Ard Biesheuvel , Michael D Kinney , Ray Ni , Zhichao Gao , Leif Lindholm Subject: Re: [PATCH 1/2] Maintainers.txt: update email address for Leif Lindholm Message-ID: <20200116213441.GE20629@bivouac.eciton.net> References: <20200114163255.10681-1-leif.lindholm@linaro.org> <20200114163255.10681-2-leif.lindholm@linaro.org> <119a8e77-6472-9be6-b12e-3bfe8e4e83b5@redhat.com> <9b4c40fe-b15c-2b44-c3f0-70a4b7f1dfc8@redhat.com> MIME-Version: 1.0 In-Reply-To: <9b4c40fe-b15c-2b44-c3f0-70a4b7f1dfc8@redhat.com> User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit On Thu, Jan 16, 2020 at 19:54:43 +0100, Philippe Mathieu-Daudé wrote: > > Phil, would it make sense for us to ask Leif to post an update to > > ".mailmap"? > > > > For example, what happens if we run "git shortlog" over a period that > > contains patches authored by *both* of Leif's email addresses? Would > > those entries be merged into a single block? Would such a merging be > > desirable (that's for Leif to decide / propose)? > > One use of .mailmap is to clean/unify the NAME part. > In this case Leif isn't affected because his NAME is the same. > > The other use is to redirect email when a user switch email. When > fixing/referring old commit, some email tools might take the old commit > author/committer. In this case .mailmap helps. I think it is desirable in > Leif particular case. The question is *why* we use the mailmap to begin with. It is not clear to me that there was an understanding at the point .mailmap was added that the intent was to provide a mapping of people to current email addresses in perpetuity. Indeed, the initial commit message says: "The .mailmap git feature helps fixing commit mistakes (in name/email)." The comment header in the file itself says: # This list is used by git-shortlog to fix a few botched name translations # in the git archive, either because the author's full name was messed up # and/or not always written the same way, making contributions from the # same person appearing not to be so or badly displayed. Now, I'm not saying that providing a way to track individuals across employments, acquisitions, or other transitions is a bad idea. And using the .mailmap is a workable way of doing this, since it needs to be manually invoked outside of shortlog (it could be disastrous if it didn't, or the default git behaviour changes in the future). *But* before we start treating it in this way, I would like to see the intended use of .mailmap documented (in the tree), and that change reviewed by at least a couple of stewards and enough others to be able to say we have a consensus. If we do, I have no objections to the below patch being added at that point - but from my viewpoint, doing it before then would mean an unreviewed change to the process as some people understood when the file was first added. Best Regards, Leif > If he agrees the maintainer pushing his patch can amend this snippet: > > -- >8 -- > diff --git a/.mailmap b/.mailmap > index ae179b0886c7..bbf3052aa3aa 100644 > --- a/.mailmap > +++ b/.mailmap > @@ -37,6 +37,7 @@ Jim Dailey > Jim Dailey > Laszlo Ersek > > Laszlo Ersek > +Leif Lindholm > Liming Gao > Liming Gao > > Liming Gao > --- > > Note he stays close to you and Liming ;) > > > > > Right now, Leif is not listed in ".mailmap" -- probably because there is > > exactly one email address associated with his authorship, all across the > > project history. But that's what's changing now, isn't it? > > > > Thanks, > > Laszlo > > > > > > > > > diff --git a/Maintainers.txt b/Maintainers.txt > > > index 146d8aca93f0..ca9da2892534 100644 > > > --- a/Maintainers.txt > > > +++ b/Maintainers.txt > > > @@ -70,7 +70,7 @@ Tianocore Stewards > > > F: * > > > M: Andrew Fish > > > M: Laszlo Ersek > > > -M: Leif Lindholm > > > +M: Leif Lindholm > > > M: Michael D Kinney > > > Responsible Disclosure, Reporting Security Issues > > > @@ -87,7 +87,7 @@ UEFI Shell Binaries (ShellBinPkg.zip) from EDK II Releases: > > > W: https://github.com/tianocore/edk2/releases/ > > > M: Ray Ni (Ia32/X64) > > > M: Zhichao Gao (Ia32/X64) > > > -M: Leif Lindholm (ARM/AArch64) > > > +M: Leif Lindholm (ARM/AArch64) > > > M: Ard Biesheuvel (ARM/AArch64) > > > EDK II Architectures: > > > @@ -95,7 +95,7 @@ EDK II Architectures: > > > ARM, AARCH64 > > > F: */AArch64/ > > > F: */Arm/ > > > -M: Leif Lindholm > > > +M: Leif Lindholm > > > M: Ard Biesheuvel > > > EDK II Continuous Integration: > > > @@ -126,13 +126,13 @@ EDK II Packages: > > > ArmPkg > > > F: ArmPkg/ > > > W: https://github.com/tianocore/tianocore.github.io/wiki/ArmPkg > > > -M: Leif Lindholm > > > +M: Leif Lindholm > > > M: Ard Biesheuvel > > > ArmPlatformPkg > > > F: ArmPlatformPkg/ > > > W: https://github.com/tianocore/tianocore.github.io/wiki/ArmPlatformPkg > > > -M: Leif Lindholm > > > +M: Leif Lindholm > > > M: Ard Biesheuvel > > > ArmVirtPkg > > > @@ -140,7 +140,7 @@ F: ArmVirtPkg/ > > > W: https://github.com/tianocore/tianocore.github.io/wiki/ArmVirtPkg > > > M: Laszlo Ersek > > > M: Ard Biesheuvel > > > -R: Leif Lindholm > > > +R: Leif Lindholm > > > ArmVirtPkg: modules used on Xen > > > F: ArmVirtPkg/ArmVirtXen.* > > > @@ -173,7 +173,7 @@ M: Alexei Fedorov > > > EmbeddedPkg > > > F: EmbeddedPkg/ > > > W: https://github.com/tianocore/tianocore.github.io/wiki/EmbeddedPkg > > > -M: Leif Lindholm > > > +M: Leif Lindholm > > > M: Ard Biesheuvel > > > EmulatorPkg > > > > > >