Context: move arch_args

Context: move output_is_precompiled_header

Partially revert const-ification of context parameters

Improve functions related to CWD

The different functions related to current working directory (CWD) have
become messy during the years:

- gnu_getcwd is a simple wrapper around getcwd(3), thus returning the
  actual canonical path.
- get_cwd returns $PWD, falling back to getcwd(3) if $PWD is not sane.
- get_current_working_dir (local function in ccache.cpp) memoizes
  x_realpath(get_cwd()) (i.e., getcwd(3) in essence...) in the global
  current_working_dir variable. Unit tests may manipulate
  current_working_dir.

Improve this by:

- Replacing gnu_getcwd with Util::get_actual_cwd.
- Replacing get_cwd with Util::get_apparent_cwd.
- Removing get_current_working_dir and placing both actual and apparent
  CWD in the context object.

Simplify (non-)freeing of orig_args on failure

Simplify string creation from string_view

Remove now redundant resetting of config

Since the configuation no longer is global there is no need to reset the
unmodified Config object passed by cct_test_begin to cc_reset.

Const-ify context parameters where appropriate

Merge pull request #542 from totph/ctx4

Context: g_config, stats_file and orig_args moved

Context: move orig_args

Context: convert stats_file to an std::string

Also adapted downstream stats functions.

Context: move stats_file

Context: move g_config

Implement Util::read_link, replacing legacy x_readlink

Implement Util::real_path, replacing legacy x_realpath

Merge pull request #541 from totph/ctx3

Context: move 9 more globals

Context: move using_precompiled_header

Context: move profile_use/profile_generate

Context: move direct_i_file

Context: move profile_dir

Context: move profile_arcs

Context: move seen_split_dwarf

Context: move generating_diagnostics

Context: move generating_stackusage

Context: move generating_coverage

Rename clang-format.sh to run-clang-format

This avoids exposing implementation details.

Build static Windows executables

This makes it possible to run e.g. the unittest binary in Wine.

Simplify result_name_from_depfile signature

Simplify Util::{base,dir}_name

This also fixes a bug in the Windows versions.

