Re: [PATCH 1/1] mingw: handle absolute paths in expand_user_path()
- Date: Tue, 6 Nov 2018 16:10:38 +0000
- From: Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxxx>
- Subject: Re: [PATCH 1/1] mingw: handle absolute paths in expand_user_path()
On 06/11/2018 15:54, Ramsay Jones wrote:
> On 06/11/2018 14:53, Johannes Schindelin via GitGitGadget wrote:
>> From: Johannes Schindelin <johannes.schindelin@xxxxxx>
>> On Windows, an absolute POSIX path needs to be turned into a Windows
>> Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx>
>> path.c | 5 +++++
>> 1 file changed, 5 insertions(+)
>> diff --git a/path.c b/path.c
>> index 34f0f98349..a72abf0e1f 100644
>> --- a/path.c
>> +++ b/path.c
>> @@ -11,6 +11,7 @@
>> #include "path.h"
>> #include "packfile.h"
>> #include "object-store.h"
>> +#include "exec-cmd.h"
>> static int get_st_mode_bits(const char *path, int *mode)
>> @@ -709,6 +710,10 @@ char *expand_user_path(const char *path, int real_home)
>> if (path == NULL)
>> goto return_null;
>> +#ifdef __MINGW32__
>> + if (path == '/')
>> + return system_path(path + 1);
> Hmm, this doesn't quite fit with the intended use of this
> function! ;-) (even on windows!)
> I haven't looked very deeply, but doesn't this affect all
> absolute paths in the config read by git_config_pathname(),
> along with all 'included config' files?
> I am pretty sure that I would not want the absolute paths
> in my config file(s) magically 'moved' depending on whether
> git has been compiled with 'runtime prefix' support or not!
So, I hit 'send' before finishing my thought ...
I can't think of a non-backwards compatible way of doing
what you want. If backward compatibility wasn't an issue,
then we could (maybe) have used some kind of pathname prefix
like 'system:/path/relative/to/git/executable', or somesuch.