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

Package Manager package support, .net 4, Unity 2019, Git for Unity, and more #1070

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

shana
Copy link
Member

@shana shana commented Aug 13, 2019

This is a large refactor to reuse the api source in https://github.com/Unity-Technologies/Git-for-Unity (via a fork in https://github.com/spoiled/Git-for-Unity), so this repo will only concern itself with building the UI.

It also changes the build system to output Package Manager packages, with .net 35 (for Unity 5.6 and above targeting the .net 2.0 profile), and .net 471 (for 2018+ targeting the .net 4 profile). Because there's so many changes to how things are packaged, there's no longer a loader trying to guess which DLLs to load/unload based on the active profile. Instead, there are separate packages for each profile.

This also indirectly switches to using dugite git packages on all platforms, and adds asmdef to all projects.

Fixes #1070 #1068 #1053 #1018 #962 #456

shana added 30 commits August 2, 2019 16:11
…for-Unity

https://github.com/spoiledcat/Git-for-Unity is now the authoritative source for
Git for Unity, so we only need to build the UI bits on top of that library.
Most of the work for Git for Unity happens at the
https://github.com/Unity-Technologies/Git-for-Unity fork, and the
spoiledcat/Git-for-Unity master repo is for hosting lfs, tracking stable
branches, and so I can actually have control over who does what on it.
@Ike-Mac
Copy link

Ike-Mac commented Oct 8, 2019

Any ETA when this will be merged to master? I'm in the process of upgrading. Thxs :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants