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

Pull in MAUI repo(s) in order to source-build MAUI manifest nupkgs #3242

Open
directhex opened this issue Feb 10, 2023 · 5 comments
Open

Pull in MAUI repo(s) in order to source-build MAUI manifest nupkgs #3242

directhex opened this issue Feb 10, 2023 · 5 comments
Labels
area-additional-repos Adding additional contributing repos

Comments

@directhex
Copy link
Member

We've now excluded the manifest prebuilts from the installer build, where those manifests don't come from source-build itself. We've also pulled in emsdk to generate some manifests from that repo, and rejiggled runtime to emit runtimes on non-mobile builds. The last set of manifests we need to worry about are the conditional ones on src/redist/targets/BundledManifests.targets:

  • Microsoft.NET.Sdk.Android
  • Microsoft.NET.Sdk.iOS
  • Microsoft.NET.Sdk.MacCatalyst
  • Microsoft.NET.Sdk.macOS
  • Microsoft.NET.Sdk.Maui
  • Microsoft.NET.Sdk.tvOS

Of those six, most of them come from https://github.com/xamarin/xamarin-macios, one from https://github.com/xamarin/xamarin-android, and the final one from https://github.com/dotnet/maui

@dotnet-issue-labeler dotnet-issue-labeler bot added area-additional-repos Adding additional contributing repos untriaged labels Feb 10, 2023
@directhex directhex added the area-prebuilts Reducing the number of prebuilt packages in the tarball label Feb 10, 2023
@MichaelSimons MichaelSimons removed area-prebuilts Reducing the number of prebuilt packages in the tarball untriaged labels Feb 16, 2023
@MichaelSimons
Copy link
Member

[Triage] @directhex - is this something on your radar to fix in the 8.0 timeframe?

@omajid
Copy link
Member

omajid commented Jun 8, 2023

Note to self: this current behaviouir results in a functional difference between a source-built SDK and a non-source-built SDK. dotnet workload list simply wouldn't show the workloads for users to install.

Edit:

As of Preview 6, the following 5 workloads are missing in a source-build SDK:

  • android
  • macos
  • maui-android
  • maui-tizen
  • maui-windows

@directhex
Copy link
Member Author

@MichaelSimons I'm prioritizing getting emsdk/wasi-sdk fully built from source, I'm not sure if I'll get time to work on much else in time.

@webczat
Copy link

webczat commented Feb 6, 2024

ping :)

@klemmchr
Copy link

I'm highly interested in this. This looks like a bug to the end user.

jonathanpeppers added a commit to jonathanpeppers/sdk that referenced this issue Jul 12, 2024
Context: dotnet/source-build#3242

If using a source-built .NET SDK, the Android workload manifests are
not included in the SDK. So, you won't see `android` workload when you
run `dotnet workload search`.

The Android workload, in general, contains private sources, but the
manifest files are completely public.

In the long run, we could consider onboarding dotnet/android to:

* https://github.com/dotnet/source-build/blob/main/Documentation/sourcebuild-in-repos/new-repo.md

But the prerequisites would be to first have the Android workload
onboard to arcade:

* https://github.com/dotnet/arcade/blob/main/Documentation/Onboarding.md

*For now*, we could include the Android workload in source-build by:

* Add a step that copies the Android workload manifests source code
  in-tree (to be used when updating manifests).

* Commit the manifest sources.

* Source-build will then include the Android workload manifests in
  the SDK by using the source code, in-tree.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-additional-repos Adding additional contributing repos
Projects
Status: Backlog
Development

No branches or pull requests

5 participants