From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f65.google.com (mail-wr1-f65.google.com [209.85.221.65]) by mx.groups.io with SMTP id smtpd.web11.617.1572404294621101880 for ; Tue, 29 Oct 2019 19:58:14 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=FR8OUd1m; spf=pass (domain: linaro.org, ip: 209.85.221.65, mailfrom: leif.lindholm@linaro.org) Received: by mail-wr1-f65.google.com with SMTP id a11so577994wra.6 for ; Tue, 29 Oct 2019 19:58:14 -0700 (PDT) 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:in-reply-to:user-agent; bh=70nVYYsTetsV9VOBbP/LsQB1b24xyhX30x+n+vRCuZE=; b=FR8OUd1mjpbJPgyIUW/IucS4tkG1D7l9RjVHXbv8Slx4/zv9T9cjrw6X0lbdkGsPVH cK87bBS9D+dQLNVlmB8/DOf3kea8fsQTXw7Bl/7ReGIHjXKnI0XRC6Xmm2Slo0ydzLeq F55NThG7k1z5R9EYm9p7C+OzeSzpz8EsIn3pSFIrLMug5nIl9lh0R7scjAfyOwGlIdNm AQPHjIgAd3f8AfJzUDNzf0F2+7GWKXb7TCgwk6fdgG+6sar8kf4CY8H8WyTjJeR14rC5 ZUL/3jPyrm8alQR0lVfXiVvALvGyQKtfwMNodXf8ux0gXf6PZh5cCF9fjWwBBJ1dfkgI lxqg== 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:in-reply-to:user-agent; bh=70nVYYsTetsV9VOBbP/LsQB1b24xyhX30x+n+vRCuZE=; b=JF5xmGOEAQ1w/pHv5zqSISUAXMshTBRysI6xBl2YtgOP2hiNJsdgITcbCXdYOllCSC dQtIXVmQNvgcaEdDohvZi5eei/UBrv5pSbzLlj+edklTGr3sIdFcHIFnN6UxyH+iK3uW CeCuwyavKfXs+u0Mcg3OEe5b6J9yev9dJ+rEUmnSQSKMuNV7O28DUltNJOvpnZefd5v6 5VitQANV5+4MA3vXi0Hr/l5GOVAlceDdGOsrsxsUFTyb/0U2LhXSB7e/Oa/5nQAIfVRw DlEnSTm0E3SKSrraAM0SN6CEUwi/sy34DjFNqEkPLdFWEMwNBr1i9GfX/8s+YK8Ik8FK nmBQ== X-Gm-Message-State: APjAAAWXLbbasY/GspqPlCTcNrrS8qsM3lr60kBS3HKH/gOv76xxvG/5 tfr3fjTHxufI0KorQY7nmE//Cg== X-Google-Smtp-Source: APXvYqyji0dphmg3+ReMlMoBb8n3GfPENLxMFJeWzYFUD4Jq9xcWRqkD+K7Wvf8LPzuvkABg6CyQmA== X-Received: by 2002:a5d:52c8:: with SMTP id r8mr17909141wrv.347.1572404293128; Tue, 29 Oct 2019 19:58:13 -0700 (PDT) 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 f14sm813607wrv.17.2019.10.29.19.58.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 29 Oct 2019 19:58:12 -0700 (PDT) Date: Wed, 30 Oct 2019 02:58:10 +0000 From: "Leif Lindholm" To: Michael D Kinney Cc: devel@edk2.groups.io, Andrew Fish , Laszlo Ersek Subject: Re: [Patch v3 22/22] Readme.md: Add CI build status badges Message-ID: <20191030025810.GV16820@bivouac.eciton.net> References: <20191029195517.20028-1-michael.d.kinney@intel.com> <20191029195517.20028-23-michael.d.kinney@intel.com> MIME-Version: 1.0 In-Reply-To: <20191029195517.20028-23-michael.d.kinney@intel.com> User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Oct 29, 2019 at 12:55:17PM -0700, Michael D Kinney wrote: > https://bugzilla.tianocore.org/show_bug.cgi?id=2315 > > Update Readme.md with status badges from the most recent > EDK II Continuous Integration(CI) build. > > Cc: Andrew Fish > Cc: Laszlo Ersek > Cc: Leif Lindholm > Signed-off-by: Michael D Kinney > --- > Readme.md | 29 ++++++++++++++++++++++------- > 1 file changed, 22 insertions(+), 7 deletions(-) > > diff --git a/Readme.md b/Readme.md > index 27e4ce0771..37183c3265 100644 > --- a/Readme.md > +++ b/Readme.md > @@ -3,6 +3,19 @@ > A modern, feature-rich, cross-platform firmware development environment > for the UEFI and PI specifications from www.uefi.org. > > +## Build Status > + > +| Host Type | Toolchain | Branch | Build Status | Test Status | Code Coverage | This may sound like nitpicking, but I think it's important for messaging: we aren't listing tolchains, we are listing toolchain profiles. For VS, we are actually adding new profiles for each version, so that statement is not misleading. However, GCC5 is highly misleading (it is the profile we use for anything gcc5 onwards). I don't think we can usefully pull in information about the toolchain actually used, so I think the name of the column should really be toolchain profile. Even given that, I think it is time we create an abstracted GCC_CURRENT toolchain profile, pointing to GCC5. But if we rename the column, that may not be a prerequisite for this set. > +| :-------- | :-------- | :---- | :----- | :---- | :--- | > +| Windows | VS2019 | master | [![Build Status](https://dev.azure.com/tianocore/edk2-ci-play/_apis/build/status/Edk2%20Windows%20VS2019%20CI?branchName=master)](https://dev.azure.com/tianocore/edk2-ci-play/_build/latest?definitionId=14&branchName=master) | [![Azure DevOps tests](https://img.shields.io/azure-devops/tests/tianocore/edk2-ci-play/14.svg)](https://dev.azure.com/tianocore/edk2-ci-play/_build/latest?definitionId=14&branchName=master) | [![Code Coverage](https://img.shields.io/badge/coverage-coming_soon-blue)](https://dev.azure.com/tianocore/edk2-ci-play/_build/latest?definitionId=14&branchName=master)| > +| Ubuntu | GCC5 | master | [![Build Status](https://dev.azure.com/tianocore/edk2-ci-play/_apis/build/status/Edk2%20Ubuntu%20GCC5%20CI?branchName=master)](https://dev.azure.com/tianocore/edk2-ci-play/_build/latest?definitionId=15&branchName=master) | [![Azure DevOps tests](https://img.shields.io/azure-devops/tests/tianocore/edk2-ci-play/15.svg)](https://dev.azure.com/tianocore/edk2-ci-play/_build/latest?definitionId=15&branchName=master) | [![Code Coverage](https://img.shields.io/badge/coverage-coming_soon-blue)](https://dev.azure.com/tianocore/edk2-ci-play/_build/latest?definitionId=15&branchName=master) | I have one problem with the above, which is that it turns an otherwise very readable in text mode file (which is half the point of .md) into what looks a little bit like a transmission error. While I agree there is value in having this on the default loaded github webpage, do we have other options? > + > +[More CI Build information](ci/readme.md) I don't see this file in the tree? Which is good, beacuse it doesn't follow name standards. > + > +## License Details > + > +[![License](https://img.shields.io/badge/license-BSD--2--Clause--Patent-blue)](License.txt) > + I don't mind this addition, but I think it should be a separate patch. I also think it would be best to hold this back until we've submoduled some of those otherwise licensed components. There is a very real likelihood of people skimming, seeing the image and ignoring the text. > The majority of the content in the EDK II open source project uses a > [BSD-2-Clause Plus Patent License](License.txt). The EDK II open source project > contains the following components that are covered by additional licenses: > @@ -20,7 +33,8 @@ contains the following components that are covered by additional licenses: > The EDK II Project is composed of packages. The maintainers for each package > are listed in [Maintainers.txt](Maintainers.txt). > > -# Resources > +## Resources > + No objections to change, but it is unrelated to this patch. > * [TianoCore](http://www.tianocore.org) > * [EDK II](https://github.com/tianocore/tianocore.github.io/wiki/EDK-II) > * [Getting Started with EDK II](https://github.com/tianocore/tianocore.github.io/wiki/Getting-Started-with-EDK-II) > @@ -29,7 +43,8 @@ are listed in [Maintainers.txt](Maintainers.txt). > * [How To Contribute](https://github.com/tianocore/tianocore.github.io/wiki/How-To-Contribute) > * [Release Planning](https://github.com/tianocore/tianocore.github.io/wiki/EDK-II-Release-Planning) > > -# Code Contributions > +## Code Contributions > + No objection to change, but it is unrelated to this patch. > To make a contribution to a TianoCore project, follow these steps. > 1. Create a change description in the format specified below to > use in the source control commit log. > @@ -57,7 +72,7 @@ To make a contribution to a TianoCore project, follow these steps. > Contributions using other licenses might be accepted, but further > review will be required. > > -# Developer Certificate of Origin > +## Developer Certificate of Origin No objection to change, but it is unrelated to this patch. > > Your change description should use the standard format for a > commit message, and must include your `Signed-off-by` signature. > @@ -104,7 +119,7 @@ By making a contribution to this project, I certify that: > this project or the open source license(s) involved. > ``` > > -# Sample Change Description / Commit Message > +## Sample Change Description / Commit Message No objection to change, but it is unrelated to this patch. > > ``` > From: Contributor Name > @@ -115,14 +130,14 @@ Full-commit-message > Signed-off-by: Contributor Name > ``` > > -## Notes for sample patch email > +### Notes for sample patch email No objection to change, but it is unrelated to this patch. > > * The first line of commit message is taken from the email's subject > line following `[Repository/Branch PATCH]`. The remaining portion of the > commit message is the email's content. > * `git format-patch` is one way to create this format > > -## Definitions for sample patch email > +### Definitions for sample patch email No objection to change, but it is unrelated to this patch. > > * `Repository` is the identifier of the repository the patch applies. > This identifier should only be provided for repositories other than > @@ -141,7 +156,7 @@ Signed-off-by: Contributor Name > * `Signed-off-by` is the contributor's signature identifying them > by their real/legal name and their email address. > > -# Submodules > +## Submodules No objection to change, but it is unrelated to this patch. / Leif > > Submodule in EDK II is allowed but submodule chain should be avoided > as possible as we can. Currently EDK II contains two submodules > -- > 2.21.0.windows.1 >