build: keep --version strictly up to date

Before this change, in development, gzip's --version output could lag
behind reality by a couple deltas or by a "-dirty" suffix.  That would
lead to spurious failure of the new --version-$VERSION PATH cross-check.
* Makefile.am (version.c, version.h): New rules.
(BUILT_SOURCES): Set/append.
(noinst_LIBRARIES, noinst_libver_a_SOURCES): Define.
(gzip_LDADD): Add libver.a.
(DISTCLEANFILES): Define.
* gzip.c (license): Use Version, not VERSION.

tests: (portability) use st, not "status" as variable name

* tests/zgrep-signal: Do not use status as a variable name,
per autoconf's documentation that it is not portable to
some shells.

tests: s/framework_failure/framework_failure_/

tests: update init.sh from gnulib

* tests/init.sh: Update from gnulib.

tests: run most tests via tests/Makefile.am

* Makefile.am (SUBDIRS): List tests after ".".
Move most test-related things from here to ...
* tests/Makefile.am: ... here.
* configure.ac (AC_CONFIG_FILES): Add tests/Makefile.
* tests/helin-segv: Adjust.
* tests/help-version: Likewise.
* tests/hufts: Likewise.
* tests/memcpy-abuse: Likewise.
* tests/mixed: Likewise.
* tests/null-suffix-clobber: Likewise.
* tests/stdin: Likewise.
* tests/trailing-nul: Likewise.
* tests/zdiff: Likewise.
* tests/zgrep-f: Likewise.
* tests/zgrep-signal: Likewise.
* tests/znew-k: Likewise.

tests: skip tests that use grep's -f and -E options, if they don't work

* tests/init.cfg (require_grep_minus_f): New function.
* tests/zgrep-f: Use require_grep_minus_f.  Use path_prepend_.

tests: arrange for skip and failure notices to go to stderr, not .log

* tests/init.cfg: New file.  Make init.sh's stderr_fileno_ match
what the "exec 9>&2" we use in TESTS_ENVIRONMENT.
* Makefile.am (EXTRA_DIST): Add it.

maint: ignore generated files

* .gitignore: Ignore recent gnulib additions.

maint: update bootstrap

* bootstrap: Use latest copy from gnulib/build-aux.

build: use gnulib's lib-ignore module

* bootstrap.conf (gnulib_modules): Add lib-ignore, in case it helps.
* Makefile.am (AM_LDFLAGS): Define it.

maint: let configure-invoked cpp emit diagnostics to config.log

* configure.ac: Do not discard CPP's stderr.

build: update gnulib submodule to latest, and adapt

* cfg.mk: Update to use new _sc_search_regexp interface.  Run this:
perl -pi -e 's/\b_prohibit_regexp\b/_sc_search_regexp/;'
-e 's/\bmsg=/halt=/; s/\bre=/prohibit=/;' cfg.mk
and then adjust backslashes so they still line up.
* cfg.mk (local-checks-to-skip): Add new sc_texinfo_acronym, to skip it.
* msdos/tailor.c (fcalloc): Mark a diagnostic for translation, to
placate stricter syntax-check, even though no one uses this file.

use assembly code matcher when possible

* configure.ac (ASCPPPOST): Backslash-escape "#" in AC_SUBST'd
variable, to keep make from seeing it as a comment-introducer.
Based on a patch by Petr Pisar.
* lib/Makefile.am (match.$(OBJEXT)): Use AM_V_GEN and AM_V_at.
* lib/match.c: Don't include <config.h>.
It would impede configure-time assembler test.
* .x-sc_require_config_h: Exempt lib/match.c from syntax-check.
* .x-sc_require_config_h_first: Likewise.

do not use stat.st_mtime of a non-regular file

* gzip.c: Include "timespec.h".
(treat_stdin): Use st_mtime only from a regular file.
This matters at least on Cygwin 1.7.1-1, for which a stdin-pipe has
the mtime of /dev/null, rather than the gzip-documented-for-pipes
"current time".  Reported by Denis Excoffier.

tests: exercise the fix for the decompression data-loss bug

* tests/null-suffix-clobber: New file.
* Makefile.am (TESTS): Add it.

gzip: fix a data-loss bug when decompressing with --suffix=''

* gzip.c (main): Disallow an empty --suffix=S also with -d.
Otherwise, "gzip -d -S '' F.gz" would ask if it's ok to remove the
existing file, "F.gz";  if you reply "yes", you'd lose all of that data.
Use of an empty suffix was already rejected in compression mode.
* gzip.1 (--suffix (-S)): Do not recommend to use "gunzip -S '' *".
Describe how the suffix is used when decompressing, too.
* NEWS (Bug fixes): mention the fix.
Reported by Ripduman Sohan.

tests: add ---presume-input-tty option, solely for testing

* gzip.c: Include <stdbool.h>.
(presume_input_tty): New global.
(main): Set it.
(treat_stdin, check_ofname): Use it.

doc: minor adjustment to README-release

* README-release: Tweak description, to sync from coreutils.

tests: add the help-version sanity tests from coreutils

* tests/help-version: New file, from coreutils.
* Makefile.am (TESTS): Add it.

tests: make distcheck invoke "make syntax-check" and other tests

* dist-check.mk: New file, from coreutils.
* cfg.mk: Include it.
* Makefile.am (distcheck-hook): New rule, to make us use it.

zcmp: consistently indicate failure with exit status of 2

* zcmp.in: Exit with status of 2 (not 1), when writing
--help or --version output fails, to be more like cmp.

tests: add more tests of gzip -cdf

* tests/mixed: Test "gzip -cdf" for a range of small uncompressed files.

tests: flip and adjust mixed test, now that the bug is fixed

* NEWS (Bug fixes): Mention the fix.
* Makefile.am (XFAIL_TESTS): Move tests/mixed from here...
(TESTS): ...to here.
* tests/mixed: Comment out the currently (always?) failing part.

gzip -cdf now handles concatenation of gzip'd and uncompressed data

* util.c (copy): Change semantics so as to honor a decremented inptr.
* gzip.c (get_method): When needed (-cdf), decrement inptr rather
than clearing it -- and output the first magic byte.

zgrep: terminate gracefully when a pipeline is interrupted by a signal

