Skip to content

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

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

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

Conversation

@PrachotanReddy
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
Author

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

@mdkinney
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

@changab
Copy link
Copy Markdown
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

@changab
Copy link
Copy Markdown
Member

changab commented Apr 16, 2026

@PrachotanReddy , I think we can withdraw this PR, use #942 and
tianocore/edk2#12162 instead.

@PrachotanReddy PrachotanReddy deleted the manageabilitypkg branch April 16, 2026 19:47
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