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

Dependency version warnings most likely resulting in build error #180

Open
Khidezeeh opened this issue Mar 14, 2021 · 3 comments
Open

Dependency version warnings most likely resulting in build error #180

Khidezeeh opened this issue Mar 14, 2021 · 3 comments

Comments

@Khidezeeh
Copy link

Describe the bug
By following the guides on how to build both Els_kom_new as well as Elskom.Sdk I encounter an issue where trying to build Els_kom_new (either via Visual Studio 2019/2019 Preview) will result in errors most likely related to the version of dependencies used is below the required ones.

To Reproduce
Build Elskom.Sdk as explained on the above mentioned guide and add the directories to NuGet.
Proceed then to build Els_kom_new as explained on the above mentioned guide
EIther open up the solution via Visual Studio or run the command
Notice the warnings:

SettingsFile 2.0.0 depends on System.Text.Json (>= 6.0.0-preview.3.21162.11) but System.Text.Json 6.0.0-preview.3.21162.11 was not found. An approximate best match of System.Text.Json 6.0.0-preview.3.21163.4 was resolved.	Els_kom
Microsoft.Extensions.DependencyInjection (>= 6.0.0-preview.3.21162.11) but Microsoft.Extensions.DependencyInjection 6.0.0-preview.3.21162.11 was not found. An approximate best match of Microsoft.Extensions.DependencyInjection 6.0.0-preview.3.21163.4 was resolved.

Notice the errors:

`string[]' does not contain a definition for 'Source' and no accessible extension method 'Source' accepting a first argument of type 'string[]' could be found (are you missing a using directive or an assembly reference?)	Els_kom (net472), Els_kom (net5.0-windows)

Most likely due to this, this error will pop up which leads to the build to fail. It might be worth mentioning that I was unable to find the versions required, even when looking into future versions of such dependencies.

Expected behavior
The solution is sucessfully built.

Screenshots
Screenshot 2021-03-14 093937
Screenshot 2021-03-14 093954

@AraHaan
Copy link
Contributor

AraHaan commented Mar 20, 2021

Wait what?, the Sdk should have been the only place to being in both of those packages, and then the program should use whatever version the compiled binaries used.

Also I changed some things in SettingsFile as I have had bugs with the plugins source urls randomly getting wiped out for some reason on the json settings file.

@AraHaan
Copy link
Contributor

AraHaan commented Apr 19, 2021

The build errors was infact from my changes to the SettingsFile project in the Sdk. And that was due to the fact that it had bugs where the source urls would somehow get wiped from the json file every time it goes to try to load/save them.

@AraHaan
Copy link
Contributor

AraHaan commented Nov 6, 2021

I currently fixed the build errors locally by making the runtime into a .NET Workload which should make it easier to build the program as well.

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

No branches or pull requests

2 participants