farmcontrol/farmcontrol-ui
farmcontrol/farmcontrol-ui
2
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

Refactor Windows path handling in windows-app-paths.js
All checks were successful
farmcontrol/farmcontrol-ui/pipeline/head This commit looks good
Details

Browse Source 
- Removed the use of `chdirSync` in favor of `process.chdir` for changing the working directory, improving compatibility with the Node.js environment.
- Streamlined the code for ensuring the correct working directory is set before launching the application on Windows.
This commit is contained in:
Tom Butcher 2026-08-03 02:37:32 +01:00
parent 3d710e2d2e
commit 4a1badb5aa
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/desktop/windows-app-paths.js
View File

@ -1,4 +1,4 @@
import { chdirSync, existsSync } from 'node:fs' import { existsSync } from 'node:fs'
import { dirname, join } from 'node:path' import { dirname, join } from 'node:path'
const NATIVE_WRAPPER_DLL = 'libNativeWrapper.dll' const NATIVE_WRAPPER_DLL = 'libNativeWrapper.dll'
@ -34,7 +34,7 @@ export function ensureWindowsWorkingDirectory() {
if (!wrapperInCwd && wrapperInBinDir) { if (!wrapperInCwd && wrapperInBinDir) {
try { try {
chdirSync(binDir) process.chdir(binDir)
} catch { } catch {
// Ignore if we cannot change directory. // Ignore if we cannot change directory.
} }