Skip to content

Commit b8e5ee1

Browse files
author
CybLow
committed
fix(deps): fix libjxl search path and add findutils dep
- Use /usr/lib64 for libjxl search (Fedora uses /usr/lib64 for both x86_64 and aarch64, unlike Debian multiarch) - Add findutils to Fedora tools deps (needed by compat layer)
1 parent 1c5c15e commit b8e5ee1

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

packages/playwright-core/src/server/registry/fedoraCompat.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -169,8 +169,8 @@ async function createLibjxlSymlink(): Promise<void> {
169169
if (fs.existsSync(symlinkPath))
170170
return;
171171

172-
// Find system libjxl.
173-
const libSearchDir = os.arch() === 'arm64' ? '/usr/lib/aarch64-linux-gnu' : '/usr/lib64';
172+
// Find system libjxl. Fedora uses /usr/lib64 for both x86_64 and aarch64.
173+
const libSearchDir = '/usr/lib64';
174174
const { stdout } = await spawnAsync('find', [libSearchDir, '-name', 'libjxl.so.0.*', '-not', '-type', 'l'], {});
175175
const systemLib = stdout.trim().split('\n')[0];
176176
if (systemLib && fs.existsSync(systemLib)) {

packages/playwright-core/src/server/registry/nativeDeps.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1208,6 +1208,7 @@ export const deps: any = {
12081208
'curl',
12091209
'binutils',
12101210
'tar',
1211+
'findutils',
12111212
],
12121213
chromium: [
12131214
'nss',

0 commit comments

Comments
 (0)