C++-ify format_command and print_command and move to where used

C++-ify logging routines

Use File wrapper for static log file object

This flushes and closes the underlying FILE object properly on exit.

Create test directories in testdir and remove it on “make clean”

Probe assembler for supported -m* flags

Since CMake uses the assembler executable for compiling assembler source
code it’s not quite correct to probe the C++ compiler (nor the C
compiler) since it may not be the same as the assembler.

Fixes #625.

Improve instructions on how to install libzstd

Update Travis-CI badge URL to travis-ci.com

Remove now unused get_extension function

C++-ify win32getshell

Make Util::to_lowercase work on nonstd::string_view

Only dump log buffer in cc_dump_debug_log_buffer if it exists

This can happen in unit tests.

Make is_precompiled_header handle the empty string

Related to PR #624.

Remove dependency on std::regex

std::regex is not available for GCC 4.8. It’s also a bit bloated. The
reason for not using POSIX regex functionality is that it’s not
available in MinGW.

Improve -x/--show-compression output

Handle EINTR correctly in copy_fd

Remove more short read optimization from #551

In addition to the removal done in 4b71108612f7.

C++-ify struct hash

Simplify code somewhat

Add ability to ignore specified compiler options when hashing (#620)

Closes #578.

Make “misc/format-files --check” exit with error code on error

Regression in ee332b44a098f8603c05a25e09168b6b5b9c96ae.

(cherry picked from commit ea5379c6e8a39605d71b52d01c4244ea8aeeae48)

Switch to doctest from Catch2 for tests

doctest is more light-weight than Catch2 and compiles faster.

Remove short read optimization from #551

I don’t feel confident about assuming that a short read for a file means
EOF in all situations. For instance, it looks like short reads can
happen of an NFS file system mounted with the “intr” option, and it can
also happen if a file entry actually is a named pipe.

Improve .clang-tidy HeaderFilterRegex patterns

The pattern for unittest now excludes third_party headers as well.

Make Result::write_embedded_file_entry static

Remove unneeded include in unittest/TestUtil.hpp

Simplify ResultRetriever somewhat

No need for a separate m_dest_stderr flag after adding m_dest_file_type.

Add “--extract-result” option

“ccache --extract-result” extracts data stored in a result file to the
current working directory.

Refactor result code into a Result::Reader/Writer classes

Introduced a Result::Reader class which remove knowledge about what to
do with the individual file parts of a result. That’s instead defined by
a Result::Reader::Consumer instance passed to Result::Reader::read. This
means a ResultFileMap is no longer used and that the consumer is free to
write data whereever it wants.

The new ResultRetriever class implements retrieval by writing to local
files like before, except that the stderr data is written directly to
STDERR_FILENO instead of landing in a temporary file.

The new ResultDumper class implements “ccache --dump-result”.

Also introduced a Result::Writer class, mostly for symmetry with the
Result::Reader class.

Extract send_cached_stderr into Util::send_to_stderr

Extract copy_raw_file into Util::clone_hard_link_or_copy_file

Fix Clang-Tidy inconsistent-declaration-parameter-name warning

Document --dump-result

Align command line option list in manual with help text

Tweak ci/build script

- Removed redundant “exit 0” command again.
- Removed now obsolete and misleading comments.
- If JOBS is set to the empty string then detect the number of CPUs instead
  of running a limitless number of jobs.

Use a known and set TERM variable to stabilize color_diagnostics tests

On some distributions TERM is unset when run via Docker which failed
color_diagnostics tests since ccache doesn’t enable color output if TERM
is unset. Fix this by using a known TERM value.

Fixes #616.

Skip “Directory is not hashed if using -gz[=zlib]” tests for GCC 6

The GCC 6 (tested with GCC 6.3) preprocessor includes the current
working directory in the preprocessed output if run with -gz, which
means that there won’t be cache hits between directories, thus failing
the “Directory is not hashed if using -gz[=zlib]” tests. Fix this by
skipping the test if the preprocessor behaves that way.

Fixes #602.

Enable verbose output from “test/run” if VERBOSE is set

Sort environment variables passed to docker container

Pass arguments to ci/build to ctest

This makes it possible to only run a subset of the tests by for instance
running “build-in-docker <name> -R <test>”.

Remove redundant docker image tag suffix

Run compilation and tests with multiple threads (#610)

Fix color_diagnostics test failure if CCACHE_DIR is set in shell rc file

If the user sets CCACHE_DIR in a shell startup file then that value will
override the one from the test suite, thus making changes in cache
statistics invisible which triggers failures.

Make one of the CI jobs build in release mode

It seems reasonable to run tests with the real release binary.

Clean up INSTALL.md

Rename producttest to test

I don’t see a value in the prefix.

Rename generated Version.cpp to version.cpp

The convention is to use capitalized filenames for source files that
implement classes and namespaces and lowercase filenames for source
files that do not.

Clean up scripts related to code formatting

- Simplified scripts.
- Made colored output work again.

Clean up scripts related to CI and Docker

Improve naming of scripts

- Use kebab case consistently.
- Remove redundant (and exposing an implementation detail) .sh extension
  for executables.

Clean up build-in-docker script

Move top-level scripts to misc

This makes the top-level directory less crowded.

Rename USE_LIBZSTD_FROM_INTERNET to ZSTD_FROM_INTERNET

Simplify code for hashing include file

Build InodeCache.cpp only when inode cache is supported

Build win32compat.cpp only on Windows

Enable more clang-tidy rules and implement suggestions

Reduce the set of included headers

Implement some suggestions from clang-tidy 10

Tweak CMakeLists.txt

Remove stale licenses (#613)

The m4 files are no longer used after the switch to cmake in f2341b2.

Tweak buffer reservation in Util::format_hex

Do not report skipped tests as failures on CMake <3.9 (#609)

Reduce number of memory allocations in hashing (#608)

Add hash_buffer_once method for hashing a given text immediately using a
stack allocated hasher. Also skip copying the hasher in hash_result as
blake3 is not modifying the hasher.

Update contribution guidelines related to clang-format usage

Update authors

(cherry picked from commit c7455f6a5385f1c19dc3b6c93cc4b37c0320ec48)

Update NEWS

(cherry picked from commit 6695d6011bc8a91e167da86fb326d54a9d005cde)

Remove win32 GetFileNameFromHandle compatibility function

The replacement function seems to be buggy, triggering failures in
x_realpath at least when the unit tests are run in Wine. The real
GetFinalPathNameByHandle function is allegedly supported in Windows
Vista and newer, so let’s just use it.

(cherry picked from commit bd742fbe20493f720ae6af00525fd3a8ee829802)

Improve comment of inode cache version and revert to version 1 again

As noted by Olle Liljenzin, changing the hash algorithm does not require
incrementing the version field, so let’s revert the change I made as
part of 2a0dd8ef.

Fix numbering of Travis CI jobs

The “allow_failures” one are always listed last, so move them last in
.travis.yml as well.

Let “ccache --hash-file” perform raw hashing again

Unintended or not, #577 (213d9883) changed the behavior of “ccache
--hash-file” to use hash_binary_file, which essentially performs
hash(hash(path)) if the i-node cache is enabled, otherwise hash(path).
This means that “ccache --hash-file” behaves differently depending on if
i-node cache is enabled and also that it’s no longer usable for
benchmarking purposes.

Fix this by simply using “hash_file” again.

Fix bug in hash_binary_file

Introduced in #577 (213d9883).

Fix location of secondary configuration

Regression in #573 (f2341b2b).

Replace BLAKE2b with BLAKE3 (#519)

Add missing “#pragma once”

Allow the config file to be a symlink

Fixes #607.

(cherry picked from commit 46172c8bc24799ae35cf300b0b5ba765bd40896a)

Add “clang-format off/on” comments for old Clang-Format versions

Rename buildenv to dockerfiles to make room for build* directories

Clean up CMake configuration

Format config.h.in with Clang-Format and fix spelling

Improve INSTALL.md after CMake-ification

I chose to mention “-DCMAKE_BUILD_TYPE=Release” explicitly since some
end users will clone the repository instead of downloading the release
archive and then the default “Debug” build won’t be a great idea.

Don’t update author list when generating documentation

Motivation:

1. The copyright notice in each source file refers to doc/AUTHORS.adoc
   for the list of contributors. If we only update AUTHORS.adoc when
   generating documention then AUTHORS.adoc will become more and more
   out of date until somebody remembers to commit the changes.
2. The list is based on commit authors, but authors may sometimes need
   an entry in .mailmap so there’s manual work involved to sanity check
   the list. It can also happen that some commits or authors may need to
   be excluded.

Therefore I prefer to keep AUTHORS.adoc manually updated at release time
(or occasionally more often) with the help of a script just like before.

Remove cmake-format check and support

Motivation:

1. It’s unsatisfying to have to remember to run “make format” after
   editing CMake files. (Clang-Format has more editor integrations, and
   most importantly there are integrations for my editor of choice.)
2. The output of cmake-format seems to be a bit unstable between
   versions.
3. I don’t like some of cmake-format’s formatting choices. This could be
   potentially be improved, though.

Let’s remove the support for now and maybe revisit later.

Docker: add Alpine to new docker setup (#606)

Add and use Fd and Finalizer classes

Add Digest class, replacing struct digest

Fix NEWS entry about switching to CMake

ccache has never used Automake.

Fix spelling

Place temporary test dirs in CMAKE_BINARY_DIR

That’s where the developer performs the build so I think that it makes
more sense this way.

Tell CTest about skipped tests

Return code 125 is chosen for familiarity with what “git bisect” uses.

Support NVCC’s “--Werror” option

doc: Improve wording in “Sharing a cache on NFS”

Improve documentation about sharing a cache on NFS (#604)

Switch to CMake (#573)

Co-authored-by: Cristian Adam <cristian.adam@gmail.com>
Co-authored-by: Joel Rosdahl <joel@rosdahl.net>

Require working script command for the color_diagnostics test suite

portability: Don’t pass “-r” to sed

Make hard-linked files read-only for safety

Mention in manual that only object files are hard-linked

As implemented in 82ecace9.

Add more tests for dependency file content

Remove redundant #includes

Always cache colored diagnostics and strip color codes as appropriate (#596)