From e1e1035e10874f062dcafd4f212469464ecffd99 Mon Sep 17 00:00:00 2001 From: Andrey Lushnikov Date: Fri, 19 Aug 2022 07:18:41 -0700 Subject: [PATCH] chore: do not specify custom `LD_LIBRARY_PATH` for Firefox (#16681) We used to bundle `libstdc++` for Firefox but we no longer do so, so this code should not be needed. --- packages/playwright-core/src/server/firefox/firefox.ts | 7 ------- 1 file changed, 7 deletions(-) diff --git a/packages/playwright-core/src/server/firefox/firefox.ts b/packages/playwright-core/src/server/firefox/firefox.ts index a55d28750b..9f8d3ef129 100644 --- a/packages/playwright-core/src/server/firefox/firefox.ts +++ b/packages/playwright-core/src/server/firefox/firefox.ts @@ -46,13 +46,6 @@ export class Firefox extends BrowserType { _amendEnvironment(env: Env, userDataDir: string, executable: string, browserArguments: string[]): Env { if (!path.isAbsolute(os.homedir())) throw new Error(`Cannot launch Firefox with relative home directory. Did you set ${os.platform() === 'win32' ? 'USERPROFILE' : 'HOME'} to a relative path?`); - if (os.platform() === 'linux') { - return { - ...env, - // On linux Juggler ships the libstdc++ it was linked against. - LD_LIBRARY_PATH: `${path.dirname(executable)}:${process.env.LD_LIBRARY_PATH}`, - }; - } return env; }