C++-ify unlink wrapper functions

x_unlink(p)   == 0 -> Util::unlink_safe(p)
x_try_unlink  == 0 -> Util::unlink_safe(p, Util::UnlinkLog::ignore_failure)
tmp_unlink(p) == 0 -> Util::unlink_tmp(p)

Revert "Merge catch2_tests into unittest’s main"

This reverts commit 4608eb6d6c41aefb7c5f534166901e051e126115.

Since unittest/main.cpp includes src/Util.hpp it’s still too painful to
compile the Catch2 runner, so let’s keep it separate.

Use raw string literals for version and usage texts

Default to zstd compression level 1

After some experiments my evaluation is that the increased compression
ratio of level 1 is worth the very slight decrease in compression speed.

Add and use win32_error_message function

Remove overengineered win32 gethostname fallback code

Let’s assume that all systems we reasonably support have access to
gethostname. If not, revisit later.

Regarding the win32 case: Since the replacement code (triggered if
HAVE_GETHOSTNAME isn’t defined) itself uses gethostname, just use
gethostname.

Add unit test for config key tables consistency

Refactor away temp_dir()

Extract internal tracing stuff to a separate file

Remove unit test comments that are hard to keep up to date

Remove now obsolete unittest utils

Convert create_file usage to Util::write_file

Merge catch2_tests into unittest’s main

The Catch2 runner was previously compiled separately since it’s slow to
compile, but after the removal of the old test framework there should be
little reason to modify unittest/main.cpp often.

C++-ify argprocessing test suite

Remove last traces of the old unittest framework

Add TestContext to test cases that create local files

Implement TestUtil::TestContext

Implement Util::wipe_path

Use Util::traverse in Util::clean_up_internal_tempdir

Use Util::traverse in Util::get_level_1_files

Implement Util::traverse

Use using instead of typedef

Convert legacy_util test suite to Catch2

Convert hashutil test suite to Catch2

Convert hash test suite to Catch2

Convert compopt test suite to Catch2

Convert argument_processing test suite to Catch2

Remove obsolete ScopeGuard

Upgrade string-view-lite to 1.4.0

Guess compiler using substring search again

Regression in 30c10e8791bd02a8a3b13beeda875adbefa8c7ab.

Remove now obsolete from_cstr function

C++-ify get_result_name_from_cpp

Speed up check-syntax rule

Fix typo in manifest_get log message

