jens/flipper
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix tmp usage in crash reporter setup

Browse Source 
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
This commit is contained in:
Pascal Hartig
2020-05-15 04:24:41 -07:00
committed by Facebook GitHub Bot
parent 67e05d1d2d
commit c54028981f
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
desktop/app/src/utils/electronCrashReporter.tsx
View File

@@ -22,7 +22,7 @@ const tmpName = promisify(tmpNameCallback) as (
// Cross platform way to find the /tmp directory or equivalent. // Cross platform way to find the /tmp directory or equivalent.
// The tempPath set should be persistent across app restarts. // The tempPath set should be persistent across app restarts.
const tempPathPromise: Promise<string> = tmpName({ const tempPathPromise: Promise<string> = tmpName({
template: '/tmp/tmp-XXXXXX', template: 'tmp-XXXXXX',
}).then((name) => resolve(name, '..', 'flipper')); }).then((name) => resolve(name, '..', 'flipper'));
export default function initCrashReporter(sessionId: string): Promise<void> { export default function initCrashReporter(sessionId: string): Promise<void> {