[mono] Add support for backslash escaped colon in MONO_PATH #90526
Conversation
|
Doesn't it strip the |
| char *tmp = *split; | ||
| int len = strlen(tmp); | ||
| // If the current string ends with a backslash, append the next string to it | ||
| while (len > 0 && tmp[len - 1] == '\\') { |
There was a problem hiding this comment.
Windows paths are expected to have backslashes, so this should probably only be executed when !Windows (or when G_SEARCHPATH_SEPARATOR_S == ':')
There was a problem hiding this comment.
I don't think it's necessarily a problem but the escaping mechanism seems rather inconsistent. This detects only backslash at the end of path that was followed by :. In Windows \: is never valid. In Linux it could be but then you can write it as \\: I guess.
There was a problem hiding this comment.
Ah, right, G_SEARCHPATH_SEPARATOR_S is going to be ; on Windows. That makes it a bit more messy.
There was a problem hiding this comment.
Yes, the idea is to escape \\: on non-windows platforms which should allow correct path resolution. I parametrized the concat token and added #ifndef HOST_WIN32 macro. Do you see any other issues with the escaping mechanism?
kotlarmilos
requested review from
lambdageek,
thaystg and
vargaz
as code owners
|
#90544 seems a cleaner approach. |
|
Closing this PR in favor of #90544. |
ghost
locked as resolved and limited conversation to collaborators
4 participants
Fixes #90422