Stowage/godot
2
0
Fork 0
mirror of https://github.com/godotengine/godot.git synced 2025-11-02 06:31:13 +00:00
Code Issues Projects Releases Packages Wiki Activity

Mono/C#: Re-structure API solution and GodotTools post-build target

Browse source 
Previously we had a placeholder solution called 'Managed' to benefit from
tooling while editing the a part of the C# API.
Later the bindings generator would create the final 'GodotSharp' solution
including these C# files as well as the auto-generated C# API.
Now we replaced the 'Managed' solution with the final 'GodotSharp' solution
which is no longer auto-generated, and the bindings generator only takes
care of the auto-generated C# API.
This has the following benefits:
- It's less confusing as there will no longer be two versions of the same file
(the original and a generated copy of it). Now there's only one.
- We no longer need placeholder for auto-generated API classes, like Node or
Resource. We used them for benefiting from tooling. Now we can just use the
auto-generated API itself.
- Simplifies the build system and bindings generator. Removed lot of code
that is not needed anymore.

Also added a post-build target to the GodotTools project to copy the output to
the data dir. This makes it easy to iterate when doing changes to GodotTools,
as SCons doesn't have to be executed anymore just to copy these new files.
This commit is contained in:
Ignacio Etcheverry 2019-12-28 19:12:32 +01:00
parent 318c693516
commit 86274b9fc9
72 changed files with 350 additions and 674 deletions
Download patch file Download diff file Expand all files Collapse all files

37
modules/mono/utils/path_utils.cpp
View file

@ -170,4 +170,41 @@ String join(const String &p_a, const String &p_b, const String &p_c, const Strin
return path::join(path::join(path::join(p_a, p_b), p_c), p_d);
}
String relative_to_impl(const String &p_path, const String &p_relative_to) {
// This function assumes arguments are normalized and absolute paths
if (p_path.begins_with(p_relative_to)) {
return p_path.substr(p_relative_to.length() + 1);
} else {
String base_dir = p_relative_to.get_base_dir();
if (base_dir.length() <= 2 && (base_dir.empty() || base_dir.ends_with(":")))
return p_path;
return String("..").plus_file(relative_to_impl(p_path, base_dir));
}
}
#ifdef WINDOWS_ENABLED
String get_drive_letter(const String &p_norm_path) {
int idx = p_norm_path.find(":/");
if (idx != -1 && idx < p_norm_path.find("/"))
return p_norm_path.substr(0, idx + 1);
return String();
}
#endif
String relative_to(const String &p_path, const String &p_relative_to) {
String relative_to_abs_norm = abspath(p_relative_to);
String path_abs_norm = abspath(p_path);
#ifdef WINDOWS_ENABLED
if (get_drive_letter(relative_to_abs_norm) != get_drive_letter(path_abs_norm)) {
return path_abs_norm;
}
#endif
return relative_to_impl(path_abs_norm, relative_to_abs_norm);
}
} // namespace path