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

UpmGitExtension doesn't appear or works in 2021.1+ versions #113

Closed
HanzaRu opened this issue Apr 14, 2021 · 11 comments
Closed

UpmGitExtension doesn't appear or works in 2021.1+ versions #113

HanzaRu opened this issue Apr 14, 2021 · 11 comments
Assignees
Labels
bug Something isn't working

Comments

@HanzaRu
Copy link

HanzaRu commented Apr 14, 2021

UpmGitExtension works fine on the latest version of 2020.3.3f1 and above, but something on unity has changed on the 2021.1 versions and beyond and UpmGitExtensions simply don't show up on them. I don't know for sure how much work is required to make it compatible but seems just an API update requirement with #ifdef UNITY_2021_OR_ABOVE.

@HanzaRu HanzaRu added the bug Something isn't working label Apr 14, 2021
@Lunar2kPS
Copy link

Awww... dang.

Yeah, it doesn't seem to have changed the Package Manager UI at all for me on Unity 2021.1.0f1.
Upon installing the UPM Git Extension (version 1.3.4) from Git URL, I got this in the Unity Console (even after a restart):

System.NullReferenceException: Object reference not set to an instance of an object
  at Coffee.CSharpCompilerSettings.ReflectionExtensions.Call (System.Object self, System.String methodName, System.Object[] args) [0x0003e] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 
  at Coffee.CSharpCompilerSettings.Utils.RequestCompilation (System.String assemblyName) [0x0007c] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 
  at Coffee.CSharpCompilerSettings.Core.DirtyScriptsIfNeeded () [0x00059] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 
  at Coffee.CSharpCompilerSettings.Core..cctor () [0x0027f] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 
UnityEditor.EditorAssemblies:ProcessInitializeOnLoadAttributes (System.Type[])

@nauroman
Copy link

Doesn't work with Unity 2021.1.4f1

System.NullReferenceException: Object reference not set to an instance of an object at Coffee.CSharpCompilerSettings.ReflectionExtensions.Call (System.Object self, System.String methodName, System.Object[] args) [0x0003e] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 at Coffee.CSharpCompilerSettings.Utils.RequestCompilation (System.String assemblyName) [0x0007c] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 at Coffee.CSharpCompilerSettings.Core.DirtyScriptsIfNeeded () [0x00063] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 at Coffee.CSharpCompilerSettings.Core..cctor () [0x0027f] in <64ac538fcfe94cc8acaaea1c65f42fdf>:0 UnityEditor.EditorAssemblies:ProcessInitializeOnLoadAttributes (System.Type[])

@BruceKristelijn
Copy link

This issue is still a thing. I would like to take a shot to solve it but it seems to come from a dll so I am not sure what the best approach is.

@jason-yak
Copy link

Yup still an issue in 2021.1.10, would love to contribute. Cool project, but yea can't seem to help on this one. Hoping a fix comes along soon so I can try this out.

@HanzaRu
Copy link
Author

HanzaRu commented Jul 8, 2021

Seems the author is out of reach. @mob-sakai there is something we can help or do you have any statement about the future of this project?

@Blackclaws
Copy link

The reason as far as I am able to glean from lookign through the project code is that it uses OpenSesame a different project by the same author which modifies the CSharp Compiler in Unity to allow access to Unity Internals which are being modified by this package. OpenSesame is NOT compatible right now with 2021 versions as they have significantly changed the compilation process it seems:

mob-sakai/CSharpCompilerSettingsForUnity#11

@bassmit
Copy link

bassmit commented Jul 16, 2021

Guys the extensions are not needed in latest lts and beyond

@Blackclaws
Copy link

How is it not needed? The package manager in 2021 only allows adding of git urls, not a lookup of versions or anything similar. Also does 2021 autoresolve git dependencies of git repos? I thought it didn't

@GeorgeGkas
Copy link

@Blackclaws you're right, the current LTS version (2021) does not automatically resolve, git dependencies of git packages. If I host a package in git and use other git packages as dependencies, the package manager will throw Cannot perform upm operation: Unable to add package error. This tool is crucial for small projects and teams.

@HanzaRu
Copy link
Author

HanzaRu commented Sep 16, 2021

Since I migrated to use direct URLs inside our projects, I will close this issue, but the problem still persists for future users of this package.

@HanzaRu HanzaRu closed this as completed Sep 16, 2021
@Blackclaws
Copy link

@john-rosa-playkids Please reopen this issue. As you said it exists still and is NOT fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

9 participants