commit | 2be4828af1c92a848af90429a9a0b44544c80553 | [log] [tgz] |
---|---|---|
author | Bob Feng <bob.c.feng@intel.com> | Thu Feb 27 22:25:24 2020 +0800 |
committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | Mon Mar 02 02:36:25 2020 +0000 |
tree | 347571ed2df979714e97980ab92aae32525f8928 | |
parent | edfe16a6d9f8c6830d7ad93ee7616225fe4e9c13 [diff] |
BaseTools: Remove invalid leading space before !INCLUDE in Makefile REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2563 This patch is to fix a incremental build regression bug which happen when using nmake. That's introduced by 818283de3f6d. If there is white space before !INCLUDE instruction, nmake will not process it. Source code's dependent header files are listed in ${deps_file} file, if it's not included successfully, nmake will not detect the change of those header file. This patch has been verified in Windows with VS2015 and Linux with GCC5. The header file add/modify/delete can trig the incremental build with this fix. There is no impact on the clean build. Cc: Andrew Fish <afish@apple.com> Cc: Laszlo Ersek <lersek@redhat.com> Cc: Leif Lindholm <leif@nuviainc.com> Cc: Michael D Kinney <michael.d.kinney@intel.com> Cc: Pierre Gondois <pierre.gondois@arm.com> Signed-off-by: Bob Feng <bob.c.feng@intel.com> Acked-by: Laszlo Ersek <lersek@redhat.com> Reviewed-by: Liming Gao <liming.gao@intel.com> Tested-by: Liming Gao <liming.gao@intel.com>
A modern, feature-rich, cross-platform firmware development environment for the UEFI and PI specifications from www.uefi.org.
The majority of the content in the EDK II open source project uses a BSD-2-Clause Plus Patent License. The EDK II open source project 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.
To make a contribution to a TianoCore project, follow these steps.
Create a change description in the format specified below to use in the source control commit log.
Your commit message must include your Signed-off-by
signature
Submit your code to the TianoCore project using the process that the project documents on its web page. If the process is not documented, then submit the code on development email list for the project.
It is preferred that contributions are submitted using the same copyright license as the base project. When that is not possible, then contributions using the following licenses can be accepted:
For documentation:
Contributions of code put into the public domain can also be accepted.
Contributions using other licenses might be accepted, but further review will be required.
Your change description should use the standard format for a commit message, and must include your Signed-off-by
signature.
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.
The test for this is as specified in the Developer's Certificate of Origin (DCO) 1.1. The contributor certifies compliance by adding a line saying
Signed-off-by: Developer Name developer@example.org
where Developer Name
is the contributor's real name, and the email address is one the developer is reachable through at the time of contributing.
Developer's Certificate of Origin 1.1 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. (d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.
From: Contributor Name <contributor@example.com> Subject: [Repository/Branch PATCH] Pkg-Module: Brief-single-line-summary Full-commit-message Signed-off-by: Contributor Name <contributor@example.com>
[Repository/Branch PATCH]
. The remaining portion of the commit message is the email’s content.git format-patch
is one way to create this formatRepository
is the identifier of the repository the patch applies. This identifier should only be provided for repositories other than edk2
. For example edk2-BuildSpecification
or staging
.Branch
is the identifier of the branch the patch applies. This identifier should only be provided for branches other than edk2/master
. For example edk2/UDK2015
, edk2-BuildSpecification/release/1.27
, or staging/edk2-test
.Module
is a short identifier for the affected code or documentation. For example MdePkg
, MdeModulePkg/UsbBusDxe
, Introduction
, or EDK II INF File Format
.Brief-single-line-summary
is a short summary of the change.Full-commit-message
a verbose multiple line comment describing the change. Each line should be less than ~70 characters.Signed-off-by
is the contributor's signature identifying them by their real/legal name and their email address.Submodule in EDK II is allowed but submodule chain should be avoided as possible as we can. Currently EDK II contains the following submodules
ArmSoftFloatLib is actually required by OpensslLib. It‘s inevitable in openssl-1.1.1 (since stable201905) for floating point parameter conversion, but should be dropped once there’s no such need in future release of openssl.
To get a full, buildable EDK II repository, use following steps of git command
$ git clone https://github.com/tianocore/edk2.git $ cd edk2 $ git submodule update --init $ cd ..
If there's update for submodules, use following git commands to get the latest submodules code.
$ cd edk2 $ git pull $ git submodule update
Note: When cloning submodule repos, ‘--recursive’ option is not recommended. EDK II itself will not use any code/feature from submodules in above submodules. So using ‘--recursive’ adds a dependency on being able to reach servers we do not actually want any code from, as well as needlessly downloading code we will not use.