From 870bb68d388eb0df7f0f591ebca7a45748c2aa67 Mon Sep 17 00:00:00 2001 From: John Ericson Date: Fri, 15 Aug 2025 00:43:36 -0400 Subject: [PATCH] `DerivationBuilder::registerOutputs`: Inline `checkSuffix` It is a simple constant that is only used once. --- src/libstore/unix/build/derivation-builder.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/libstore/unix/build/derivation-builder.cc b/src/libstore/unix/build/derivation-builder.cc index 7520933c4..c6027ed61 100644 --- a/src/libstore/unix/build/derivation-builder.cc +++ b/src/libstore/unix/build/derivation-builder.cc @@ -1422,8 +1422,6 @@ SingleDrvOutputs DerivationBuilderImpl::registerOutputs() outputs to allow hard links between outputs. */ InodesSeen inodesSeen; - Path checkSuffix = ".check"; - std::exception_ptr delayedException; /* The paths that can be referenced are the input closures, the @@ -1839,7 +1837,7 @@ SingleDrvOutputs DerivationBuilderImpl::registerOutputs() if (newInfo.narHash != oldInfo.narHash) { miscMethods->noteCheckMismatch(); if (settings.runDiffHook || settings.keepFailed) { - auto dst = store.toRealPath(finalDestPath + checkSuffix); + auto dst = store.toRealPath(finalDestPath + ".check"); deletePath(dst); movePath(actualPath, dst);