Skip to content

Conversation

@JohnMcPMS
Copy link
Member

@JohnMcPMS JohnMcPMS commented Nov 10, 2025

CP of #5859

Change

The primary motivation is to support directories below the install location to contain configuration units that we will associate with the package. This is achieved by refactoring the association logic from a Package x Unit loop into a tree structure that is colored by package install locations. This also has the benefit of making a O(N^2) algorithm into an O(N).

Units are first inserted into the tree based on their file path. Then the install location of each package is recorded onto that tree as well. Finally, during the export of each package, all resources at the install location and any that are descended from it but not under another package are included.

Microsoft Reviewers: Open in CodeFlow

…nstall location (microsoft#5859)

## Change
The primary motivation is to support directories below the install
location to contain configuration units that we will associate with the
package. This is achieved by refactoring the association logic from a
Package x Unit loop into a tree structure that is colored by package
install locations. This also has the benefit of making a O(N^2)
algorithm into an O(N).

Units are first inserted into the tree based on their file path. Then
the install location of each package is recorded onto that tree as well.
Finally, during the export of each package, all resources at the install
location and any that are descended from it but not under another
package are included.
@JohnMcPMS JohnMcPMS merged commit 53044ac into microsoft:release-v1.12 Nov 10, 2025
4 checks passed
@JohnMcPMS JohnMcPMS deleted the 5859-on-12 branch November 10, 2025 22:16
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.

2 participants