Re: libtool wrapper: mingw32ce does not support errno
On Thu, 9 Oct 2008, Ralf Wildenhues wrote:
Hello Vincent,
* Vincent Torri wrote on Thu, Oct 09, 2008 at 07:02:56AM CEST:
here is a patch that removes compilation errors with mingw32ce because it
does not support the errno system.
Is __MINGW32CE__ even defined for that system? Because the patch just
looks like it's doing exactly the opposite of what you need (and that
means it would work for you only if __MINGW32CE__ is not defined).
arg, i didn't copy/paste from my libtool script and used #ifdef instead of
#ifndef. And yes, that macro exists when that compiler is used.
I don't know if it is the best solution, but it is working for me. So if
you have a better solution, feel free to modify that patch
You've gone this far now, how about you try to redo the patch based on
the comment above, then try to write a ChangeLog entry for it, too, and
then we look at it again? I'm sure you can do it!
done, but seriously, you can do it too...
Vincent Torridiff --git a/ChangeLog b/ChangeLog
index cabc93d..a41264f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2008-10-09 Ralf Wildenhues [EMAIL PROTECTED]
+
+ Do not use errno system in func_emit_cwrapperexe_src()
+ with the mingw32ce compiler
+ * libltdl/config/ltmain.m4sh (func_emit_cwrapperexe_src): Do not
+ include errno.h and remove calls related to errno when using the
+ mingw32ce compiler.
+
2008-10-05 Ralf Wildenhues [EMAIL PROTECTED]
Atomic shared library install permissions on HP-UX.
diff --git a/libltdl/config/ltmain.m4sh b/libltdl/config/ltmain.m4sh
index 696b6ab..a09c5be 100644
--- a/libltdl/config/ltmain.m4sh
+++ b/libltdl/config/ltmain.m4sh
@@ -2780,7 +2780,9 @@ int setenv (const char *, const char *, int);
#include assert.h
#include string.h
#include ctype.h
-#include errno.h
+#ifndef __MINGW32CE__
+# include errno.h
+#endif
#include fcntl.h
#include sys/stat.h
@@ -3171,7 +3173,11 @@ EOF
if (rval == -1)
{
/* failed to start process */
+#ifndef __MINGW32CE__
LTWRAPPER_DEBUGPRINTF (((main) failed to launch target \%s\: errno =
%d\n, lt_argv_zero, errno));
+#else
+ LTWRAPPER_DEBUGPRINTF (((main) failed to launch target \%s\\n,
lt_argv_zero));
+#endif
return 127;
}
return rval;
@@ -3400,8 +3406,12 @@ chase_symlinks (const char *pathspec)
}
else
{
+#ifndef __MINGW32CE__
char *errstr = strerror (errno);
lt_fatal (Error accessing file %s (%s), tmp_pathspec, errstr);
+#else
+ lt_fatal (Error accessing file %s, tmp_pathspec);
+#endif
}
}
XFREE (tmp_pathspec);
libtool wrapper: mingw32ce does not support errno
hey,
here is a patch that removes compilation errors with mingw32ce because it
does not support the errno system.
I don't know if it is the best solution, but it is working for me. So if
you have a better solution, feel free to modify that patch
thank you
Vincent Torridiff --git a/libltdl/config/ltmain.m4sh b/libltdl/config/ltmain.m4sh
index 696b6ab..6bf0c5b 100644
--- a/libltdl/config/ltmain.m4sh
+++ b/libltdl/config/ltmain.m4sh
@@ -2780,7 +2780,9 @@ int setenv (const char *, const char *, int);
#include assert.h
#include string.h
#include ctype.h
-#include errno.h
+#ifdef __MINGW32CE__
+# include errno.h
+#endif
#include fcntl.h
#include sys/stat.h
@@ -3171,7 +3173,11 @@ EOF
if (rval == -1)
{
/* failed to start process */
+#ifdef __MINGW32CE__
LTWRAPPER_DEBUGPRINTF (((main) failed to launch target \%s\: errno =
%d\n, lt_argv_zero, errno));
+#else
+ LTWRAPPER_DEBUGPRINTF (((main) failed to launch target \%s\\n,
lt_argv_zero));
+#endif
return 127;
}
return rval;
@@ -3400,8 +3406,12 @@ chase_symlinks (const char *pathspec)
}
else
{
+#ifdef __MINGW32CE__
char *errstr = strerror (errno);
lt_fatal (Error accessing file %s (%s), tmp_pathspec, errstr);
+#else
+ lt_fatal (Error accessing file %s, tmp_pathspec);
+#endif
}
}
XFREE (tmp_pathspec);
Re: libtool wrapper: mingw32ce does not support errno
Hello Vincent, * Vincent Torri wrote on Thu, Oct 09, 2008 at 07:02:56AM CEST: here is a patch that removes compilation errors with mingw32ce because it does not support the errno system. Is __MINGW32CE__ even defined for that system? Because the patch just looks like it's doing exactly the opposite of what you need (and that means it would work for you only if __MINGW32CE__ is not defined). I don't know if it is the best solution, but it is working for me. So if you have a better solution, feel free to modify that patch You've gone this far now, how about you try to redo the patch based on the comment above, then try to write a ChangeLog entry for it, too, and then we look at it again? I'm sure you can do it! Thanks, Ralf --- a/libltdl/config/ltmain.m4sh +++ b/libltdl/config/ltmain.m4sh @@ -2780,7 +2780,9 @@ int setenv (const char *, const char *, int); #include assert.h #include string.h #include ctype.h -#include errno.h +#ifdef __MINGW32CE__ +# include errno.h +#endif [...]
