-
Notifications
You must be signed in to change notification settings - Fork 50
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
[Fix] iOS builds on Windows with projects located at normal or long path lengths #317
Conversation
Instead of copying the .framework files from the nuget cache into the project use them directly on the NativeReference task. This fixes an issue where the MSBuild Copy task fails on Windows due it's 260 char limit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @jkasten2)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @tanaynigam)
OneSignalSDK.Xamarin.nuspec
line 4 at r2 (raw file):
<package xmlns="http://schemas.microsoft.com/packaging/2012/06/nuspec.xsd"> <metadata> <version>4.1.5</version>
Version should not be changed in this PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @tanaynigam)
OneSignalSDK.Xamarin.nuspec
line 4 at r2 (raw file):
Previously, jkasten2 (Josh Kasten) wrote…
Version should not be changed in this PR.
Last commit didn't revert this. Looks like, commit d0c0f01 reverted all the other nuspec changes instead.
d0c0f01
to
d6f0ffb
Compare
* Rename Binding resource directory in `.targets` and `.nuspec` file from `content/OneSignalSDK.Xamarin.iOS.resources` to `res/iOS`
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 2 files at r4, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @jkasten2)
Description
One Line Summary
Resolve most iOS builds failing on Windows due Windows due 260 character file path limits.
Details
Motivation
It is common for Xamarin developers to use a Windows machine and a remote macOS machine to build. Builds were failing unless your project's root was an uncommonly short absolute path.
Scope
Only effects iOS builds, only the linking part to
.xcframeworks
.Changes made
.targets
MSBuild task instead directly reference them..target
and.nuspec
files fromcontent\OneSignalSDK.Xamarin.iOS.resources
tores\iOS
to shorten the absolute file pathsExample of error this addresses
Issues this resolves
Testing
Manual testing
Window - remote mac build
Mac
Affected code checklist
Checklist
Overview
Testing
Final pass
This change is![Reviewable](https://camo.githubusercontent.com/1541c4039185914e83657d3683ec25920c672c6c5c7ab4240ee7bff601adec0b/68747470733a2f2f72657669657761626c652e696f2f7265766965775f627574746f6e2e737667)