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

SctPkg: Drop ArmCompilerIntrinsicsLib #105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nhivp
Copy link
Member

@nhivp nhivp commented Sep 30, 2024

edk2 PR tianocore/edk2#6048 moves
ArmCompilerIntrinsicsLib to MdePkg and into MdeLibs.dsc.inc.

This patch drops all of the references to the ArmPkg location as these *_SCT.dsc are now getting it from MdeLibs.dsc.inc. This is done in one shot as it is a breaking change and has no functional impact.

edk2 PR tianocore/edk2#6048 moves
ArmCompilerIntrinsicsLib to MdePkg and into MdeLibs.dsc.inc.

This patch drops all of the references to the ArmPkg location as
these *_SCT.dsc are now getting it from MdeLibs.dsc.inc. This is
done in one shot as it is a breaking change and has no functional
impact.

Signed-off-by: Nhi Pham <[email protected]>
@anncheng-arm
Copy link

Hi @edhay @sunnywang-arm
This fix looks straightforward and is working well without any build errors. Do you think you could review it and get it merged?

@sunnywang-arm
Copy link
Contributor

Looks good to me as well. For merging the change, we need to get maintainers (https://github.com/tianocore/edk2-test/blob/master/Maintainers.txt) to work on it.

@sunnywang-arm
Copy link
Contributor

By the way, as I know, Edhaya is working on CI/CD for pull requests, so we may need to wait for CI/CD to get ready.

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.

3 participants