chore: Clean up config.h

refactor: Move Util::localtime to util

chore: Improve code style

refactor: Move Util::is_precompiled_header to argprocessing

refactor: Move Util::to_lowercase to util

chore: Add note about using util::rename instead of fs::rename

refactor: Use std::filesystem::current_path

refactor: Improve std::filesystem wrappers

I couldn't figure out how to make the templated versions support both

    path current_path();

and

    void current_path(const path& p);

so using preprocessor macros instead.

refactor: Use std::filesystem::remove_all

chore: Improve util::format_argv_for_logging

refactor: Move Util::format_argv_for_logging to util

refactor: Move Util::get_{actual,apparent}_cwd to util

refactor: Use std::filesystem::create_director{ies,y}

refactor: Move Util::split_into_* to util

refactor: Move Util::size_change_kibibyte to LocalStorage

refactor: Move Util::set_cloexec_flag to util

refactor: Create util::filesystem wrappers returning nonstd::expected

refactor: Move Util::fallocate to util

chore: Fix grammar in comment

refactor: Move UmaskScope to util

refactor: Move Util::{g,s}et_umask to util

refactor: Use fs alias for std::filesystem

fix: Pass error code to fs::temp_directory_path as intended

As noted by Thibault Odorico.

refactor: Move Util::parse_duration to util

refactor: Use std::filesystem::current_path

refactor: Use std::filesystem::canonical

refactor: Use std::filesystem::temp_directory_path

chore: Tweak code style

bump: Upgrade to hiredis 1.2.0

