Just wait on this

Ill be adding to neo-cli instead, that will fix it. No rushed need for revert. If no release if due to out come this minute. Lets fix the problem 1st.

There isnt enough information. given for revert

The fact is that without neo-project/neo-modules@e9e7a07 the module does not build.
But even with it is produces error when running, missing some dependencies. (it just occurs on nodes with RpcServer)

I forced added rpcserver published dlls into client (everything built with same framework and same environment), and still some errors.

Plus in this introduces a new problem

The fact is that without neo-project/neo-modules@e9e7a07 the module does not build. But even with it is produces error when running, missing some dependencies. (it just occurs on nodes with RpcServer)

I forced added rpcserver published dlls into client (everything built with same framework and same environment), and still some errors.

YES THIS HAS TO DUE WITH neo-cli. and it will be fixed in a minute. Add this to neo-cli not neo.dll

btw if test are broken then github action test should be more thorough

https://github.com/neo-project/neo-modules/pull/864/files

Has to do with the host not targeting right framework

The fact is that without neo-project/neo-modules@e9e7a07 the module does not build. But even with it is produces error when running, missing some dependencies. (it just occurs on nodes with RpcServer)
I forced added rpcserver published dlls into client (everything built with same framework and same environment), and still some errors.

YES THIS HAS TO DUE WITH neo-cli. and it will be fixed in a minute. Add this to neo-cli not neo.dll

Sure, if there is a good solution.

Just do not merge other PRs until we solve this.
I created another one on neo-modules in case this one is merged.
If not, let's just close it.

Has to do with the host not targeting right framework

In the case I have here, as we discussed recently, we are targeting net7.0.
Building everything (neo-cli and modules) and publishing with FROM mcr.microsoft.com/dotnet/sdk:7.0.404-jammy

Running with FROM mcr.microsoft.com/dotnet/aspnet:7.0.14-jammy

I can try with another commands. The setup here is simple to modify, everything is one line command and dockerized.

as in host I mean the one running the process. In this case its neo-cli . Is it the same for you?

see #3067, close this.

Plus don't forget 12-hour waiting period before merge....

Plus don't forget 12-hour waiting period before merge....

Plus, don't forget to push PRs that do not break everything...aehuahuea

The other PR fixed, @shargon

@vncoelho this works with neo-node and neo-gui, and future clients, because plugins are used by neo, but you choose @vncoelho

Has to do with the host not targeting right framework

The plugin is installed by the host, but is used by the framework, In my opinion, until we solve the dependency problem, we should revert it and find the good solution (not move the problem to another place)

why not update the exe projects. It's only 3 lines of code.

I don't understand why we are moving backward here. You have no good reasons. It's not like an release is around the corner.
You want things in sequences and PR #3067 does just that.

You have no good reasons.

The other PR is not a fix, it moves the problem, the real problem is load the dependencies from the plugin folder, this only moves the problem to another place

You want things in sequences and PR #3067 does just that.

Plus we dont build GUI anymore for release. Since 3.6.0 No GUI In Release

@shargon ,I really appreciate your attention.
But I am not expert in these libraries and frameworks to decide.

In fact,maybe revert is good decision.
As I understood, neo-cli needs its dll during execution. And plugins need aspnet framework for compilation.

What is not clear to me is the image @cschuchardt88 is mentioning.

Wait a sec, if this is reverted, isn't it mean we still using core instead of standard? @cschuchardt88

Wait a sec, if this is reverted, isn't it mean we still using core instead of standard? @cschuchardt88

We are trying to find the good solution

Its RestServer plugin its autoloading stuff it shouldnt

AFTER TESTING EVERYTHING THERE IS NO PROBLEMS!!! IT'S JUST USER ERROR!!!

@vncoelho

We will fix exes or modules instead of neo

@shargon we need it in both