zgrep is not terminated gracefully when its grep/sed pipeline
is terminated by a signal.  For example, a command like
zgrep -F .TH /usr/share/man/man1/*.gz | head
continues working long after the "head" process completes.
Another example, a command like
zgrep unmatched-pattern /usr/share/man/man1/*.gz
cannot be interrupted by sending a SIGQUIT with Ctrl-\ key, it outputs
zgrep: line 221: test: : integer expression expected
and goes on.
* zgrep.in: Terminate gracefully when the grep/sed pipeline is
terminated by a signal.
* tests/zgrep-signal: New test.
* Makefile.am (TESTS): Add it.

maint: teach "make syntax-check" the space-only indentation rule

* cfg.mk (sc_prohibit_tab_based_indentation): New rule, from coreutils.
(sc_prohibit_emacs__indent_tabs_mode__setting): Likewise.
* bootstrap: Remove "indent-tabs-mode: nil" directive.
* .x-sc_prohibit_tab_based_indentation: New file.

global: convert indentation-TABs to spaces

Transformed via this shell code:
t=$'\t'
git ls-files \
  | grep -vE '(^|/)((GNU)?[Mm]akefile|ChangeLog)|\.(am|mk)$' \
  | grep -vE 'tests/pr/|help2man' \
  | xargs grep -lE "^ *$t" \
  | xargs perl -MText::Tabs -ni -le \
    '$m=/^( *\t[ \t]*)(.*)/; print $m ? expand($1) . $2 : $_'

wrapper scripts: write diagnostics to stderr, not to stdout

* zforce.in: In case of usage error, output short error diagnostics to
stderr instead of printing help text to stdout.
* zmore.in: Likewise.
* znew.in: Likewise.

gzip -cdf mishandles some concatenated input streams: test it

* tests/mixed: Exercise "gzip -cdf" bug.
* Makefile.am (XFAIL_TESTS): Add it.
Mark Adler reported the bug.

tests: move the hufts-segv test to its own file

* tests/hufts: New test.
* Makefile.am (TESTS): Add tests/
(check-local): Remove the hufts-segv test from this rule.

tests: begin moving tests into their own files

* Makefile.am (TESTS): Add tests/stdin.
(check-local): Move the stdin check to its own file:
* tests/stdin: New script.

tests: remove unnecessary use of "path_prepend_ ."

It is unnecessary, since Makefile.am's TESTS_ENVIRONMENT setting
already adds the top build directory, and besides, is only
marginally useful when running stand-alone, since it presumes
that the script is being run from the top build directory.
* tests/helin-segv: Remove unnecessary use of path_prepend_.
* tests/memcpy-abuse: Likewise.
* tests/trailing-nul: Likewise.
* tests/zdiff: Likewise.
* tests/zgrep-f: Likewise.

tests: note that znew-k test depends on length of temp file name

* tests/znew-k: Tweak diagnostic, factor and add a surprising comment.

build: update gnulib submodule to latest

fix "znew -K" to work without use of compress utility

* znew.in: Change -K option to imply -t, do not use compress(1).
* znew.1: Document it.
* tests/znew-k: New test.
* Makefile.am (TESTS): Add it.

maint: update README-release procedure

* README-release: sync from coreutils.

post-release administrivia

* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.

version 1.4

* NEWS: Record release date.

tests: exercise the segfault fix

* tests/helin-segv: New test.
* Makefile.am (TESTS): Add it.

gzip -d: do not clobber stack for valid input on x86_64

* unlzw.c (unlzw): Avoid integer overflow.
Aki Helin reported the segfault along with an input to trigger the bug.
* NEWS (Bug fixes): Mention it.

maint: avoid a minor "make syntax-check" failure

* .x-sc_file_system: Exempt NEWS, since it includes a literal
from an old diagnostic.

maint: remove unused file: README-alpha

* README-alpha: Remove unused file.

build: update gnulib submodule to latest

gzip -d: use memcpy more often

* inflate.c (inflate_codes): Use memcpy (rather than slower
memcopy-like code) in more cases.

build: do not override gnulib-provided AM_CFLAGS options

Avoid a warning from automake:
lib/Makefile.am:24: AM_CFLAGS multiply defined in condition TRUE ...
lib/gnulib.mk:30: ... `AM_CFLAGS' previously defined here
lib/Makefile.am:20:   `lib/gnulib.mk' included from here

* lib/Makefile.am (AM_CFLAGS): Append $(WARN_CFLAGS) and
$(WERROR_CFLAGS), i.e., use "+=", not "=".
This was introduced via 2009-12-17 commit 0341fc22,
"build: with --enable-gcc-warnings, use -Werror",
but fortunately is not a bug, because the definition
it would have overridden was always empty.

gzip -d would fail with a CRC error...

...for some inputs, and some memcpy implementations.  It is possible
that an offending input has to be compressed "from FAT filesystem
(MS-DOS, OS/2, NT)", since the sole reproducer no longer evokes a
CRC error when uncompressed and recompressed on a GNU/Linux system.
Also, using an unpatched reverse-memcpy-gzip on over 100,000 inputs
on a GNU/Linux system did not turn up another reproducer.
* inflate.c (inflate_codes): Don't call memcpy with overlapping regions.
Properly detect when source and destination overlap.
* tests/memcpy-abuse: New test, to trigger misbehavior.
* Makefile.am (TESTS): Add it.
* NEWS (Bug fixes): Mention it.
Reported by Alain Magloire in
http://thread.gmane.org/gmane.comp.gnu.gzip.bugs/307

tests: switch to gnulib's init.sh test infrastructure

* tests/test-lib.sh: Remove file.
* tests/init.sh: New file, from gnulib.
* tests/trailing-nul: Use the new file.
* tests/zdiff: Likewise.
* tests/zgrep-f: Likewise.
* Makefile.am (EXTRA_DIST): List tests/init.sh, not test-lib.sh.

build: update gnulib submodule to latest

maint: record update-copyright options for this package

* cfg.mk: Next time, just run "make update-copyright".

maint: update all FSF copyright year lists to include 2010

Use this command:
git ls-files |grep -vE '^(\..*|COPYING|gnulib)$' |xargs \
env UPDATE_COPYRIGHT_USE_INTERVALS=1 build-aux/update-copyright

maint: newer gnulib; don't hard-code my GPG key ID

* cfg.mk (gpg_key_ID): Remove definition, now that maint.mk automates it.
* gnulib: Update to lastest.

build: update gnulib submodule to latest

maint: remove lots of obsolete #if...HAVE_* checks

Remove many always-true cpp tests like #ifdef HAVE_UNISTD_H,
 #ifdef HAVE_FCNTL_H and #ifdef SSIZE_MAX.
* gzip.c: As above.
* gzip.h: Likewise.
* inflate.c: Likewise.
* tailor.h: Likewise.
* unlzw.c: Likewise.
* util.c: Likewise.
* zip.c: Likewise.

build: add a syntax-check

* cfg.mk (sc_prohibit_obsolete_HAVE_HEADER_H): New rule.

build: with --enable-gcc-warnings, use -Werror

* Makefile.am (AM_CFLAGS): Enable $(WERROR_CFLAGS).
* lib/Makefile.am (AM_CFLAGS): Enable both $(WARN_CFLAGS) and
$(WERROR_CFLAGS).

build: quiet warnings from util.c

* configure.ac (warnings): Add -Wno-overflow and -Wno-type-limits.

build: avoid warning about possibly-no-return functions

* gzip.h (read_error, write_error): Mark these functions as "no-return".

build: avoid warning about ignored chown/fchown return value

* bootstrap.conf (gnulib_modules): Add ignore-value.
* gzip.c: Include "ignore-value.h".
(copy_stat): Explicitly ignore chown and fchown failure

build: update gnulib submodule to latest

build: unlzw.c: avoid warnings about unused macros

* configure.ac: Turn off -Wunused-macros.

build: avoid warnings about unused macros

* unzip.c (LOCTIM): Comment out unused macro.
* deflate.c (EQUAL): Remove definition.  Use "0" at sole point of use.

build: util.c: avoid warnings about add_envopt

* util.c (add_envopt): The parameter "env" was used for two conflicting
purposes.  One use required a const char* parameter, while the other
was used as an argument to free, which must not be "const".
Rename the parameter and use a new local variable for the second role.

build: avoid many const-related warnings

* gzip.c: Add "const" to many variables, to avoid compiler warnings.
* util.c (add_envopt): Make 3rd parameter const
(gzip_error): Make sole parameter const.
* gzip.h: Update prototypes.

build: avoid warnings from -Wstrict-prototypes

* gzip.c (main): Declare using a prototype.
(progerror): Likewise.  And make parameter const.

build: use gnulib's fdopendir module

* bootstrap.conf (gnulib_modules): Add fdopendir.
* gzip.c (treat_dir): Don't depend on HAVE_FDOPENDIR.
Gnulib now guarantees its availability.
* configure.ac: Don't check for fdopendir here.

build: enable many warnings

* configure.ac: Add support for --enable-gcc-warnings.
* bootstrap.conf (gnulib_modules): Add manywarnings.
* Makefile.am (AM_CFLAGS): Add (WARN_CFLAGS) # $(WERROR_CFLAGS)

maint: tweak formatting of bootstrap.conf

* bootstrap.conf (gnulib_modules): Unindent list.

maint: cfg.mk: remove factored-out ftp host/dir definitions

* cfg.mk (gnu_ftp_host-alpha, gnu_ftp_host-beta gnu_ftp_host-stable):
(gnu_rel_host, url_dir_list): Remove definitions.  The defaults,
now provided by maint.mk, are the same.
* gnulib: Update for latest, including those maint.mk additions.

build: "make stable" emitted an invalid gnupload command

* cfg.mk (gnu_ftp_host-stable): Rename from gnu_ftp_host-major.
* README-release: Change another s/major/stable/.

post-release administrivia

* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.

version 1.3.14

* NEWS: Record release date.

build: accommodate new syntax-check test

* amiga/tailor.c (_expand_args): Change each of three uses of
"exit(20)" to "exit(EXIT_FAILURE)".
* sample/add.c: Include <stdlib.h>.
(main): Use EXIT_FAILURE and EXIT_SUCCESS, not 1 and 0.
* sample/sub.c (main): Likewise.
* sample/zread.c (main): Likewise.

build: update gnulib submodule to latest

build: link with $(LIB_CLOCK_GETTIME)

* Makefile.am (gzip_LDADD): Add $(LIB_CLOCK_GETTIME), now that
utimens pulls in gettime.
(FILES_TO_CHECK): Remove $(gzip_LDADD), now that it may contain -lrt.

msdos: fix syntax in Makefile.djg

* msdos/Makefile.djg (gzip): Change each of two leading 8-space
sequences to a TAB.  Reported by Allan Mui.

build: do not use AC_PREFIX_PROGRAM

* configure.ac: Do not use "AC_PREFIX_PROGRAM(gzip)".
It would induce behavior that is so surprising that it is probably
universally considered to be buggy, these days.  Reported by Zube
in http://thread.gmane.org/gmane.comp.gnu.gzip.bugs/273

gzip: add tests for today's bug fix

* tests/trailing-nul: New file.  Test for today's fix.
* Makefile.am (TESTS): Add new script.
* NEWS (Bug fixes): Mention it.

gzip: don't fail when there is exactly one trailing NUL byte

* gzip.c (get_method): Require the second byte of magic only if
the first byte was nonzero.

build: enable automake color- and parallel-test options

* configure.ac (AM_INIT_AUTOMAKE): Enable color-tests and parallel-tests.

zgrep: portability improvements; exercise "-f -"

* zgrep.in: Adjust loop not to use seq; it's not portable enough.
Fail if we don't find a free file descriptor.
(exists): New function; Use it in place of less portable "test -e".
Testing for existence of /dev/fd/$fd doesn't work on Solaris 10,
since all 256 always exist (as char devices), but testing for
/proc/$$/fd/$fd does work, so do that instead.
* Makefile.am (TESTS): Add tests/zgrep-f.
* tests/zgrep-f: New test; exercise this bug.
* NEWS (Bug fixes): Mention it.

zgrep: handle "-f -" the same way that it works with grep

Before this change, echo needle|zgrep -f - haystack.gz would not work.
* zgrep.in: When the pattern comes from stdin, redirect it to a
different file descriptor, since we're about to use stdin.

zdiff would exit 2 (error) rather than 1 for differences

* zdiff.in: Save and restore cmp's exit status around a case
statement that would otherwise clobber its value.
* NEWS (Bug fixes): Mention it.
* tests/zdiff: New test; exercise both bugs.
* tests/test-lib.sh: New file.  From coreutils.
* Makefile.am (EXTRA_DIST): Add tests/test-lib.sh.
(TESTS): Add tests/zdiff.
(TESTS_ENVIRONMENT): Define.  From coreutils.
(EXTRA_DIST): Append all $(TESTS).

zdiff: fix malfunction when operating on two gzip'd inputs

zdiff would fail to print differences in two compressed inputs
* zdiff.in: Don't let cmp output mix with echo'd gzip exit
status values.  Report and fix from Jörg-Volker Peetz via
<http://bugs.debian.org/434275>
* NEWS (Bug fixes): Mention it.

build: update gnulib submodule to latest

build: update gnulib submodule to latest

maint: clarify web-doc-update instructions

* README-release: sync a tiny change from coreutils.

build: update gnulib submodule to latest

doc: use gnu-web-doc-update module

* bootstrap.conf (gnulib_modules): Add it.

doc: describe release procedure

* README-release: New file.

maint: make cfg.mk slightly more generic

* cfg.mk (url_dir_list): Don't hard-code "coreutils".  Use $(PACKAGE).

post-release administrivia

* NEWS: Add header line for next release.
* .prev-version: Record previous version.
* cfg.mk (old_NEWS_hash): Auto-update.

version 1.3.13

* NEWS: Record release date.

build: update gnulib submodule to latest

maint: update bootstrap script

* bootstrap (with_gettext): Update from coreutils.

maint: pull in two release-related modules from gnulib

* bootstrap.conf (gnulib_modules): Add announce-gen and gnupload.

build: use more gnulib modules for better POSIX compliance

* bootstrap.conf (gnulib_modules): Add modules exposed via
make CFLAGS=-DGNULIB_POSIXCHECK 2>&1 \
|perl -lne '/.* use gnulib module (\S+).*/ and print $1' \
|sort |uniq -c|sort -nr
Add these: calloc close fclose fprintf-posix lstat malloc
perror printf-posix realloc

doc: update to FDL 1.3, minor clean-up

* NEWS, README, TODO: Update FDL s/1.2/1.3/.

doc: update gzip.texi

* doc/gzip.texi: FDL 1.3+
Consistently (de)capitalize section names.
"User's" is not correct; seems best to simplify the title.
Throw in some "GNU"'s.
Make direntries be what we actually want.
Detabify.

maint: temporarily exempt failing syntax-check tests

* cfg.mk (local-checks-to-skip): Add the list of currently-failing
tests, to exempt them from "make syntax-check".

gzip: don't use an uninitialized file descriptor

gzip interprets an argument of "-" as indicating stdin, but when
"-" is not the first name on the command line, it doesn't work.
* gzip.c (treat_stdin): Initialize "ifd".
Patch and fine description by Lasse Collin in
http://www.mail-archive.com/bug-gzip@gnu.org/msg00213.html
* Makefile.am (check-local): Exercise the fix.  Based on the above.
* NEWS (Bug fixes): Mention this.