Re: git build on msys fails (and fix)

On Mon, Dec 1, 2008 at 2:36 PM, Johannes Sixt [off-list ref] wrote:

dhruva schrieb:

quoted

Hello,
 I build git on msys environment regularly. With the recent changes to
add cache preload facility (commit ID:
671c9b7e315db89081cc69f83a8f405e4aca37bc) brings in dependency on
pthreads. This is not available in the msys distribution which was
part of the git build distro. I installed libpthread for mingw to get
the related headers and libraries. I had to make some minor changes to
libpthread installation. I copied the 'setjmp.h' from my other mingw
installation and had to had edit it to remove some mingw specific
macros.

@:/e/users/dhruva/stub/repo/git/git
[520]$ diff -u /e/tools/gnu/include/setjmp.h /mingw/include/setjmp.h

--- /e/tools/gnu/include/setjmp.h       Sun Oct  5 04:01:15 2008
+++ /mingw/include/setjmp.h     Sun Nov 30 16:18:00 2008

@@ -36,7 +36,7 @@
  * The function provided by CRTDLL which appears to do the actual work
  * of setjmp.
  */
-_CRTIMP int __cdecl __MINGW_NOTHROW _setjmp (jmp_buf);
+int __cdecl _setjmp (jmp_buf);

 #define        setjmp(x)       _setjmp(x)

@@ -44,7 +44,7 @@
  * Return to the last setjmp call and act as if setjmp had returned
  * nVal (which had better be non-zero!).
  */
-_CRTIMP void __cdecl __MINGW_NOTHROW longjmp (jmp_buf, int)

__MINGW_ATTRIB_NORETURN;
+void __cdecl longjmp (jmp_buf, int);

 #ifdef __cplusplus
 }

Also,
 had to copy a pthread library with a dfferent name to the standard name:
$cp libpthreadGC2.a libpthread.a

The build went fine and am using the new binaries.

Would you please push your addition to the mob branch of
git://repo.or.cz/msysgit.git? The push URL is repo.or.cz:/srv/git/msysgit.git

(But notice that some commits are on the mob branch currently that are not
in any other branch; hence, you better build on top of them so that they
are not lost when you push to mob.)

-- Hannes

I am not using git from the above repo. I am using it from:
git://git2.kernel.org/pub/scm/git/git.git

I will clone from the repo you have mentioned and push my changes,
that will take some time. I have started the clone now...

-dhruva

-- 
Contents reflect my personal views only!