Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UefiPayloadPkg: Testing CI build #10749

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ChaselChiu
Copy link
Contributor

Testing CI build

Description

<Include a description of the change and why this change was made.>

<For each item, place an "x" in between [ and ] if true. Example: [x] (you can also check items in GitHub UI)>

<Create the PR as a Draft PR if it is only created to run CI checks.>

<Delete lines in <> tags before creating the PR.>

  • Breaking change?
    • Breaking change - Does this PR cause a break in build or boot behavior?
    • Examples: Does it add a new library class or move a module to a different repo.
  • Impacts security?
    • Security - Does this PR have a direct security impact?
    • Examples: Crypto algorithm change or buffer overflow fix.
  • Includes tests?
    • Tests - Does this PR include any explicit test code?
    • Examples: Unit tests or integration tests.

How This Was Tested

<Describe the test(s) that were run to verify the changes.>

Integration Instructions

<Describe how these changes should be integrated. Use N/A if nothing is required.>

@ChaselChiu ChaselChiu force-pushed the upl_ci branch 2 times, most recently from 25304fd to cda5f7f Compare February 14, 2025 02:43
@ChaselChiu ChaselChiu force-pushed the upl_ci branch 2 times, most recently from 71c05d6 to cd7a5dc Compare February 21, 2025 00:42
Create a new PlatformBuild.py that can be called by stuart_build.
UniversalPayloadBuild.py is consumed by new script to maintain backward
compatibility and eliminate duplicate build steps.

Supported builds:
1. UEFI package build: directly call BaseTools build for EFI bin
2. UPL ELF/FIT build: call UniversalPayloadBuild.py
3. (new) UPL ELF - stuart build: follow stuart build steps to call
   PlatformBuild.py (edk2toolext is mandatory)

Signed-off-by: Chasel Chiu <[email protected]>
When building with "build -p pkgname/pkgname.dsc -a IA32 -a X64",
it will fail as BUILD_ARCH was not defined and output directory was
unexpectedly as "UefiPayloadPkg$(BUILD_ARCH)"

As many platforms have adopted UPL build (UniversalPayloadBuild.py)
which always defines "BUILD_ARCH", the fix is mainly to keep this
backward compatibility (BUILD_ARCH always defined), but also support
BUILD_ARCH undefined scenario.

Signed-off-by: Chasel Chiu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant