-
Notifications
You must be signed in to change notification settings - Fork 519
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
Unable to publish net6.0 iOS App #15365
Comments
The first one fails because we need to know the runtime identifier very early in the build process, and at that point we don't know whether we're building or publishing, and we default to a simulator runtime identifier, since that's the most common use case. The second one fails due to NuGet/Home#11653. You should be able to work around this by adding something like this in the project file: <PropertyGroup Condition="'$(IsPublishing)' == 'true' And '$(TargetFramework)' == 'net6.0-ios'">
<RuntimeIdentifier>ios-arm64</RuntimeIdentifier>
</PropertyGroup> and then
|
Might I suggest an update to the MAUI Docs on publishing |
This does seem to resolve the issue however I am only seeing the IPA in the publish directory. Is there a flag we can pass to get the dSYM? |
@dansiegel try passing |
I've updated csproj to automatically set the Runtime based on TargetFramework & Configuration. Then adding the parameter you mentioned I end up with: dotnet publish -f:net6.0-ios -c Release /p:ArchiveOnBuild=true \
/p:ApplicationDisplayVersion=0.1.0 /p:ApplicationVersion=15 \
-o ~/repos/demo/MauiCIDemo/publish /p:NoDSymUtil=false This doesn't seem to make any difference. This results in the default behavior where the dSYM is generated it's just in
What I would expect to see is the dSYM in the same publish directory as the IPA. |
Ah yes, we put the .dSYM next to the .app we build, and then we don't move/copy it elsewhere after that. It should be easy enough to copy it to the publish directory too. |
I've created an enhancement request for this: #15384 (to avoid conflating multiple problems in a single issue here). |
Could you link me to the exact document you read to make sure we're updating the right one? |
@rolfbjarne makes perfect sense to move the dSYM request to its own issue...
https://docs.microsoft.com/en-us/dotnet/maui/ios/deployment/overview#publish A few issues I have with the publishing docs..
|
Ok, that doc is stored here: https://github.com/dotnet/docs-maui/blob/main/docs/ios/deployment/overview.md#publish |
This issue was moved to dotnet/docs-maui#681 |
Steps to Reproduce
This is running the publish from the MacOS Terminal
dotnet publish -f:net6.0-ios -c Debug /p:ArchiveOnBuild=true /p:ApplicationDisplayVersion=0.1.0 /p:ApplicationVersion=15 -o ~/repos/demo/MauiCIDemo/publish
dotnet publish -f:net6.0-ios -c Debug /p:ArchiveOnBuild=true /p:ApplicationDisplayVersion=0.1.0 /p:ApplicationVersion=15 -o ~/repos/demo/MauiCIDemo/publish -r ios-arm64
(also tried with/p:RuntimeIdentifier=ios-arm64
which had the same result)Expected Behavior
The first command should work... The second command should have resolved the error encountered on the first.
Actual Behavior
After running the first command:
After running the second command:
You'll notice btw that while the first error comes out of the
Microsoft.iOS.Sdk
, specifying theios-arm64
runtime actually picks up theMicrosoft.MacCatalyst.Sdk
which is what throws the error for an unrecognized runtime.Environment
dansiegel@dans-mbp MauiCIDemo % dotnet --version 6.0.301 dansiegel@dans-mbp MauiCIDemo % dotnet workload list Installed Workload Ids Installation Source ----------------------------------------------- wasm-tools SDK 6.0.300 macos SDK 6.0.300 maui-tizen SDK 6.0.300 maui-maccatalyst SDK 6.0.300 maui-ios SDK 6.0.300 maui-android SDK 6.0.300 ios SDK 6.0.300 maccatalyst SDK 6.0.300 maui SDK 6.0.300 tvos SDK 6.0.300 tizen SDK 6.0.300 android SDK 6.0.300 Use `dotnet workload search` to find additional workloads to install.
Build Logs
See above
Example Project (If Possible)
The text was updated successfully, but these errors were encountered: