-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
v2 linux: SecondInstanceData.WorkingDirectory not initialized #3133
Comments
It's inconsistent or incomplete. Looks same for windows: wails/v2/internal/frontend/desktop/windows/single_instance.go Lines 51 to 54 in dc5f0b6
but different for darwin: wails/v2/internal/frontend/desktop/darwin/single_instance.go Lines 35 to 40 in dc5f0b6
wails/v2/pkg/options/options.go Lines 183 to 194 in dc5f0b6
Using dir of executable in common case is not equal to CWD. |
@APshenkin - is this usage correct for this feature? Any assistance would be appreciated 🙏 |
Will take a look |
* [#3133] SingleInstanceData: Fill missing WD as is * Apply suggestions from code review Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> * Fix: missing `import` for AI suggestions * Add changelog entry --------- Co-authored-by: Lea Anthony <[email protected]> Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Description
v2.7 instoduced
SingleInstanceLock
, but on linux theSecondInstanceData.WorkingDirectory
is always empty:wails/v2/internal/frontend/desktop/linux/single_instance.go
Lines 55 to 57 in dc5f0b6
To Reproduce
Expected behaviour
Screenshots
Attempted Fixes
No response
System Details
Additional context
No response
The text was updated successfully, but these errors were encountered: