Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

Suggestion: Open a resource avaliable for download as packed DLL or download from nuget. #10

Closed
BeanTang opened this issue Oct 24, 2019 · 3 comments · Fixed by #21
Closed
Labels
enhancement New feature or request

Comments

@BeanTang
Copy link

Because sometime we used to reference a project dll which referenced json.dll for unity.

Such as I have a Project about Integrated the Netwrok communications. And the project reference the json.dll.
Then Build the project to dll which was used by Unity.

@BeanTang BeanTang added the enhancement New feature or request label Oct 24, 2019
@BeanTang
Copy link
Author

I try download the source code to build.
But open the solution ,it throw a lot of dependencies warning and error.

@applejag applejag changed the title Suggestion: Open a resource avaliable for download is packed DLL or download from nuget. Suggestion: Open a resource avaliable for download as packed DLL or download from nuget. Oct 24, 2019
@applejag
Copy link
Owner

Good note. Related to #7. Will consider releasing on NuGet so you can reference it in your class libraries.

For now a possible workaround is as I mentioned in #7:

In a different project of mine I did a workaround where I referenced the /Src/Newtonsoft.Json/Newtonsoft.Json.csproj project from this repo instead of the one from NuGet. That way when it compiled it did so without the strong name reference.

This was referenced Dec 30, 2019
@applejag applejag assigned applejag and unassigned applejag Dec 30, 2019
@applejag
Copy link
Owner

From release 12.0.102 the package will be signed and a NuGet publish will no longer be needed. If you need to use Newtonsoft.Json outside of Unity, such as in a class library developed in Visual Studio, then use the official Newtonsoft.Json. Newtonsoft.Json-for-Unity is only a replacement for Newtonsoft.Json within Unity.

This issue will be closed due to the suggestions motivation having been dismissed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants