maint: improve module indicators

* m4/gnulib-common.m4 (gl_MODULE_INDICATOR_SET_VARIABLE)
(gl_MODULE_INDICATOR, gl_MODULE_INDICATOR_FOR_TESTS): Fit in 80
columns, and avoid extra macro expansion.

Signed-off-by: Eric Blake <eblake@redhat.com>

fdopendir: work around FreeBSD bug

Without a declaration, at least tar would core dump on 64-bit
FreeBSD because gcc only used 32 bits of the resulting pointer.

* m4/dirent_h.m4 (gl_DIRENT_H_DEFAULTS): New witness.
* m4/fdopendir.m4 (gl_FUNC_FDOPENDIR): Set it.
* modules/dirent (Makefile.am): Substitute it.
* lib/dirent.in.h (fdopendir): Supply missing FreeBSD
declaration.
* doc/posix-functions/fdopendir.texi (fdopendir): Document the
fix.
Reported by Christian Weisgerber <naddy@mips.inka.de>.

Signed-off-by: Eric Blake <eblake@redhat.com>

Emit #pragma system_header after the inclusion guard, not before.

Fix indentation.

save-cwd: don't leak a file descriptor when the caller execs

* lib/save-cwd.c (save_cwd): set the close-on-exec flag for the
saved file descriptor.
* modules/save-cwd (Depends-on): Depend on cloexec.

Remove vestiges of fts-lgpl module.

Fix definition of tests witness macro.

Fix definition of tests witness macro.

Fix ioctl's protoype on glibc systems.

Normalize copyright line formatting.

exclude: fix the case of globs vs. EXCLUDE_INCLUDE

* lib/exclude.c (excluded_file_pattern_p): Fix logic error that
made it so grep -r --include=GLOB* ... did not work.

maint.mk: prohibit use of test's -o and -a operators

* top/maint.mk (sc_prohibit_test_minus_ao): New rule.

Remove unused GNULIB_XYZ macro definitions.

Mark privileged tests modules.

Split C++ tests into separate tests modules.

gnulib-tool: Allow 'foo-tests' module even if there is no module 'foo'.

gnulib-tool: Add support for special categories of tests.

ansi-c++-opt: Allow turning off the C++ build by default.

unistd: Avoid #define replacements in C++ mode.

Fix link errors on mingw.

lib-ignore: Determine different options for different compilers.

Remove unused GNULIB_XYZ macro definitions.

Add a remark about certain modules.

Resolve conflict between the two kinds of module indicators.

autoupdate

Distinguish two kinds of module indicators.

New module description field 'Comment'.

Addendum to 2010-02-07 commit.

Use GNULIB_POSIXCHECK instead of GNULIB_PORTCHECK.

Avoid side effects from tests-related modules on the compilation of lib.

Factorize common .m4 code.

Fix a compilation error on Cygwin with g++ >= 4.3.

Rename gl_GETLOGIN.

Rename gl_GETLOGIN_R.

Rename gl_INET_NTOP.

Rename gl_INET_PTON.

Rename gl_MBSLEN.

Rename gl_FORKPTY and gl_OPENPTY.

top/maint.mk (sc_texinfo_acronym): Don't infloop if there is no *.texi files.

Reported by Eric Blake <eblake@redhat.com>.

autoupdate

autoupdate

maint: use pragma consistently across replacement headers

Several headers set #pragma GCC system_header prior to checking
any multiple inclusion guards; maintenance is easiest if all
replacement headers follow the same paradigm.

* lib/ctype.in.h (system_header): Hoist for consistent placement.
* lib/dirent.in.h (system_header): Likewise.
* lib/errno.in.h (system_header): Likewise.
* lib/float.in.h (system_header): Likewise.
* lib/getopt.in.h (system_header): Likewise.
* lib/iconv.in.h (system_header): Likewise.
* lib/inttypes.in.h (system_header): Likewise.
* lib/langinfo.in.h (system_header): Likewise.
* lib/locale.in.h (system_header): Likewise.
* lib/math.in.h (system_header): Likewise.
* lib/netdb.in.h (system_header): Likewise.
* lib/netinet_in.in.h (system_header): Likewise.
* lib/pty.in.h (system_header): Likewise.
* lib/sched.in.h (system_header): Likewise.
* lib/se-selinux.in.h (system_header): Likewise.
* lib/search.in.h (system_header): Likewise.
* lib/spawn.in.h (system_header): Likewise.
* lib/stdarg.in.h (system_header): Likewise.
* lib/stdint.in.h (system_header): Likewise.
* lib/string.in.h (system_header): Likewise.
* lib/strings.in.h (system_header): Likewise.
* lib/sys_file.in.h (system_header): Likewise.
* lib/sys_ioctl.in.h (system_header): Likewise.
* lib/sys_socket.in.h (system_header): Likewise.
* lib/sys_times.in.h (system_header): Likewise.
* lib/sys_utsname.in.h (system_header): Likewise.
* lib/sys_wait.in.h (system_header): Likewise.
* lib/sysexits.in.h (system_header): Likewise.
* lib/unistd.in.h (system_header): Likewise.
* lib/wctype.in.h (system_header): Likewise.

Signed-off-by: Eric Blake <eblake@redhat.com>

arpa/inet: fix mingw compilation warning

Mingw doesn't have <arpa/inet.h>, so a conditional use of the
pragma was never encountered, and led to mingw's gcc issuing
warnings about duplicate declarations under -Wredundant-decls.
Making the #pragma unconditional (as was already done in at
least fcntl.in.h) shuts up gcc.

* lib/arpa_inet.in.h (system_header): Hoist to be unconditional.
Reported by Matthew Bolte.

Signed-off-by: Eric Blake <eblake@redhat.com>

Avoid collision between gnulib wrapper and libintl wrapper.

Use ANSI C.

Minor formatting changes.

ChangeLog entry formatting.

autoupdate

maint.mk (sc_Wundef_boolean): Skip test if no config.h.

Signed-off-by: Simon Josefsson <simon@josefsson.org>

pt_chown: Make it work with automake < 1.11.

pt_chown: Don't depend on GPLed modules.

top/maint.mk (sc_texinfo_acronym): Add rule.

Based on suggestions from karl@freefriends.org (Karl Berry).

untabify

gethostname: further tweaks

* lib/unistd.in.h (includes): Only worry about <winsock2.h> if we
are overriding gethostname.
Suggested by Bruno Haible.

Signed-off-by: Eric Blake <eblake@redhat.com>

Fix comments.

gethostname: fix build on mingw

Since commit 4e86671a, gethostname failed to compile on mingw.
Gnulib's sys/socket.h includes winsock2.h, which then includes
unistd.h prior to declaring gethostname.  The fix is to ensure
that our replacement unistd.h does not declare any replacements
until we are sure that winsock2.h is completely included.

* lib/unistd.in.h (includes): Work around fact that mingw
<winsock2.h> re-includes <unistd.h>, by avoiding any
redeclarations if we are being included by <winsock2.h>.
Reported by Matthias Bolte.

Signed-off-by: Eric Blake <eblake@redhat.com>

forkpty: Provide replacement on AIX, HP-UX, IRIX, Solaris.

Use the common template. Make test stricter.

Use the common module description template.

Tests for module 'login_tty'.

New module 'login_tty'.

login_tty: Documentation.

pty: Consistent macro naming.

Tests for openpty: Make stricter.

openpty: Provide replacement on AIX, HP-UX, IRIX, Solaris.

setenv: Tweaks.

New module 'unlockpt'.

maint.mk: prohibit inclusion of "intprops.h" without use

* top/maint.mk (sc_prohibit_intprops_without_use): New rule.

New module 'grantpt'.

New module 'pt_chown'.

Tests for module 'ptsname'.

New module 'ptsname'.

Tests for module 'ttyname_r'.

New module 'ttyname_r'.

signal: Undefine macro definitions in C++ mode.

Ensure no #include statements inside extern "C" { ... }.

Make _GL_WARN_ON_USE usable in C++ and C mode in the same compilation unit.

Fix last commit.

pipe: Set errno upon failure.

lchown: Avoid "unused variable" warning.

Work around unlink() bug on MacOS X 10.5.6.

Fix link errors on Solaris 8.

regcomp.c: make non-_LIBC implementation of build_range_exp consistent

The _LIBC implementation of build_range_exp correctly honors the
RE_NO_EMPTY_RANGES flag when checking for reversed range endpoints.
However, the non-_LIBC implementation would ignore that syntax-bit
flag and return REG_ERANGE unconditionally.
This change makes it honor that flag.
* lib/regcomp.c (build_range_exp) [!_LIBC]: Add a parameter: "syntax".
Make two pointer parameters "const".
Use "syntax" bits in order to honor RE_NO_EMPTY_RANGES.
(parse_bracket_exp): Update caller.

regex.m4: correct the reversed range endpoint ([b-a]) test

* m4/regex.m4: When requiring that [b-a] evoke failure,
use RE_NO_EMPTY_RANGES.  This makes this entire configure-time
test pass once again for x86-based systems.

scandir: Fix link error on Solaris 8.

getusershell: Fix documentation.

getusershell: Provide declaration, missing on Solaris 9.

wctype: Provide iswblank function.

Tests of module 'pty' in C++ mode.

logb: fix documentation

* doc/posix-functions/logb.texi (logb): Gnulib fixes the cygwin
1.5 declaration bug.

Signed-off-by: Eric Blake <eblake@redhat.com>

forkpty, openpty: prefer glibc's const-safe prototype

This silences a compiler warning for test-forkpty.c.

* lib/forkpty.c (rpl_forkpty): New file.
* lib/openpty.c (rpl_openpty): Likewise.
* modules/forkpty (Files): Distribute it.
* modules/openpty (Files): Likewise.
* m4/pty_h.m4 (gl_PTY_H_DEFAULTS): Add new witnesses.  Move decl
check...
* m4/pty.m4 (gl_FORKPTY, gl_OPENPTY): ...here.  Request
replacement for for non-const BSD signature.
* modules/pty (Makefile.am): Substitute witnesses.
* lib/pty.in.h (forkpty, openpty): Declare replacements.
* tests/test-forkpty.c: Update signature check.
* tests/test-openpty.c: Likewise.
* doc/glibc-functions/forkpty.texi (forkpty): Document the fix.
* doc/glibc-functions/openpty.texi (openpty): Likewise.
Reported by Bruno Haible.

Signed-off-by: Eric Blake <eblake@redhat.com>

forkpty, openpty: split functions into new modules

The pty module was doing too much - replacing the header and checking
for functions.  In the future, when we provide a working forkpty for
Solaris and HP-UX, and a stub for mingw, then the separation will be
even more useful.

* modules/pty (Makefile.am): Substitute new witnesses.
(Libraries): Move library detection...
* modules/forkpty: ...into new module.
* modules/openpty: Another new module.
* modules/pty-tests: Rename and split...
* modules/forkpty-tests: ...to this...
* modules/openpty-tests: ...and this.
* tests/test-pty.c: Rename and split...
* tests/test-forkpty.c: ...to this...
* tests/test-openpty.c: ...and this.
* m4/pty_h.m4 (gl_PTY_H_DEFAULTS): Add new witnesses.
(gl_PTY): Split library searching...
* m4/pty.m4 (gl_PTY_LIB): ...into new file.
(gl_FORKPTY, gl_OPENPTY): New macros.
* lib/pty.in.h (forkpty, openpty): Honor new witnesses.
* NEWS: Mention the split.
* MODULES.html.sh (Misc): Document the modules.
* doc/glibc-functions/forkpty.texi (forkpty): Likewise.
* doc/glibc-functions/openpty.texi (openpty): Likewise.

Signed-off-by: Eric Blake <eblake@redhat.com>

pty: improve replacement header

Build pty.h in the same way as all other replacement headers.

* lib/pty.in.h: New file.
* modules/pty (Files): Ship it.
(Makefile.am): Always build replacement.
* m4/pty.m4: Rename...
* m4/pty_h.m4: ...to this.
(gl_PTY): Modernize setting of witness macros; update check of
forkpty to take proper advantage of cache.
(gl_PTY_MODULE_INDICATOR, gl_PTY_H_DEFAULTS): New macros.

Signed-off-by: Eric Blake <eblake@redhat.com>

getopt: avoid compiler warning

* lib/getopt.c (attribute_hidden): Remove unused macro.

Signed-off-by: Eric Blake <eblake@redhat.com>

Fix link errors on Solaris 8.

Fix bug introduced on 2010-03-14.

Fix typo introduced on 2009-12-31.

test-vc-list-files-git: make more robust

Unset GIT_DIR and GIT_WORK_TREE environment variables to avoid
testing the wrong location.

* tests/test-vc-list-files-git.sh: Unset problematic environment
variables.  Chain commands together.

Signed-off-by: Eric Blake <eblake@redhat.com>

pty: Make forkpty check work.

Portability fix in lib/inttostr.c.