From c54028981fb2bbcfea84d54077773ae7626cf4dd Mon Sep 17 00:00:00 2001 From: Pascal Hartig Date: Fri, 15 May 2020 04:24:41 -0700 Subject: [PATCH] Fix tmp usage in crash reporter setup Summary: This happened during startup: {F237135281} From `tmp` docs: > IMPORTANT NOTE: template no longer accepts a path. Use the dir option instead if you require tmp to create your temporary filesystem object in a different place than the default tmp.tmpdir. Reviewed By: mweststrate Differential Revision: D21570416 fbshipit-source-id: 170886d0fda5e21cb23fe836fcde33eb457a4c1b --- desktop/app/src/utils/electronCrashReporter.tsx | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/desktop/app/src/utils/electronCrashReporter.tsx b/desktop/app/src/utils/electronCrashReporter.tsx index de7eed255..fc92e4673 100644 --- a/desktop/app/src/utils/electronCrashReporter.tsx +++ b/desktop/app/src/utils/electronCrashReporter.tsx @@ -22,7 +22,7 @@ const tmpName = promisify(tmpNameCallback) as ( // Cross platform way to find the /tmp directory or equivalent. // The tempPath set should be persistent across app restarts. const tempPathPromise: Promise = tmpName({ - template: '/tmp/tmp-XXXXXX', + template: 'tmp-XXXXXX', }).then((name) => resolve(name, '..', 'flipper')); export default function initCrashReporter(sessionId: string): Promise {