Support clang options for compiling with sampled profiles (#589)

Add support for -fprofile-sample-use and -fprofile-sample-accurate.

Remove legacy args API

Remove last usage of legacy args->argv wrapper

Remove last usage of legacy args_strip function

Remove last usage of legacy args_set function

Remove last usage of legacy args_remove_first function

Remove last usage of legacy args_pop function

Remove last usage of legacy args_init_from_string function

Remove last usage of legacy args_init_from_gcc_atfile function

Remove last usage of legacy args_init function

Remove last usage of legacy args_extend function

Remove last usage of legacy args_add_prefix function

Remove last usage of legacy args_add function

C++-ify process_args

Extract code for handling dependency environment variables to a function

C++-ify final parts of process_arg

C++-ify language functions

C++-ify parts of process_arg

C++-ify compopt functions somewhat

Remove obsolete CHECK_DOUBLE_EQ macro

Remove obsolete path_exists function

Handle Intel compiler option “-xHost” correctly

Fixes #587.

Use args[i] consistently to refer to the argument in process_arg

A shorter alias is nice but breaks symmetry when code needs to process
args[i + 1], etc.

C++-ify -optf/--options-file processing in process_arg

This also makes the implementation acually work with multiple option
files delimited by commas.

C++-ify parts of process_arg

Improve Args::from_gcc_atfile

Scanning for the null character from the string is more efficient.

Simplify detect_pch

C++-ify detect_pch

C++-ify ArgumentProcessingState members

Improve ordering of ArgumentProcessingState members

C++-ify ArgsInfo::debug_prefix_maps

C++-ify ArgsInfo::arch_args

C++-ify ArgsInfo::sanitize_blacklists

Extract argument processing code to a separate file

Fix build error with --enable-tracing

Extract make_relative_path to Util

Don’t regenerate version.cpp on every build

This fixes a problem where version.o will always be out-of-date, thus
triggering a rebuild and relink (regression in 827237fd).

Add Args::empty method

Let process_args use orig_args from ctx

Don't depend on how echo interprets backslash escapes (#588)

The escape sequence '\n' should be translated to a new line character
when printed to version.cpp. Default mode might be implementation
dependent. At least in Fedora /usr/bin/echo defaults to -E. On the
other hand older versions might not support the -e option that enables
backslash escapes where it is not default.

Split the command into two, one for each output line, which should
produce same output on all platforms.

Fix variable shadowing after d0eb89de

Improve handling of profiling options

Handling of -fprofile-{generate,use}[=path] was implemented in PR #2
(2011). ccache has since then gained support for Clang, which is not
GCC-compatible for -fprofile-{generate,use}[=path]. Furthermore, GCC 9
changed the semantics of -fprofile-{generate,use}=path, making it
incompatible with ccache’s original implementation.

One crucial problem with the implementation is that there is no error
handling when the expected profiling data file cannot be found. This
means that there will be false positives cache hits for Clang and GCC
9+.

Fix this by:

* Checking for different profiling data file locations to handle
  different compilers and versions.
* Bailing out if no profiling data file can be found.

Also:

* Implemented support for Clang’s -fprofile-instr-{generate,use}[=path]
  options.
* Implemented support for -fauto-profile[=path].
* Removed the conversion of absolute path to the profile directory.
  91a2954e says that the directory is rewritten to increase the hit
  rate, but I don’t understand how that could be the case.
* Added tests for the profiling options. Exception:
  -fauto-profile[=path], since that would require running the perf tool.

Fixes #582.

(cherry picked from commit c9fdcef576dd13a2ea2355bcd866b5c561cb1562)

Whitelist clang-tidy warning cppcoreguidelines-pro-type-const-cast

We’re using const_cast for the argument argv to execve(2) which doesn’t
modify the argument.

Don’t construct std::string from nullptr

As suggested by clang-tidy.

Use empty() to check for emptiness

As suggested by clang-tidy.

Work around false positive clang-tidy warning about redundant .c_str()

Remove redundant .c_str() calls

As suggested by clang-tidy.

Handle self-assignment in Args::operator=

As suggested by clang-tidy.

Add noexcept to Args move constructor/operator=

As suggested by clang-tidy.

Clean up version output test

Mention the “always use curly braces” convention in CONTRIBUTING.md

Fix syntax of src/.clang-tidy

Indent .sh files with 4 spaces

Disable hardlink testing on AFS (#584)

AFS sort of supports hardlinks, but not really - you can only hardlink
within directories, not across them. This means that the hardlink test
suite probe will pass, but ccache is unable to create hardlinks to its
cache directory and the test suite will fail. Modify
SUITE_hardlink_PROBE() to use a cross-directory hardlink to correctly
detect this case.

Co-authored-by: Alexander Lanin <alex@lanin.de>
Co-authored-by: Joel Rosdahl <joel@rosdahl.net>

Extract formatting logic from make (#585)

This way it is executable without running configure first.

Add test for human readable version output (#583)

Make update authors script find Co-authored-by that is not in trailer

Clean up update authors script

Diverse CMake preparations (#580)

* Prepare gitignore for builds in subdirectory
* Ignore to be generated Version.cpp
* Move travis to buildenv as it's just another image
* Move install_cuda out of .travis as it's not travis specific
* Prepare docker images for C++ and CMake
* Update authors + add script to do so
* Fix required C compiler
* Fix some formatting issues with clang-format 10

Remove obsolete unittest/test_lockfile.cpp

It should have been removed in c181fdd9.

Fixes #581. See also #569.

Move calculation of “result/manifest globals” into Context

Make Context a class instead of a struct

I like it better that way.

Remove unused #include

Call update_manifest_file in do_cache_compilation()

This reduces complexity of from_cache() and to_cache() somewhat.

C++-ify Context::result_name

C++-ify Context::ignore_headers