From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-1.mimecast.com (us-smtp-1.mimecast.com [207.211.31.120]) by mx.groups.io with SMTP id smtpd.web12.8569.1586438331966964807 for ; Thu, 09 Apr 2020 06:18:52 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=csKU3bQl; spf=pass (domain: redhat.com, ip: 207.211.31.120, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1586438331; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=vQvCwV6OMlZLjDbWr9loHNggfenxNmCrupxjs99m3dY=; b=csKU3bQlo6CIAT4q8WRDPCECxLt6KVORNBdrSO/dIUMg1B4JD03RiwmVhGXo1FtMTTykpV 2aZ2iT7AdPiU554NUuUC/4CHiqs0mWBwTfn2c5L5lF1fYuSwyJczNEr0fx2vu36du8aXxC Th5Gcqo7RpEfD80mQ+o5YfbgIqX6EtY= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-441-fH1mLGgUNEWenRkNTqM2zQ-1; Thu, 09 Apr 2020 09:18:45 -0400 X-MC-Unique: fH1mLGgUNEWenRkNTqM2zQ-1 Received: from smtp.corp.redhat.com (int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 8EC5818B9FC4; Thu, 9 Apr 2020 13:18:44 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-112-213.ams2.redhat.com [10.36.112.213]) by smtp.corp.redhat.com (Postfix) with ESMTP id 1F6541195A4; Thu, 9 Apr 2020 13:18:42 +0000 (UTC) Subject: Re: [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI To: Leif Lindholm Cc: michael.kubacki@outlook.com, devel@edk2.groups.io, Ard Biesheuvel References: <20200408181327.4324-1-michael.kubacki@outlook.com> <20200409092345.GZ14075@vanye> From: "Laszlo Ersek" Message-ID: <81bf053e-d35b-67a6-5869-73087b9fc857@redhat.com> Date: Thu, 9 Apr 2020 15:18:42 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20200409092345.GZ14075@vanye> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.11 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit On 04/09/20 11:23, Leif Lindholm wrote: > On Thu, Apr 09, 2020 at 11:17:31 +0200, Laszlo Ersek wrote: >> Hi Michael, >> >> On 04/08/20 20:13, michael.kubacki@outlook.com wrote: >>> From: Sean Brogan >>> >>> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570 >>> >>> Add new Azure Pipeline definitions to build and run ArmVirtPkg with: >>> * Ubuntu GCC5 >>> Add PyTool based build of ArmVirtPkg >>> Add extdep for managing the iasl dependency >>> Add ArmVirtPkg.ci.yaml for Core CI >>> Add README-pytools for details and instructions >>> >>> Cc: Laszlo Ersek >>> Cc: Ard Biesheuvel >>> Cc: Leif Lindholm >>> Signed-off-by: Sean Brogan >> >> The commit message is missing your Signed-off-by. You can & should >> keep Sean's of course, but since you are submitting it, I think yours >> (too) is required, per "Developer Certificate of Origin" in >> "Readme.md". > > Michael cannot attest for Sean's legal opinions about this > contribution, only Sean can do that. Not so sure about that (see below), but here I'm not questioning that. > The attribution of authorship is already covered by the From: tag. Agreed. My point is that Michael is technically partaking in the contribution of this patch, even if he's only forwarding (upstreaming) Sean's patch verbatim. Therefore the language in "Readme.md" applies to him as a contributor: > # Code Contributions > To make a contribution to a TianoCore project, follow these steps. > [...] > 2. Your commit message must include your `Signed-off-by` signature > [...] > > # Developer Certificate of Origin > > Your change description should use the standard format for a > commit message, and must include your `Signed-off-by` signature. > > [...] > > By making a contribution to this project, I certify that: > > (a) The contribution was created in whole or in part by me and I > have the right to submit it under the open source license > indicated in the file; or > > (b) The contribution is based upon previous work that, to the best > of my knowledge, is covered under an appropriate open source > license and I have the right under that license to submit that > work with modifications, whether created in whole or in part > by me, under the same open source license (unless I am > permitted to submit under a different license), as indicated > in the file; or > > (c) The contribution was provided directly to me by some other > person who certified (a), (b) or (c) and I have not modified > it. > > [...] So, two points: - under case (c), I do think Michael would attest for Sean's legal opinions, - even if it's not case (c) that applies, a Signed-off-by is needed from Michael, for one of the other cases. The DCO that's included in "Readme.md" spells out the following goal (or "spirit"): > In order to keep track of who did what, all patches contributed must > include a statement that to the best of the contributor's knowledge > they have the right to contribute it under the specified license. If we don't include an S-o-b from Michael on the patch, his participation in the upstreaming of this code change will disappear entirely. The Committer field will refer to the mergify bot, the Author field will refer (correctly) to Sean, and the [f]act of Michael posting the patch to the list will be lost from the git history. The one S-o-b that we could technically do without, IMO, is Sean's. Laszlo