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)

Add tests for verifying dependency file content

(cherry picked from commit c972eb6e824624956444e945bdfd9768c7e74123)

Revert "Only pass implicit -MQ to preprocessor if needed"

This reverts commit 19c3729d30640fc2c78242cb46136e619fdbd802 and also
removes the incorrect and superfluous condition that the fix fixes.

The reason for reverting is that the change introduced a nasty bug: the
dependency file will get an incorrect object file location if 1) build
directory != source directory and 2) -MF is not specified explictly by
the user.

Details: The dependency file is created by the preprocessor pass which
does not have access to the final output file name, so the preprocessor
uses the default object location derived from the source file location
instead.

Note that this partly reverts a compatibility improvement for EDG-based
compilers (see issue #460).

It should be possible to pass the dependency arguments to the compiler
instead of the preprocessor to make -MD/-MMD without -MQ work, but
that’s too risky to be done as a bug fix.

Fixes #499.

(cherry picked from commit 6d4537695f8753556362c6c5521360ca9b596ab2)

Add “--nocpp2” option to misc/performance

(cherry picked from commit dbf7e85f67cf785636341ce07380e7151b582ee6)

Test .incbin detection for assembler compilation

(cherry picked from commit c9530b9d423b3e1b52f2fb7b3cf3376eb3044384)

Remove the unify mode

The unify mode has not received enough attention and has at least these
bugs:

1. The direct mode doesn’t work if the unify mode is enabled. This is
   because the unify mode doesn’t call into process_preprocessed_file
   which stores the paths and hashes of included files needed by the
   direct mode.
2. The .incbin directive detection has no effect when using the unify
   mode. This is again because the unify mode doesn’t use
   process_preprocessed_file which is where the .incbin detection takes
   place.
3. The unifier’s tokenizer doesn’t understand C++11 raw string literals.
4. The unifier ignores comments, but comments may have semantic meaning
   to modern compilers, e.g. “fall through” comments.

Bugs 3 and 4 are fixable by improving the unifier’s tokenization
algorithm, but since it’s a bit tricky it likely won’t be worth the
effort, especially not as a bug fix.

Bugs 1 and 2 are also fixable by unifying the two code paths, but that’s
a non-trivial effort.

In addition to the bugs, I believe that the usefullness of the unify
mode is low:

* It’s only applicable when not using -g.
* It won't be enabled for C++ unless somebody fixes bug 3.
* It can make line numbers in warning messages and __LINE__ expansions
  incorrect.
* Since comments should not be ignored, the unify mode can only make a
  difference for some types of whitespace changes, like adding or
  removing blank lines or changing a+b to a + b. (a + b is already
  normalized to a + b by the preprocessor.)

Therefore I’ll just remove the unify mode to fix the bugs.

Fixes #497.

(cherry picked from commit 947a72ce3712a6901e7ddc43a50c5df40739113e)

Remove superfluous clang-format comments

Unittest fixes on Windows (#496)

Update .mailmap

(cherry picked from commit 38419c8dc3712b46020fd02cef6f608bf66c13fe)

Improve comments related to save_timestamp

(cherry picked from commit 0d0def13a52d7b68376636776e71a9af5bf01bb4)