fix: Do not add redundant newlines for stdout

Tokenizer is used with include_empty, and the output typically ends with
\n.

For each line, tokenizer returns it without the \n, and the function
appends
it. But if the output ends with \n, the tokenizer returns an additional
empty string, and a redundant LF is written to stdout.

Another issue can be if the last line of the original output doesn't end
with \n at all, ccache added it anyway.

It was probably unnoticed until now since gcc has no output at all, while
cl outputs the source file name.

feat: Support preserving the delimiters on tokenizer

test: Simplify tokenizer tests

fix: Only use /run/user/<UID>/ccache-tmp if writable

The fix for #984 addressed a problem when /run/user/0 already exists and
ccache is run with fakeroot. However, it didn’t handle the case when
/run/user/0/ccache-tmp already exists, which will happen for instance if
the real root user has run ccache at least once.

Fix this by using access(2) to verify that the ccache-tmp directory is
writable. Note: Can’t just check the mode bits of the directory since
they appear OK since fakeroot fakes the UID.

Fixes #1044.

fix: Add ASM and ASM_MASM to project() languages (#1043)

fix: Improve handling of .gcno files

When support for caching a compilation with coverage (producing a .gcno
file), the commit[1] made sure to avoid rewriting the input path to
relative with the motivation “also make sure to use the source file
path, since this is in the notes”. However, this seems to be unnecessary
since a relative input file path will be written as is to the .gcno, not
the absolute path. This is the case for at least GCC 4.7+ and Clang
3.6+. Fix this by potentially converting the input path to relative even
when generating coverage.

When investigating the above issue, I noticed that GCC 9+ includes the
current working directory (CWD) in the .gcno file. This means that we
have include the CWD in the hash when compiling with
-ftest-coverage/--coverage in order to replicate what the compiler would
produce. Since this makes it impossible get cache hits when compiling in
different directories, a new gcno_cwd sloppiness has been added for
opting out of hashing the CWD, with the tradeoff of potentially getting
an incorrect directory in the .gcno file.

[1]: 02d3f078bd2495b8db2264ae0b2c692b4c5ba1bd

Fixes #1032.

refactor: Sort sloppiness parsing/generation

chore: Disable some clang-tidy checks that don’t contribute much

docs: Add reference to reference build configs (#1031)

fix: Bail out on too hard MSVC environment variables CL and _CL_

To handle the CL and _CL_ variables properly, ccache needs to parse them
according to [1] and then include the compiler options as part of
regular argument processing. Until that is done we have to bail out and
just run the original compiler.

[1]: https://docs.microsoft.com/en-us/cpp/build/reference/cl-environment-variables?view=msvc-170

Fixes #1022.

fix: Prefer CMake find module for hiredis package

refactor: Simplify string append in process_arg

chore: Improve error message for read error in from_cache

fix: Prefer CMake find module for zstd package (#1030)

feat: Support directory for MSVC -Fo option (#1019)

fix: Don't exit with error on read failure from cached file (#1036)

fix: Don’t cache preexisting object file when using -fsyntax-only

Fixes #1034.

fix: Authenticate with Redis before database selection (#1020)

fix: Work around endianness problem in Util::read_text_file

The code in Util::read_text_file for converting UTF-16LE to UTF-8 only
works on little-endian machines. This makes the unit test fail on
big-endian machines.

Since the conversion is only needed on Windows (for Visual Studio, which
creates UTF-16LE .rsp files) in practice, work around the problem by
only doing the conversion in Windows builds.

Fixes #1014.

test: Don’t require C++-capable compiler for the test suite

docs: Add missing word in release notes

chore: Prepare for v4.6

chore: Update NEWS

style: Add missing newline at end of file

chore: Sort codespell allowlist

refactor: Make Util::is_absolute_path_with_prefix return optional

style: Tweak comments

chore: Improve hint in source code format error message

fix: Don’t increment direct_cache_miss for forced recache

fix: Make conversion to relative paths more reliable on Windows (#1011)

chore: Add Vili Väinölä to .mailmap

fix: Hash object file directory for MSVC compilers

The object file produced by MSVC includes the full path to the object
file even without debug flags.

bump: Upgrade to BLAKE3 1.3.1

docs: Tweak hard link config option description

fix: Respect umask when making hard linked files read only

Fixes #1007.

refactor: Extract get_umask function to Util

chore: Touch up Util::read_text_file code

fix: Support UTF-16LE .rsp files (#1005)

docs: Add documentation that compiler plugins are hashed too (#1006)

feat: Remove automatic enabling of ccache for ccache build

When enabling ccache using the standard recommended method (via PATH or
via masquerading symlinks), UseCcache.cmake will introduce double
invocations of ccache. I find this a bit annoying and I also think that
it should be up to the user to enable or not enable ccache and which
settings to use.

chore: Remove obsolete GCC 4.8 workarounds

fix: Ensure that raw filenames are static in length

PrimaryStorage::evict assumes that the entry number is encoded in the
filename as one character, so make sure that this precondition holds in
Result::{Reader,Writer} as well.

refactor: Clean up code related to -Fp/-Yu with concatenated path

fix: Fix clang-cl display name (#1003)

style: Improve names related to clang-cl

docs: Add missing documentation of compiler type “clang-cl”

fix: Don’t finalize ZstdCompressor twice

This fixes an issue with older libzstd versions, which don’t like when
you continue writing data to a stream after calling ZSTD_endStream.

feat: Add support for clang-cl (#993)

Co-authored-by: Luboš Luňák <l.lunak@centrum.cz>

feat: Add option to use Bearer token with HTTP backend (#1001)

bump: Upgrade to cpp-httplib 0.10.2

feat: Enable HTTP keep-alive by default (#1000)

feat: Add --inspect option instead of --dump-manifest/--dump-result

“Inspect” sounds more appropriate than “dump” for viewing metadata of
cache files. Also, it’s convenient to have one option instead of two
different.

enhance: Add File::File(FILE*) constructor

fix: Do not handle -o<file> for MSVC (#994)

Co-authored-by: Luboš Luňák <l.lunak@centrum.cz>

bump: Upgrade to BLAKE3 1.3.0

bump: Upgrade to fmt 8.1.1

bump: Upgrade to doctest 2.4.8

bump: Upgrade to zstd 1.5.2

feat: Detect MSVC compiler case insensitive and with exact match

feat: Make it possible to configure compiler type msvc

refactor: Rename compiler type cl to msvc

fix: Ensure existing directory before copying from secondary storage

Fixes #998.

doc: Fix typo and missing word

test: Verify -fsyntax-only handling without -c

refactor: Simplify -fsyntax-only logic

feat: Add support for MSVC's -Zs option (#995)

Co-authored-by: Luboš Luňák <l.lunak@centrum.cz>

Fix upload-redis after cache entry format change (#973)

feat: Support MSVC /Yu option (compiling with PCH) (#991)

Co-authored-by: Luboš Luňák <l.lunak@centrum.cz>

fix: Unconstify variable to allow automatic move

As suggested by Clang-Tidy.

fix: Make src/ccache.cpp cacheable by ccache itself

“.incbin” should not be present verbatim in the source code since that
makes ccache consider the code uncacheable.

Regression in 2d2de82bb9231b981b40486b093a2c0002200c40.

doc: Improve documentation of --config-path

See also discussion #986.

Fix stats output for secondary storage (#988)

fix: Check writability of /run/user/<UID>/ccache-tmp before using

Closes #984.

refactor: Decouple Manifest data structure from how it’s stored

Manifest is now a standalone data structure that knows how to serialize
and deserialize itself but not how it’s embedded in a cache entry.

bump: Upgrade to xxHash 0.8.1

bump: Upgrade to cpp-httplib 0.10.1

test: Improve code style

style: Avoid operator++ in expression

fix: Avoid .incbin false positive (#983)

fix: Remove duplicate -Xarch second argument (#981)

style: Add some comments and tweaks

fix: Fix copying of binary files on Windows (#979)

fix: Don’t run trim_dir test suite unless cleanup tests are enabled

Fixes #974.

refactor: Move conversion to slash option outside file test

refactor: Remove redundant conditions for MSVC options

fix: Handle stdout from distcc-pump when used as a prefix

The compiler guessing doesn’t work if pump is used as a
prefix/CCACHE_PREFIX, so let’s just not depend on such guesswork.

Also, since the distcc-pump wrapper isn’t really a compiler type, it
actually makes sense not to treat it as such.

feat: Support caching stdout for all compiler types

enhance: Add Result::write_data method

fix: MSVC's -link option is too hard (#972)

fix: Handle MSVC flags -MD[d], -MT[d] and -LT[d] properly (#971)

Fix and simplify handling of options for MSVC (#954)

Add support for caching stdout from MSVC (#962)

fix: Ignore CCACHE_NOCPP2=1 with MSVC (#970)

doc: Correct reference to the debug_dir option

refactor: Simplify code related to send_to_fd

fix: Correct error message in send_to_fd

Merge pull request #968 from jhnc-oss/fix-newline-on-windows

fix: handle newlines on Windows properly

fix: Use default object file extension .obj for MSVC (#964)

fix: Make AVX2 code work also with MSVC (#963)