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

Refactor context initialization code into a new set_up_context function

Add Util::matches_dir_prefix_or_file helper function

To be used for “ignore_headers_in_manifest”.

Add cross-platform Util::is_dir_separator helper function

Remove now superfluous cast to (void)

Clean up function signatures with (void)

Merge pull request #555 from totph/rm_strtok

Remove legacy strtok_r()

Adapt hash_common_info() to use split_into_strings()

Adapt *_depfile() functions to use split_into_views()/_strings()

Adapt process_preprocessed_file() to use split_into_strings()

Adapt hash_command_output() to use split_into_strings()

Adapt dependencies_env processing to use split_into_views()

Adapt find_executable() to use split_into_strings()

Make win32getshell() return std::string

Adapt args_init_from_string() to use split_into_strings()

Adapt add_prefix() to use split_into_strings()

Util: Add split_into_views/split_into_strings functions

Util: Don’t include path in error description from read_file/write_file

By only returning the error description the caller is more flexible when
composing their own error message.

Upgrade to fmt 6.2.0

Improve order of Travis CI jobs

Improve Travis CI job descriptions

Fix typos in 64-bit MinGW Travis CI job

Add test case for Args::from_gcc_atfile

Add an Args class with backward compatibility for the old args API (#575)

The idea of this is to make a proper C++ API while at the same time
implementing most of the old legacy args API. This makes it possible to
refactor call sites gradually instead of in a series of large
refactoring steps.

Given that “struct args* args” is replaced with “Args args”, the
following types of legacy API usage can be kept without source changes:

1. The args_* functions can be used as is.
2. args->argv[i] still works for read access.

Things that aren’t emulated:

1. args->argc (use args.size() instead).
2. args->argv to access the underlying char* array (use
   args.to_argv().data() instead).
3. Mutating the args->argv[i] string in place (don’t do that).

In other words, code that uses args_foo(args, ...) and args->argv[i] can
easily be converted to args.bar(...) and args[i] (with std::string
arguments instead of char*) at will and in arbitrarily small steps.

Make ccache build on Solaris 11.4 (#572)

These changes enable building on Oracle Solaris 11.4 for both i386 and sparc.

Enable check_format and clang-tidy checks in CI (#562)

Improve const-ness of parameters related to argv