Introduce and use TemporaryFile class

In addition to improving code clarity, this fixes a bug where the
fallback code for “-fdiagnostics-color” tried to operate on a closed
file descriptor.

Add CI job that verifies -DENABLE_TRACING

Make -DENABLE_TRACING work again

Remove now unused legacy create_tmp_file

Look for the correct phrase when detecting unsupported GCC color option

The implementation from #596 uses the spelling “command-line” instead of
“command line” (which is what is actually emitted by GCC and specified
in #224).

Make it possible to keep the test directory when running test suite

Remove test directory for skipped test suite

Update NEWS

(cherry picked from commit 4293f48b6670a99eaa1d5701718d8ef3f2ddd70d)

Use apostrophes and citation marks in NEWS.adoc consistently

(cherry picked from commit d524e19712e832bc9cf3db841c8b5ccd37c0e9f9)

Allow (require) argument to -Werror for NVCC

Fixes #598.

(cherry picked from commit c5349149864104146dab2c9bcd67a1b2ff565f06)

Add knowledge about -fprofile-{correction,reorder-functions,values}

Fixes #615.

(cherry picked from commit c1bf116630f22952732feeffaa71b827c1e813bd)

Handle EINTR properly in Result::Writer::write_embedded_file_entry

Avoid some format/free usage

Remove no longer needed cleanup of CACHEDIR.TAG in $CCACHE_DIR

The cleanup functionality was added in 3.2 and it should be safe to
assume that it has performed its task for almost everybody now.

Remove now unused legacy util functions

Remove usage of x_strdup

Use Util::dir_name instead of legacy dir_name function

Use Util::read_file instead of legacy read_file function

Add size_hint parameter to Util::read_file

Similar to the legacy read_file’s size_hint parameter.

Let Util::write_file default to binary mode

That makes more sense since that’s how it’s used in practice.

Use Util::read_file instead of legacy read_text_file

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