Skip to content

Features/ManageabilityPkg: Move Package from edk2-platforms to edk2#922

Open
PrachotanReddy wants to merge 1 commit intotianocore:masterfrom
PrachotanReddy:manageabilitypkg
Open

Features/ManageabilityPkg: Move Package from edk2-platforms to edk2#922
PrachotanReddy wants to merge 1 commit intotianocore:masterfrom
PrachotanReddy:manageabilitypkg

Conversation

@PrachotanReddy
Copy link

ManageabilityPkg is not platform-specific and is consumed as a general-purpose package. Move it from edk2-platforms into the edk2 tree to better reflect its scope and reduce cross-repo dependencies.

@abdattar
Copy link
Contributor

abdattar commented Jan 8, 2026

looks good to me, usually large changes will be first ported edk2-staging branch first before merging to edk2 master branch.
@changab any comments.

@PrachotanReddy
Copy link
Author

checking in to see if there's any other comments that need to be resolved to close this PR

@mdkinney
Copy link
Member

There are many general purpose feature packages in edk2-platforms.

What is the reason that this one package should be moved?

What is the criteria being used to propose moving package from edk2-platforms to edk2?

@PrachotanReddy
Copy link
Author

This is a host standard interface for manageability and therefore should be considered a core feature rather than a platform feature. Additionally this allows downstream platforms not on edk2-platforms to consume this directly.
@gtossk2 @samerhaj

@changab
Copy link
Member

changab commented Jan 21, 2026

We must ensure that all platforms under edk2-platform have the correct references of ManageabiltyPkg on edk2.

  1. Let's get ManageabilityPkg merged to edk2 first.
  2. Then update all platforms to use ManageabiltyPkg from edk2.
  3. Remove ManageabiltyPkg on edk2-platform.

However, @mdkinney, do you agree with this migration?

ManageabilityPkg is not platform-specific and is consumed
as a general-purpose package. Move it from edk2-platforms into the
edk2 tree to better reflect its scope and reduce cross-repo
dependencies.
Allow downstream platforms (not in edk2-platforms) to also consume it
directly from edk2.

Signed-off-by: Prachotan Bathi <prachotan.bathi@arm.com>
Copy link
Contributor

@abdattar abdattar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@changab
Copy link
Member

changab commented Jan 27, 2026

@PrachotanReddy, I will give my approval and merge this change once we update all platforms that reference ManageabilityPkg under ekd2-platforms to edk2.
Thanks for this contribution.

@mdkinney
Copy link
Member

Please do not merge yet. I am still running some tests.

@changab
Copy link
Member

changab commented Feb 15, 2026

Please check here: #942 for the edk2 CI fix and address the feedback.
@PrachotanReddy @abdattar @mdkinney @leiflindholm @ardbiesheuvel @makubacki

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.

4 participants