diff options
| author | Jeroen van Rijn <Kelimion@users.noreply.github.com> | 2021-03-06 23:09:15 +0100 |
|---|---|---|
| committer | Jeroen van Rijn <Kelimion@users.noreply.github.com> | 2021-03-06 23:09:15 +0100 |
| commit | cefde23232fc30a1cc1b9d1bf9bba666efad8b57 (patch) | |
| tree | 3caa1739409b7096362dc54143f6de5644a9a24c /src/main.cpp | |
| parent | 083cec6c88b37001a57eac1ae9377b4ef73cf8a3 (diff) | |
Remove double removal of extension when using the -out option.
When specifying the out parameter, the extension was stripped twice.
If your path contains a ".", this caused issues.
e.g.
cd "C:\Repro\Path With a . In The Name\"
odin run repro.odin -keep-temp-files -out repro.exe
This would cause the files to end up as:
"C:\Repro\Path With a.exe", "C:\Repro\Path With a.ll", "C:\Repro\Path With a.bc" and "C:\Repro\Path With a.obj"
With this patch it works as expected, with or without a . in the file path.
Diffstat (limited to 'src/main.cpp')
| -rw-r--r-- | src/main.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/main.cpp b/src/main.cpp index b1fc8fd35..0897e5fa6 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -875,12 +875,6 @@ bool parse_build_flags(Array<String> args) { String path = value.value_string; path = string_trim_whitespace(path); if (is_build_flag_path_valid(path)) { - #if defined(GB_SYSTEM_WINDOWS) - String ext = path_extension(path); - if (ext == ".exe") { - path = substring(path, 0, string_extension_position(path)); - } - #endif build_context.out_filepath = path_to_full_path(heap_allocator(), path); } else { gb_printf_err("Invalid -out path, got %.*s\n", LIT(path)); |