Skip to content

BaseTools/build.py: Use full source file path for dependency generation#12301

Closed
Kowsik-003 wants to merge 3 commits intotianocore:masterfrom
Kowsik-003:Kowsik/Relative_Path
Closed

BaseTools/build.py: Use full source file path for dependency generation#12301
Kowsik-003 wants to merge 3 commits intotianocore:masterfrom
Kowsik-003:Kowsik/Relative_Path

Conversation

@Kowsik-003
Copy link
Copy Markdown

@Kowsik-003 Kowsik-003 commented Mar 17, 2026

Description

In .inf under [Sources] section relative paths of source files were not considered for duplicate checks during .deps file generation. Modified changes to consider relative paths mentioned in .inf for .deps creation.

  • 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

When building UefiCpuPkg, can't find that deps file created for all the AmdSev.c files in the respective paths.

Integration Instructions

NA

In .inf under [Sources] section relative paths of source files were not
considered for duplicate checks during .deps file generation.
Modified changes to consider relative paths mentioned in .inf for .deps
creation.

Signed-off-by: kowsiks <kowsiks@ami.com>
@Kowsik-003 Kowsik-003 force-pushed the Kowsik/Relative_Path branch from 3363a74 to 3d24561 Compare March 23, 2026 07:32
@Kowsik-003 Kowsik-003 closed this Mar 31, 2026
@Kowsik-003 Kowsik-003 deleted the Kowsik/Relative_Path branch March 31, 2026 08:03
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