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

Allow forced building of mobile manifest pkgs, outside mobile builds #80920

Merged
merged 1 commit into from
Jan 25, 2023

Conversation

directhex
Copy link
Contributor

We may need to build our manifests separately from our mobile packages, this change allows us to force the build

@ghost
Copy link

ghost commented Jan 20, 2023

Tagging subscribers to this area: @directhex
See info in area-owners.md if you want to be subscribed.

Issue Details

We may need to build our manifests separately from our mobile packages, this change allows us to force the build

Author: directhex
Assignees: -
Labels:

area-Infrastructure-mono

Milestone: -

@steveisok steveisok requested a review from lewing January 20, 2023 17:19
@steveisok
Copy link
Member

@lewing this is geared towards allowing source build to build the manifests (and only the manifests) on linux.

@directhex directhex merged commit 51a36ff into dotnet:main Jan 25, 2023
directhex added a commit that referenced this pull request Feb 6, 2023
* Split manifest building from mono-packages.proj into a standalone subset, in order to fix up #80920 so building manifests works when the runtime is CoreCLR (as is typically the case on source-build)
steveisok pushed a commit that referenced this pull request Feb 7, 2023
…ubset (#81719)

Backport of #81598

Fix up #80920 so building manifests works when the runtime is CoreCLR (as is typically the case on source-build)

Co-authored-by: Jo Shields <directhex@apebox.org>
@ghost ghost locked as resolved and limited conversation to collaborators Feb 24, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants