Re: [msysGit] Bugreport: msysGit-fullinstall-1.8.0-preview20121022.exe error

[Erik Faye-Lund]

First of all, thanks for a good bug report. Notes are inline.

On Wed, Nov 14, 2012 at 9:56 AM, <barry...@gmail.com> wrote:
> I just downloaded and installed
> msysGit-fullinstall-1.8.0-preview20121022.exe (over my older Git
> installation in "C:\Program Files\Git\" ) and it stopped with this error:
>

You are aware of the difference between Git for Windows and msysGit, right?
Git for Windows is intended for people who just want to use Git, while
msysGit is for people who want to modify Git for Windows.

If you just want to use Git, you can probably work around this issue
by using the Git for Windows installer rather than the msysGit
installer.

> RC git.res
> 'C:\Program' is not recognized as an internal or external command,
> operable program or batch file.
> C:\Program Files\Git\msysGit\mingw\bin\windres.exe: preprocessing failed.
> make: *** [git.res] Error 1

Huh. This seems to be because you chose to install to a path with
spaces in it, the absolute path was used for windres, and the build
system didn't quote that path properly.

But I don't understand *how* we came to the last two bits. When I look
at the windres-bits in the build system, we seem to use relative paths
only. windres is picked from the system-path, without any extra magic.

Other msysGit people: Was this changed between
msysGit-fullinstall-1.8.0-preview20121022.exe and the current devel?
The only difference I can see is 1cbcfae (wincred: add install target
and avoid overwriting configured variables), and it doesn't seem
relevant...

[Pat Thoyts]

On 14 November 2012 09:18, Erik Faye-Lund <kusm...@gmail.com> wrote:
>
> But I don't understand *how* we came to the last two bits. When I look
> at the windres-bits in the build system, we seem to use relative paths
> only. windres is picked from the system-path, without any extra magic.
>
> Other msysGit people: Was this changed between
> msysGit-fullinstall-1.8.0-preview20121022.exe and the current devel?
> The only difference I can see is 1cbcfae (wincred: add install target
> and avoid overwriting configured variables), and it doesn't seem
> relevant...
>

This appears to be a built-in deficiency of 'windres'. If you rename
your msysgit directory to include a space (eg: "msys git") then simply
run 'windres' it reports this error.

pat@develop /git ((v1.8.0.msysgit.0))
$ windres
'C:\src\msys' is not recognized as an internal or external command,

operable program or batch file.

C:\src\msys git\mingw\bin\windres.exe: preprocessing failed.

We do actually point out on
https://github.com/msysgit/msysgit/wiki/InstallMSysGit that you should
not install msysgit into Program Files. We might make that a stonger
warning. You should be running into problems with admin permissions as
well unless you chose to disable the user control stuff.