Added 'make check_format' to makefile rules (#525)

'make format' formats all source files with clang-format, and
'make check_format' only checks and on failure prints a diff,
then exits with the appropriate status.

Because quick interactive formatting is one use case, xargs is
used for parallelism. Proper make-parallelism would require
-k and -j arguments to work.

Merge pull request #538 from totph/ctx2

Context: move 7 more globals

Context: move generating_dependencies

Context: move actual_language

Context: move output_dwo

Context: move output_dia

Context: move output_cu

Context: move output_cov

Context: move output_dep

Test out-of-tree build in Travis-CI

Fix out of tree build (#540)

The CPPFLAGS contain "-I.", but not "-I$(srcdir)", which means "src/ScopeGuard.hpp" was only
found when building from the source root.

Simplify code

Make clang-format comments stable across clang-format versions

Now at least clang-format 6, 7, 8 and 9 format all code the same.

Fix formatting

Context: move output_obj, adapt dump_debug_log_buffer_exitfn

Context: add ArgsInfo and move input_file

Context struct added, returned by initialize()

The goal is to eventually move almost all global state into Context.

Use Config for init_log()

Remove superfluous “>” in README

Mention “Supported platforms, compilers and languages” page in README

Improve the failed() and fatal() mechanisms

The failure() and fatal() functions now exit by throwing exceptions that
are caught by the top level functions. This makes it possible to “throw
Failure” in functions that don’t have access to orig_args (or the future
context object).

While at it, renamed top-level functions to better reflect their
purpose.

Remove declaration of unimplemented warn function

Restore sortedness of Util functions

Use correct spelling of the verb “set up”

Extract setup_config() from initialize() (#534)

Repeated initialize-calls in ccache_main_options() added duplicate exit
functions. These calls had the side effect of passing a config relevant
changes from one command line switch to the next. Instead setup_config()
is now called afterwards.

Move logging into separate file (#532)

Use apparent CWD when checking for CWD in preprocessed output

Also when checking debug prefix maps.

This fixes a problem when $PWD includes a symlink part and the user has
set “hash_dir = false”.

Closes #531.

(cherry picked from commit 92145eb0762109b5f0114a6f4de2fea7d4ac016f)

Remove “Description” header from PR and feature issue templates

The header seems to be more in the way than being helpful.

Win32: Close reading file for recompression (#535)

Fixes: https://github.com/ccache/ccache/issues/530

Remove one more “=” from the help text

Missed in a44ef1844b59bf902ad6015645fb0028a54f67cb.

Improve help text and documentation of command line options

Removed the “=” in “-a, --a-long-option=VALUE” to make it easier to
understand that “-a=VALUE” isn’t supported. Also improved some of the
placeholder names.

Closes #529.

Improve error message for “ccache -o=K=V”

Previously:

    ccache: error: unknown configuration option ""

Now:

    ccache: error: unknown configuration option "=K"

Related to #529.

Log when starting/stopping recompression

doc: Remove information that is better to have on the web site

doc: Move description of preprocessor mode before direct mode

This way the depend mode is documented directly after the direct mode
which makes more sense since they are related.

Add improvement issue template

Improve issue templates

Compensate for bug in clang-format version 8 and 9

Add short description of ArgsInfo

As suggested by Alexander Lanin.

Simplify code a bit

Fix memory leak

Refactor away the last instance of from_owned_cstr

Merge pull request #515 from totph/cc_proc

Make cc_process_args() only use local variables

Simplify make_relative_path() argument and result ownership

Make cc_process_args() only use local variables

cc_process_args writes into a new ArgsInfo struct and only accesses
local variables (minus guessed_compiler). For now these are mostly
copied into the global variables again after the function call.

Most ccache.cpp global variables moved to legacy_globals.cpp.

Add a ScopeGuard struct

Enables automatic cleanup of raw pointers and other resources.
For now used simplify resource management in cc_process_args.

Add from_owned_cstr/from_cstr functions

Format code with clang-format

Make Config noncopyable and add clear_and_reset() method (#522)

Prevents changes from getting lost by writing them to a
temporary copy of the main Config object.

Also restore deleted comments from the move of
primary/secondary_config_path.

Derive header list from source list if possible (#521)

Skip read-only tests on file systems that lack such support

Fixes #510.

Make failure to write a stats file a soft error (#516)

If a stats file update for some reason fails, the exception will bubble
up to ccache_main which just prints the error and exits with an error.
Let’s consider such failures non-fatal and just log them.

Split ccache.hpp into per-.cpp file headers

This reduces code dependencies that hamper refactoring.

Remove obsolete COMPAR_FN_T check and typedef

Move global primary/secondary_config_path into Config (#514)

Ignore files from VS (Code) (#508)

Make failed tests create a testdir.failed symlink (#511)

When the test suite fails 'testdir.failed' now points to the random
'testdir.9876/' directory. This make iterative debugging of failing
tests easier by always using the symlink name or re-issuing 'cd $PWD'
when 'cd'-ed into the 'testdir.failed' directory.

All artifacts can still be removed via 'rm -r testdir.*'.

Simplify parse_int slightly (#509)

Add configuration for the No Response Probot

Upgrade to fmt 6.1.2

Send dependency arguments to compiler if run_second_cpp is true

If we run the compiler on the real source code on a cache miss (i.e.,
run_second_cpp is true) we can send the dependency arguments (-MD, etc.)
to the compiler instead of the preprocessor. When doing this there is no
need to add an implicit -MQ option to get the correct object file
location in the dependency file. Do so and also avoid adding an implicit
-MF option since that isn’t needed either. This should re-add support
for EDG-based compilers (see 6d453769 and #460).

If we run the compiler on the preprocessed source code on a cache miss
(i.e., run_second_cpp is false) we still need to let the preprocessor
generate the dependency file (the compiler doesn’t generate a dependency
file when compiling preprocessed source code) and thus we need to add
implicit -MQ and -MF.

This change reverts a fix (97b27781) for the Intel C++ compiler, but if
the Intel compiler still has problems the user can simply avoid setting
run_second_cpp to false.

Test extra_args_to_hash output of cc_process_args

Include “Co-authored-by” attributions when calculating authors

(cherry picked from commit cb301769895adc941845b16d124e0bff292a74c3)

Improve error message for bad boolean environment variable value

Before:

    ccache: error: CCACHE_DIRECT: invalid boolean environment variable
    value "0" for CCACHE_NODIRECT (did you mean to set
    "CCACHE_DIRECT=true"?)

After:

    ccache: error: CCACHE_NODIRECT: invalid boolean environment variable
    value "0" (did you mean to set "CCACHE_DIRECT=true"?)

Make it harder to misinterpret semantics of boolean environment settings

Closes #502.

Co-authored-by: Steffen Dettmer <steffen.dettmer@rail.bombardier.com>
(cherry picked from commit f2481b1d5988cb4ccc3010c4ad6933e9c93e6d02)

Avoid passing compilation-only options to the preprocessor redux

25e73c1f (“Include compiler-only arguments in the hash”) attempted to
fix a regression in 5d8585b5 (“Don’t pass -Werror and compilation-only
options to the preprocessor”). It succeeded fixing the regression all
right, but it also essentially reverted the essence of 5d8585b5 (#312)
since compiler-only arguments once again are passed to the preprocessor.
Sigh.

Fix this for real and also write a test that proves it.

(cherry picked from commit bc2005e994030f787c014ce2491cfc4c7057eac6)

misc/performance: Fix --no-cpp2 and --no-stats options

They broke in an unsuccessful cherry-pick (0827d070) as noted in #503.

(cherry picked from commit 0948fb13f07c756b8e056efdec46560e0faf37d1)

Prepare for v3.7.7

(cherry picked from commit 26912b280b839e102482db8c9e63f00a95b58486)

Prepare for v3.7.7

(cherry picked from commit 6cfc49fc46b16d0b3fe9c3cde8ddf3199a36ced6)

Update NEWS

(cherry picked from commit 5c6954378780a7d07faa33d5bc31bdda1a4b8bb1)

Make sure to always log a “Result:” line

Fixes #500.

A side effect of this is that read-only cache misses now will update the
“cache miss” statistics counter, which is consistent with the cache hit
cases (which already now update the counters).

(cherry picked from commit 1d2b76a71b12c2ffdd107f89a3d9caa6cd223d44)