From: Brandon Casey <drafnel@xxxxxxxxx>
An entry in the config.mak.in file is necessary for the NEEDS_LIBGEN variable
to appear in the config.mak.autogen file with the value assigned by the
configure script.
Signed-off-by: Brandon Casey <drafnel@xxxxxxxxx>
---
Junio,
You probably want to apply or squash this somewhere.
As I noted in the original patch, the autoconf part was untested.
I actually did some light testing on this one. I created the configure file
on linux and ran it on Solaris and IRIX. Both produce an error which looks
something like:
configure[1234]: syntax error at line 4806 : `;' unexpected
And line 4806 looks like:
for ac_lib in ; do
...
It works with bash though, and it works with /bin/sh on Solaris 10. On
Solaris 10, the configure script correctly detects that hstrerror cannot be
used without -lresolv, and basename can be used without -lgen. In
config.mak.autogen, NEEDS_RESOLV is set to 'YesPlease' and NEEDS_LIBGEN is
unset. On my Solaris 7, bash is not available, but the informational messages
indicate the same results as for Solaris 10. On IRIX, hstrerror() can be used
without -lresolv and basename cannot be used without -lgen. In
config.mak.autogen, NEEDS_RESOLV is unset, and NEEDS_LIBGEN is set to
'YesPlease'.
So, I think this patch should be the final one.
-brandon
config.mak.in | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/config.mak.in b/config.mak.in
index dd60451..67b12f7 100644
--- a/config.mak.in
+++ b/config.mak.in
@@ -34,6 +34,7 @@ NO_LIBGEN_H=@NO_LIBGEN_H@
NEEDS_LIBICONV=@NEEDS_LIBICONV@
NEEDS_SOCKET=@NEEDS_SOCKET@
NEEDS_RESOLV=@NEEDS_RESOLV@
+NEEDS_LIBGEN=@NEEDS_LIBGEN@
NO_SYS_SELECT_H=@NO_SYS_SELECT_H@
NO_D_INO_IN_DIRENT=@NO_D_INO_IN_DIRENT@
NO_D_TYPE_IN_DIRENT=@NO_D_TYPE_IN_DIRENT@
--
1.6.3.1.24.g152f4
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html