-
Notifications
You must be signed in to change notification settings - Fork 445
Conversation
Don't stress yourself too much about |
And now that you have done work on it, I also feel bad suggesting removing it... 🎖 Kudos though. |
I'm not stressed about it at all 😄 Actually in the .cmds I just added the 3rd TargetFramework parameter, I've not done the changes once I noticed it was outdated in other features too. |
Had a quick flick through the code and looks solid. On the note of not being able to have two versions of the framework in one package - is it worth discussing when you would drop support of 3.5 in the future? I guess the earlier this happens the better in the long run for developers. |
Thanks for the patch! This isn't forgotten, I found a way to switch between versions depending on which runtime you're on, at least until 2018.3 when hopefully Unity will support configuring that out of the box, so I'll be pulling this in to that fix this week 👍 |
Oh that's nice, I'm just curious to see how you made that. :) |
Any news on this? |
It should be in the next release. We've been juggling a few projects and had to get the authentication fixes in this release taken care of first. Hang in there please. |
Ok so I figured out the way to do a side by side ship of both profiles in Unity, which is #889. It doesn't require any changes to the build scripts, it's just shipping 4.6 versions of some dlls and a loader to switch between them. This should Just Work(tm) |
Allows to compile a Unity 2018 .NET 4.5+ compatible package, as described in #824
Description of the Change
./package.sh /Applications/Unity Release v4.5
.netXX
to the.unitypackage
file nameAsyncBridge.Net35.dll, ReadOnlyCollectionsInterfaces.dll, System.Threading.dll
)TaskEx.cs
files to directly call the system functionspackage.sh
to clean up the .meta out out the.unitypackage
Possible Drawbacks
package.cmd
still has two TODO itens to match the functionality with the.sh