search

godotengine / godot

Godot Engine – Multi-platform 2D and 3D game engine
https://godotengine.org
MIT License
91.57k stars 21.27k forks source link

[C#] Assembly export #31390

Closed ghost closed 4 years ago

ghost commented 5 years ago

I am currently trying to build a modding framework on top of the Godot, and I am having some troubles with it. I would like to propose some sort of an additional point in the config/export toolset to allow people to manually decide where to put their assemblies because for my game I would like to put GodotSharp.dll and GodotSharpEditor.dll as well as %GameName%.dll somewhere outside from .pck so other mods could easily reference them. I don't know how possible this is, or how valid of a request this is, but I simply could not find any other way around this problem.

neikeq commented 5 years ago

It's definitely possible and I'm actually considering doing it by default as some libraries (like Microsoft.CodeAnalysis.Scripting) expect the assemblies to be in the file system; and since we already export a data folder with the executable any way, why not place the assemblies there.

mhilbrunner commented 4 years ago

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.

The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.

If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!

neikeq commented 4 years ago

This was actually a duplicate of #29349. This is already implemented by #33982

sericaer commented 3 years ago

This was actually a duplicate of #29349. This is already implemented by #33982

Could you tell me how to use this option? Thanks a lot!