From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (NAM10-DM6-obe.outbound.protection.outlook.com [40.107.93.118]) by mx.groups.io with SMTP id smtpd.web11.312.1587759681646733377 for ; Fri, 24 Apr 2020 13:21:21 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@microsoft.com header.s=selector2 header.b=K9UqZP3v; spf=pass (domain: microsoft.com, ip: 40.107.93.118, mailfrom: bret.barkelew@microsoft.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Ov1twfeJqzD8NCDfvkchG4TuhmAYRJv67kOokPx897k3S7lG6q98yIDr2R6AuGmocyVUzS3Yy39ybKcf1rO2uuBTSYatFAus5Aa0axDGHB8KEFzta5XVvawaw2Q5ZgjZhT3QFq/w7BRG9IFAGKq+RtFi25967cBfv3KlFnRoY+pIgg4N0MmH/p4z1rZrEA5/ImCy+t8I4cu6goI3EJmKaVDdH01wdabKzME090LsaxWDWVVsJlScZn+66i2zGx+TE00IPrf+fNUD0r43liYquAjpGBDbXO3Sv11h5jFxYwmmog7drXm0+GLe3xfBwE81rz3q+w42OVclx91fwhp1+w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=RSkm1KlPEzYlPn7atisLxCZRBr+OCXxnua2/W+gGH1Q=; b=aC2JL77Ziez+f1klAgYp9H+g843FPFfYsUbYl8dlZOwqp8vwTtMrNMRHBgVDlyW1dZM1/S8PUiKWv24IuI0Wnwc1j7sB9atX+xUStL6j+Di/MleTC2/E4EofCIEVjXXc1FiXRj/kCnhU+eYUwKbm1BHJ+XKdfNrZC92KwSt5W01NJyTEILS++YYXdiYGj3m+0k0vaFca+5YYCeEbhXp+Juh9ZeFZckflBpfh8Y6RpsTIrL1CrOxNuIsJ9IvxMBYhlER3NYR/OA12nAstX6B9kg7UMs1vqiAE0qJDwPW/lHYdprmwULK87Zx5LjYe+fcX6HWmoO/ovrLzTAC8U4thSQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=RSkm1KlPEzYlPn7atisLxCZRBr+OCXxnua2/W+gGH1Q=; b=K9UqZP3vtuIzJF4wsLZWP7MNaR1qY1B5J/RplcBdU6Nzl2jDzdEt5CjMuXCE3Y448J5hESbQy4UGyItuXtO3DKY9JQN9D8d0nEV20Ezwr7/hHWrItSHkJONIPFialnfZjbB5C+uTKNpXDI1UCFrom28j0pXAKjcAc6B1nQgSCfQ= Received: from CY4PR21MB0743.namprd21.prod.outlook.com (2603:10b6:903:b2::9) by CY4PR21MB1538.namprd21.prod.outlook.com (2603:10b6:910:8f::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2958.2; Fri, 24 Apr 2020 20:21:20 +0000 Received: from CY4PR21MB0743.namprd21.prod.outlook.com ([fe80::90d:10d9:c5bc:5318]) by CY4PR21MB0743.namprd21.prod.outlook.com ([fe80::90d:10d9:c5bc:5318%12]) with mapi id 15.20.2958.010; Fri, 24 Apr 2020 20:21:20 +0000 From: "Bret Barkelew" To: "devel@edk2.groups.io" , "lersek@redhat.com" , Sean Brogan , "michael.kubacki@outlook.com" CC: Andrew Fish , Ard Biesheuvel , Jordan Justen , Leif Lindholm , Liming Gao , "Kinney, Michael D" , Ray Ni Subject: Re: [edk2-devel] [EXTERNAL] Re: [PATCH v2 0/6] Edk2 Platform and Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg Thread-Topic: [edk2-devel] [EXTERNAL] Re: [PATCH v2 0/6] Edk2 Platform and Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg Thread-Index: AQHWF+e0SHJSPyUL0k6VpbIB12XDJaiF4TeAgAJ5iICAAGFCsw== Date: Fri, 24 Apr 2020 20:21:20 +0000 Message-ID: References: <5fcd80ac-03d1-a806-6a69-4602bc0d352c@redhat.com> , In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=True;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2020-04-24T20:21:16.8639378Z;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Privileged authentication-results: spf=none (sender IP is ) smtp.mailfrom=Bret.Barkelew@microsoft.com; x-originating-ip: [174.21.83.205] x-ms-publictraffictype: Email x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: d9eebeb6-439d-49c7-40e3-08d7e88d0d0c x-ms-traffictypediagnostic: CY4PR21MB1538:|CY4PR21MB1538:|CY4PR21MB1538: x-ms-exchange-transport-forked: True x-ld-processed: 72f988bf-86f1-41af-91ab-2d7cd011db47,ExtAddr x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:8882; x-forefront-prvs: 03838E948C x-forefront-antispam-report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:CY4PR21MB0743.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFTY:;SFS:(4636009)(396003)(346002)(366004)(376002)(136003)(39860400002)(82950400001)(66476007)(66446008)(64756008)(66556008)(55016002)(66946007)(110136005)(54906003)(8936002)(316002)(4326008)(82960400001)(81156014)(33656002)(2906002)(26005)(8990500004)(186003)(966005)(9686003)(478600001)(5660300002)(7416002)(7696005)(76116006)(86362001)(8676002)(6506007)(52536014)(10290500003)(71200400001)(53546011)(156123004)(547064002);DIR:OUT;SFP:1102; received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 0hJndGYeSgYsYwUDBziK7jPyQnNFwQ5dIUMPHnYlygJPHWd0dopC9T7TRvtkC3vO59hwft6YClfg1ajI3Qy+Fo0fn4xTn0pSxrPl7vpf8KEYP6FS9txnAV7sObIa0K1EpI9bK7/DrUTmYwtXOS1KhgVUF9nBEec7XHW1T2PWpC0WuwjzTEpuHqq4QTmepwJZlB5YIdV/s75OYZbYWZaAiICK6pxIP7l7d1mi5OQGXVGPafEaopB3gA/rPkwM3SaENPeY/uaUHD8+6yEmtgcyA9qwU/K6O2KoB22LGSIeIiqhXrNGLc4TOk2LEC71Yqz17DqvllY1ORLHG2JfrU9NX3vgds8USCJHFp1KgipYVor6R+xh5H5NXqYpA3Zpt2LrOlwmeAym72TYr75SeFgGzTtIem9rBhqW/J4WX/JxjuoiTX+EfhfiX6yZaOzhEHR3pMUlB529xeAZ8Lq5LhIs++2TpFkWZIOnM98IdpUkltFY/1YRJ4KOOlbE99Ime3QB2jOd+0ai4GAgJn1EPG35kfvUII3dYgcmjsOj9Eu5ctZuu6VJjmmbu3ps+QkEUulJhw/yjeHuQrwwKZ176oRAKXQvrnuIcXSm8C/5AaDRmM8= x-ms-exchange-antispam-messagedata: 2lk1QtgPqKIw8iRS09gSKDe6WqzK0VtPybHSVqlWh+98xDCLMED85Mb9tGJMpaB8ZolVbZ+Owm+Mbs9Ci8hzxKqLaUD4LslX1Zalo7gYiiIl7MeepuS1oMbNoX5JXfM3B0cbolc+mOVliDKmDf5d4tweC2CFv6PdN4ClDf8Rp4NUWXY1ho9wIRwgPZsO8pLE6QGw4YUqnFavNQjsVt9BFkHC0BJobV0Ee9AuWYgeiklGy9Hhiv7rFOF5hhQi6J98O9ZtC908irbkJrTuZb/AUjLBFXVNJlnE5ZehXTbyStY8cA+GTLkx664Fg4CifnRGRKWbcLO8BhfiG7t5OobGGxHIpTZyJzZ2yThzfvQaaC7CC1axAIzFu2A/ApEXPgvZUGPycq2wZic3i2vJJE+oAn5OL6HqmteyyLHHdowDfujTQYoSAc2I3IMg4cGHfxtfUaxA96MEfC8tL+LxDYWxObxxc05vth/NDYeB5p6El8h1S9PVysPCaxpWPWnRrSS+3nKbqbvZdxO2fKbVrMOfhA2i9CBUWLgKItikEZEonMjINgy7zKn9sB8X/6mZmbvBahE1hmyHADuAE4TAue1Sz2bBXJ5sAUWjInbRDh46qWKJN1qKGf+NTPHO9Pv7TY/e/vtsPaTlLTVD0sG2fEvImg1PorbTzAo+HXWE2LhzomPJx6V3TcohC6Z1zwWgIBUuERMkDvF7GfZDVtyncySMXTdjg2awJnHCMCnzIvjiQZc2wp1NOPFhJP2kV1VBtRY8JwMFcD6oRvrJIsNwz+hJUUTTLWeI4jZkjXefI9oN7Wc= MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: d9eebeb6-439d-49c7-40e3-08d7e88d0d0c X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Apr 2020 20:21:20.1909 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: 4uDzPIBq+JbXAf/PStW2qoGR73hhTCVzyiqc8TskJ5+hbAeocUMcMXeMVvPMRMTBQ6YVLPm9gtjwI5c3hlBlxQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR21MB1538 Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_CY4PR21MB07437827273B416429AF63BAEFD00CY4PR21MB0743namp_" --_000_CY4PR21MB07437827273B416429AF63BAEFD00CY4PR21MB0743namp_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Reviewed-by: Bret Barkelew - Bret ________________________________ From: devel@edk2.groups.io on behalf of Laszlo Erse= k via groups.io Sent: Friday, April 24, 2020 7:33:11 AM To: Sean Brogan ; michael.kubacki@outlook.com <= michael.kubacki@outlook.com>; devel@edk2.groups.io Cc: Andrew Fish ; Ard Biesheuvel = ; Bret Barkelew ; Jordan Justen ; Leif Lindholm ; Liming Gao ; Kinney, Michael D ; Ray Ni Subject: Re: [edk2-devel] [EXTERNAL] Re: [PATCH v2 0/6] Edk2 Platform and = Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg On 04/23/20 02:45, Sean Brogan wrote: > I was hoping that because it wasn't "HTML tag soup" that build status co= uld be front and center in the package readme as I find that more in line w= ith expectations on github based projects. Nesting it deeper in the packa= ge just means less people find it when looking at your package. But I cou= ld see a few ways to solve it so I am looking for feedback and alignment be= fore doing anything else. > > Option 1. > > 1. Add the Platform CI build status for the three platforms to the Edk2 = repo readme since these are platforms in edk2 this makes sense and brings t= he most visibility to their status and the existence of the builds. > 2. Convert edk2 repo readme to RST so it avoids "HTML tag soup" > 3. Move the remaining part of the OvmfPkg/Readme.rst file into OvmfPkg/P= latformCI/Readme.md (I prefer MD when not doing large tables of links as i= t is easier to write and slightly easier to read as plain text). > 4. Do step 3 for ArmVirtPkg and EmulatorPkg too > > Option 2. > 1. Leave the ReadMe.rst at the root of each package but only include the= build status table and add a link to the PlatformCi/ReadMe.md > 2. Move the Pytool and Platform CI focused readme contents to PlatformCI= /ReadMe.md > > Option 3. > > Something else. > > Finally does anyone have a strong preference for ReadMe vs README vs rea= dme vs Readme vs ReAdMe? I see a mix of these in the edk2 code tree. Sorry, I didn't mean to ignore this sub-thread, I just got to see the other email (about the V10 branch) first. My understanding is that V10 has come to a resolution about the questions above; and I'm happy with V10 (as I stated in that thread). Thanks! Laszlo --_000_CY4PR21MB07437827273B416429AF63BAEFD00CY4PR21MB0743namp_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Reviewed-by: Bret Barkelew <bret.barkelew@micros= oft.com>

 

- Bret

 

From: devel@edk2.groups.io= <devel@edk2.groups.io> on behalf of Laszlo Ersek via groups.io <l= ersek=3Dredhat.com@groups.io>
Sent: Friday, April 24, 2020 7:33:11 AM
To: Sean Brogan <sean.brogan@microsoft.com>; michael.kubacki@= outlook.com <michael.kubacki@outlook.com>; devel@edk2.groups.io <d= evel@edk2.groups.io>
Cc: Andrew Fish <afish@apple.com>; Ard Biesheuvel <ard.bie= sheuvel@arm.com>; Bret Barkelew <Bret.Barkelew@microsoft.com>; Jor= dan Justen <jordan.l.justen@intel.com>; Leif Lindholm <leif@nuviai= nc.com>; Liming Gao <liming.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Ray Ni <ray.ni@intel.com>
Subject: Re: [edk2-devel] [EXTERNAL] Re: [PATCH v2 0/6] Edk2 Platfo= rm and Core CI for ArmVirtPkg, EmulatorPkg, and OvmfPkg
 
On 04/23/20 02:45, Sean Brogan wrote:
> I was hoping that because it wasn't "HTML tag soup" that bu= ild status could be front and center in the package readme as I find that m= ore in line with expectations on github based projects.   Nesting= it deeper in the package just means less people find it when looking at your package.   But I could see a few ways to s= olve it so I am looking for feedback and alignment before doing anything el= se.
>
> Option 1.
>
> 1. Add the Platform CI build status for the three platforms to the Ed= k2 repo readme since these are platforms in edk2 this makes sense and bring= s the most visibility to their status and the existence of the builds.
> 2. Convert edk2 repo readme to RST so it avoids "HTML tag soup&q= uot;
> 3. Move the remaining part of the OvmfPkg/Readme.rst file into OvmfPk= g/PlatformCI/Readme.md  (I prefer MD when not doing large tables of li= nks as it is easier to write and slightly easier to read as plain text).&nb= sp; 
> 4. Do step 3 for ArmVirtPkg and EmulatorPkg too
>
> Option 2.
> 1. Leave the ReadMe.rst at the root of each package but only include = the build status table and add a link to the PlatformCi/ReadMe.md
> 2. Move the Pytool and Platform CI focused readme contents to Platfor= mCI/ReadMe.md
>
> Option 3.
>
> Something else.
>
> Finally does anyone have a strong preference for ReadMe vs README vs = readme vs Readme vs ReAdMe?  I see a mix of these in the edk2 code tre= e.  

Sorry, I didn't mean to ignore this sub-thread, I just got to see the
other email (about the V10 branch) first. My understanding is that V10
has come to a resolution about the questions above; and I'm happy with
V10 (as I stated in that thread).

Thanks!
Laszlo




--_000_CY4PR21MB07437827273B416429AF63BAEFD00CY4PR21MB0743namp_--