build, doc: Warn on unprivileged Guix daemon usage #36241

pull Ronin95 wants to merge 2 commits into bitcoin:master from Ronin95:guix-warn-unprivileged changing 2 files +57 −0
  1. Ronin95 commented at 5:00 AM on September 14, 2026: none

    Motivation and Developer Experience Improvement This PR addresses issue #36239. Developers who use Guix to build Bitcoin Core locally may run into confusing dependency build failures, such as errors when compiling python-distlib or node, if their Guix daemon is running as an unprivileged user.

    Recent unprivileged Guix installations use Linux user namespaces. These can cause some packages to fail when building locally without substitutes because the build environment has different permissions. For example, /etc may be unexpectedly writable, or builds may fail with UID/GID errors.

    The problem is that developers only discover this after a long build has already failed, often with error messages that do not clearly point to the real cause.

    This PR improves the developer experience by:

    1. Adding a check to contrib/guix/guix-build that detects when guix-daemon is running as a non-root user and shows a warning before the build starts.
    2. Updating contrib/guix/INSTALL.md with clear instructions for changing an unprivileged Guix daemon back to a privileged (root) configuration.

    By catching this configuration issue before the build starts, we save developers hours of wasted compilation time and reduce the support burden on maintainers diagnosing environmental build failures.

    Testing Because this check depends on the local system environment, no new automated tests were added. The changes to guix-build were tested manually on Ubuntu 26.04. The check correctly detects when the daemon is running as an unprivileged user and does not report false positives when the Guix daemon is running normally as root.

  2. guix:warn on unprivileged daemon (RFC #36239) f1ab5a7fc2
  3. doc: document unprivileged to privileged guix daemon migration 8a8c383f91
  4. DrahtBot commented at 5:00 AM on September 14, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/36241.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline and AI policy for information on the review process. A summary of reviews will appear here.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  5. hebasto commented at 11:14 AM on September 14, 2026: member

    I suggest keeping this PR as a draft until the ongoing discussion in #36239 reaches a rough consensus.

  6. fanquake marked this as a draft on Sep 14, 2026
  7. maflcko commented at 3:10 PM on September 17, 2026: member

    Yeah, closing for now. No need to keep this open while the issue is itself in "draft"

  8. maflcko closed this on Sep 17, 2026


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-09-20 16:52 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me