From 24f5d7a9c3dbeaee34858058b74b15f72ff27134 Mon Sep 17 00:00:00 2001
From: John Ericson <John.Ericson@Obsidian.Systems>
Date: Wed, 28 May 2025 12:49:13 -0400
Subject: [PATCH] Fix warning when `HAVE_EMBEDDED_SANDBOX_SHELL` is not set

Clang doesn't like the double indent that is needed for the `if...else`
that is CPP'd away. Adding braces is fine in the `if...else...` case,
and fine as a naked block in the CPP'd away case, and properly-indented
both ways.
---
 src/libstore/unix/build/linux-derivation-builder.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/libstore/unix/build/linux-derivation-builder.cc b/src/libstore/unix/build/linux-derivation-builder.cc
index efbe4c8bb..23850c373 100644
--- a/src/libstore/unix/build/linux-derivation-builder.cc
+++ b/src/libstore/unix/build/linux-derivation-builder.cc
@@ -685,7 +685,9 @@ struct LinuxDerivationBuilder : DerivationBuilderImpl
                 chmod_(dst, 0555);
             } else
 #  endif
+            {
                 doBind(i.second.source, chrootRootDir + i.first, i.second.optional);
+            }
         }
 
         /* Bind a new instance of procfs on /proc. */