Re: [EXTERNAL] Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-24 Thread Bret Barkelew via groups.io
] Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI On 04/19/20 22:56, Rebecca Cran wrote: > Where are those pages linked from (i.e. how would people find them)? I've used the following three methods to find content in the wiki: (1) web search (2) if

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-20 Thread Laszlo Ersek
On 04/19/20 22:56, Rebecca Cran wrote: > Where are those pages linked from (i.e. how would people find them)? I've used the following three methods to find content in the wiki: (1) web search (2) if you open and click on the "Pages" box

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-20 Thread Laszlo Ersek
On 04/19/20 11:35, Ard Biesheuvel wrote: > On 4/19/20 10:29 AM, Sean via groups.io wrote: >> Laszlo, >> >> Regarding your comments about disliking the verbosity of the markdown >> table/html table for build status both in Core Ci and now these >> Platform CI readme files. >> >> As a learning

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-19 Thread Rebecca Cran
-Original Message- From: Laszlo Ersek Sent: Thursday, April 16, 2020 7:52 AM To: Sean Brogan Cc: devel@edk2.groups.io; Ard Biesheuvel Subject: [EXTERNAL] Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI My excuse for not polishing it more

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-19 Thread Ard Biesheuvel
On 4/19/20 10:29 AM, Sean via groups.io wrote: Laszlo, Regarding your comments about disliking the verbosity of the markdown table/html table for build status both in Core Ci and now these Platform CI readme files. As a learning experience I updated the OvmfPkg readme to use reStructuredText

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-19 Thread Sean via groups.io
Sean -Original Message- From: Laszlo Ersek Sent: Thursday, April 16, 2020 7:52 AM To: Sean Brogan Cc: devel@edk2.groups.io; Ard Biesheuvel Subject: [EXTERNAL] Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI On 04/15/20 22:38, sean.brogan

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-16 Thread Laszlo Ersek
Hi Sean, On 04/15/20 21:31, sean.brogan via [] wrote: > My corporate mail server changes the subject line which also destroys > threading so I figured this was better. Maybe not as i have no > control over the CC/To lines of the email that is sent. Do Bret's > responses which add "[External]"

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-16 Thread Laszlo Ersek
On 04/15/20 22:38, sean.brogan via [] wrote: > On Wed, Apr 15, 2020 at 10:18 AM, Laszlo Ersek wrote: > >> >> ArmVirtPkg/ArmVirtPkg.ci.yaml >> ArmVirtPkg/PlatformCI/Ubuntu-GCC5.yml >> ArmVirtPkg/PlatformCI/PlatformBuild.py >> ArmVirtPkg/PlatformCI/README-pytools.md >>

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-15 Thread Sean via groups.io
On Wed, Apr 15, 2020 at 10:18 AM, Laszlo Ersek wrote: > > ArmVirtPkg/ArmVirtPkg.ci.yaml > ArmVirtPkg/PlatformCI/Ubuntu-GCC5.yml > ArmVirtPkg/PlatformCI/PlatformBuild.py > ArmVirtPkg/PlatformCI/README-pytools.md > ArmVirtPkg/PlatformCI/iasl_ext_dep.yaml I am ok with the above except one thought

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-15 Thread Sean via groups.io
On Wed, Apr 15, 2020 at 09:58 AM, Laszlo Ersek wrote: > > I have not ignored your email. Instead, I have not seen it. > > The reason is that you messaged the list only -- you didn't put me in > the To: or Cc: headers, despite addressing me by name in the message body. I use the groups.io web

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-15 Thread Ard Biesheuvel
On 4/15/20 7:18 PM, Laszlo Ersek wrote: On 04/09/20 21:57, Sean via groups.io wrote: Ard/Laszlo, On the topic of Package files. 1. The *.ci.yaml is designed to be at the root of the package and this follows all other Core ci usage.  The Core CI will only look for this file at the root as of

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-15 Thread Laszlo Ersek
On 04/09/20 21:57, Sean via groups.io wrote: > Ard/Laszlo, > > On the topic of Package files. > 1. The *.ci.yaml is designed to be at the root of the package and this > follows all other Core ci usage.  The Core CI will only look for this file at > the root as of now. OK. This makes sense to

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-15 Thread Laszlo Ersek
Hi Sean, On 04/15/20 08:55, Sean via groups.io wrote: > Any thoughts? I have not ignored your email. Instead, I have not seen it. The reason is that you messaged the list only -- you didn't put me in the To: or Cc: headers, despite addressing me by name in the message body. While I do follow

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-15 Thread Sean via groups.io
Any thoughts?  I would like to get this in before any more failures get checked in causing more dependencies. I have a branch here: https://github.com/spbrogan/edk2/tree/PlatformAndCoreCIForOvmfArmVirtEmulatorPackages_v7 but am waiting for more feedback to make V2 patchset. Thanks Sean

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Sean via groups.io
Ard/Laszlo, On the topic of Package files. 1. The *.ci.yaml is designed to be at the root of the package and this follows all other Core ci usage.  The Core CI will only look for this file at the root as of now. 2.  The PlatformBuild.py file is exactly like a build.bat or build.sh file found

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Michael Kubacki
Thanks for looking into this. I agree per case (c) that I should add my S-o-b. I will plan to add both mine and Sean's S-o-b in v2 of the series. Thanks, Michael On 4/9/2020 6:18 AM, Laszlo Ersek wrote: On 04/09/20 11:23, Leif Lindholm wrote: On Thu, Apr 09, 2020 at 11:17:31 +0200, Laszlo

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Leif Lindholm
On Thu, Apr 09, 2020 at 15:18:42 +0200, Laszlo Ersek wrote: > 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.kuba...@outlook.com wrote: > >>> From: Sean Brogan > >>> > >>>

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Laszlo Ersek
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.kuba...@outlook.com wrote: >>> From: Sean Brogan >>> >>> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=2570 >>> >>> Add new Azure Pipeline

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Leif Lindholm
On Thu, Apr 09, 2020 at 11:17:31 +0200, Laszlo Ersek wrote: > Hi Michael, > > On 04/08/20 20:13, michael.kuba...@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:

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Laszlo Ersek
Hi Michael, On 04/08/20 20:13, michael.kuba...@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

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Ard Biesheuvel
On 4/8/20 8:13 PM, michael.kuba...@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

Re: [edk2-devel] [PATCH v1 2/6] ArmVirtPkg: Add Platform CI and configuration for Core CI

2020-04-09 Thread Laszlo Ersek
On 04/08/20 20:13, michael.kuba...@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