Github messages for voidlinux
 help / color / mirror / Atom feed
From: r-ricci <>
Subject: Re: [PR PATCH] [Updated] xbps-src: don't export CCACHE_COMPRESS=1
Date: Wed, 31 Aug 2022 16:03:35 +0200	[thread overview]
Message-ID: <20220831140335.eSt-S57Ile3lYu-b_KchtK3mFEmXhHTJnRkT8ic6V0k@z> (raw)
In-Reply-To: <>

[-- Attachment #1: Type: text/plain, Size: 1962 bytes --]

There is an updated pull request by r-ricci against master on the void-packages repository ccache-compress

xbps-src: don't export CCACHE_COMPRESS=1
Compression is enabled by default since ccache 4.0 and this variable
prevents disabling it through a config file (useful e.g. on compressed

AFAIK there's no way to disable compression when `CCACHE_COMPRESS` is present, since the environment has the highest precedence, according to the manual:

The priorities of configuration options are as follows (where 1 is highest):

 1. Environment variables.

 2. The primary (cache-specific) configuration file (see below).

 3. The secondary (system-wide read-only) configuration file
     <sysconfdir>/ccache.conf (typically /etc/ccache.conf or

 5. Compile-time defaults.

<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

#### New package
- This new package conforms to the [package requirements]( **YES**|**NO**

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
#### Local build testing
- I built this PR locally for my native architecture, (ARCH-LIBC)
- I built this PR locally for these architectures (if supported. mark crossbuilds):
  - aarch64-musl
  - armv7l
  - armv6l-musl

A patch file from is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-ccache-compress-38986.patch --]
[-- Type: text/x-diff, Size: 925 bytes --]

From 17f39b898b6874113439ad9c5d713f7c7aea2c5e Mon Sep 17 00:00:00 2001
From: Roberto Ricci <>
Date: Tue, 30 Aug 2022 18:26:31 +0200
Subject: [PATCH] xbps-src: don't export CCACHE_COMPRESS=1

ccache enables compression by default since version 4.0.
This variable is redundant and prevents disabling compression through
a config file (useful e.g. on compressed filesystems).
 xbps-src | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xbps-src b/xbps-src
index f93f5c5827fd..ba8c92b15c4c 100755
--- a/xbps-src
+++ b/xbps-src
@@ -679,7 +679,7 @@ if [ "$XBPS_CCACHE" ]; then
     export CCACHE_DIR="$XBPS_HOSTDIR/ccache"
     # Avoid not using cached files just due to compiler mtime
     # changes when e.g. bootstrapping
+    export CCACHE_COMPILERCHECK=content
     export PATH="$CCACHEPATH:$PATH"
     mkdir -p $CCACHE_DIR

  parent reply	other threads:[~2022-08-31 14:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-30 16:57 [PR PATCH] " r-ricci
2022-08-31 12:47 ` [PR REVIEW] " sgn
2022-08-31 13:30 ` r-ricci
2022-08-31 14:03 ` r-ricci [this message]
2022-11-20 17:58 ` [PR PATCH] [Closed]: " r-ricci

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220831140335.eSt-S57Ile3lYu-b_KchtK3mFEmXhHTJnRkT8ic6V0k@z \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).