feat: Support multiple -Xarch_* arguments matching -arch (#1131)

refactor: Move Util::rename to util

refactor: Use fs::read_symlink instead of custom implementation

refactor: Get rid of asprintf usage and Windows compatibility wrapper

refactor: Use fs::create_hard_link instead of custom implementation

refactor: Move copy_file to util

refactor: Move clone_hard_link_or_copy_file to local storage class

refactor: Move environment utility functions to util

refactor: Move functions for endian conversion to util

refactor: Move data type conversion utilities to util

refactor: Improve InodeCache::get signature

chore: Remove now unused Hash::HashType distinction

After 561be2085df94b3c35dd803e56668e6feafc93b7 hash debugging will never
be enabled for a Hash instance that hashes binary data, so remove the
now unused Hash::HashType enum and related code.

refactor: Use nonstd::span for data+size in more places

refactor: Use std::array for hash digests instead of custom class

refactor: Improve format_base16/format_base32hex implementations

refactor: Move format_base16/format_base32hex to util

chore: Add missing #include statements

bump: Upgrade to BLAKE3 1.4.1

chore: Tweak code style

feat: Kill Windows compiler subprocess when ccache is aborted (#1274)

feat: Support escaping dollar signs in configuration values

feat: Support environment variables in all configuration items

As mentioned in #1307.

chore: Remove stray -warnings-as-errors entry in .clang-tidy

chore: Improve log message for nonexistent header file for manifest

chore: Make some constructors explicit

feat: Expand environment variables in debug_dir configuration (#1307)

fix: Improve nvcc handling of -Xcompiler and --Werror (#1304)

fix: Improve MSVC .rsp file parsing (#1303)

feat: Let ignore_options/CCACHE_IGNOREOPTIONS also skip option

The ignore_options (CCACHE_IGNOREOPTIONS) configuration makes ccache
exclude matching compiler options from the input hash. This commit also
makes ccache skip matching compiler options from any special processing,
similar to how --ccache-skip works.

As discussed in #1284 and #1301.

chore: Update typos configuration

feat: Don't hash /Fd, /FS or /MP (#1298)

/Fd is to specify the file name of output PDB. But currently ccache only
supports /Z7 which doesn't generate PDB at compile time.

/FS forces writes to PDB to be serialized through MSPDBSRV.EXE.
It shouldn't affect generated files and it has no effect for /Z7.

/MP enables /FS. It creates multiple compiler instances and simultaneously
compile multiple source files which is not supported by ccache.

chore: Fix bad markup in NEWS.adoc

chore: Update authors

chore: Update NEWS

chore: Update .mailmap

bump: Upgrade to cpp-httplib 0.12.6

bump: Upgrade to BLAKE3 1.4.0

refactor: Use std::size to compute array size (#1297)

build: Fix Xcode build of zstd (#1296)

Reproducer (CMake 3.26.4, Xcode 14.3):

    cmake -B _build -GXcode -DCMAKE_OSX_ARCHITECTURES=x86_64 \
          -DZSTD_FROM_INTERNET=ON -DREDIS_STORAGE_BACKEND=OFF
    cmake --build _build

See facebook/zstd#3622

fix: Keep the order of MSVC showIncludes output (#1295)

The bug is introduced in commit 3ec0d4ec961cb0c67ff1603b83a6026d9f3abbb5.

fix: Don't pass -v to the preprocessor

The -v option tells the compiler to print various diagnostics to stdout,
including randomly generated filenames. When ccache calls the
preprocessor and -v is on the command line, the random information will
become part of the input hash, making the preprocessor mode always
produce cache misses.

Fix this by only passing -v when compiling, not preprocessing.

Closes #1293.

Merge pull request #1290 from rkitover/cmake-fixes

chore: Add basic CMakeSettings.json for VS IDE

Add a basic CMakeSettings.json for the Visual Studio IDE using defaults
and Ninja as the generator, for x64, x86 and arm64.

Signed-off-by: Rafael Kitover <rkitover@gmail.com>

fix: Quiet cmake warning for extracted timestamps

For zstd and hiredis downloaded from the Internet, set cmake policy
CMP0135 to NEW to set the timestamps of extracted archive members to the
time of extraction to suppress the cmake dev warning on newer versions
of cmake.

Check that the policy exists so that this works on older versions of
cmake.

Signed-off-by: Rafael Kitover <rkitover@gmail.com>

chore: Re-enable 64 bit msbuild ci jobs

Fixed by 70bf22a5 (fix: Disable masm on msbuild for blake3 with VS,
2023-05-22).

References #1278

Signed-off-by: Rafael Kitover <rkitover@gmail.com>

fix: Disable masm on msbuild for blake3 with VS

Because of some bug with either msbuild or the cmake generator, the C
compiler flags are passed to the masm assembler making it fail.

Use the C intrinsic versions for blake3 when msbuild is in use.

Fix #1278

Signed-off-by: Rafael Kitover <rkitover@gmail.com>

bump: Update blake3 1.3.1 -> 71a2646

Update blake3 to master, commit 71a2646 .

Signed-off-by: Rafael Kitover <rkitover@gmail.com>

docs: Clarify --evict-older-than semantics

Closes #1292.

fix: Support Windows drive letter in file URLs for remote storage

Closes #1291.

fix: Make --trim-max-size accept 0 for no limit

test: Simplify util::read_file_part test again

build: Enable _GLIBCXX_ASSERTIONS in debug build mode

See also #1288.

fix: Fix undefined behavior in util::read_file_part for zero count

If util::read_file_part's count argument is 0 (which can currently only
happen in unit tests), we'll do &result[0] where result is
default-initialized std::unique_ptr<uint8_t[]>. This is not allowed even
though we won't actually dereference the pointer. Found by compiling
with -D_GLIBCXX_ASSERTIONS.

Fixes #1288.

test: Fix typo (missing line continuation) (#1287)

Fixes  typo in ef634bdb292e1e24b8d1b5490e7857144a77c0fd.

chore: Update authors

chore: Update NEWS

bump: Upgrade to cpp-httplib 0.12.3

test: Fix test failure in GitHub CI mingw64 jobs

doc: Add remote file storage example with URL-encoded space

As suggested in #1281.

bump: Upgrade to doctest 2.4.11

Fixes #1194.

chore: Suppress clang-tidy warnings we don't want to fix

refactor: Improve calculate_result_and_manifest_key parameters

As suggested by clang-tidy
(readability-suspicious-call-argument,-warnings-as-errors).

refactor: Use constructor member initializer lists

As suggested by clang-tidy
(cppcoreguidelines-prefer-member-initializer).

refactor: Use std::unique_ptr for data in util::Bytes

chore: Sort .clang-tidy check list

chore: Add comments for all statistics counters

As suggested in discussion #1271.

refactor: Extract MSVC debug option logic to a function

fix: Limit MSVC debug flag detection to documented parameters (#1263)

fix: Don't add additional depend mode options after Clang -- option

With the depend mode enabled, ccache executes the original command line
verbatim, potentially with the addition of a few extra options. However,
for clang or clang-cl it is not possible to simply append the additional
options to the end of the original command line since there may be a
"--" option to indicate the end of options.

Fix this by inserting the additional options directly after the compiler
instead of at the end.

Fixes #1273.

bump: Update to zstd 1.5.5 (#1272)

fix: Find Ccache version from lightweight Git tag

Ccache calculates its version from a matching Git tag by using the "git
describe" command, which only considers annotated tags. All ccache tags
are annotated. However, when running a GitHub action job for a tag the
local Git repository is (at least by default) a shallow clone with the
tag being converted to a lightweight tag. This makes "git describe" not
see the tag.

Fix this by adding --tags to "git describe" to make it consider
lightweight tags as well.

Fixes #1270.