From 515665908ed75e1a5c7cc504a87c4b4355e7ac14 Mon Sep 17 00:00:00 2001
From: Andrey Lushnikov <lushnikov@chromium.org>
Date: Thu, 13 Aug 2020 11:39:50 -0700
Subject: [PATCH] chore: add comment to clarify /sbin

---
 src/server/validateDependencies.ts | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/server/validateDependencies.ts b/src/server/validateDependencies.ts
index a63e9b7161..cfe530dc64 100644
--- a/src/server/validateDependencies.ts
+++ b/src/server/validateDependencies.ts
@@ -220,6 +220,9 @@ async function missingDLOPENLibraries(browser: BrowserDescriptor): Promise<strin
   const libraries = DL_OPEN_LIBRARIES[browser.name];
   if (!libraries.length)
     return [];
+  // NOTE: Using full-qualified path to `ldconfig` since `/sbin` is not part of the
+  // default PATH in CRON.
+  // @see https://github.com/microsoft/playwright/issues/3397
   const {stdout, code, error} = await spawnAsync('/sbin/ldconfig', ['-p'], {});
   if (code !== 0 || error)
     return [];