Skip to content
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

Do not lock default setting value of syncthing daemon executable path in bundle #242

Merged
merged 11 commits into from
Mar 2, 2025

Conversation

calmh
Copy link
Member

@calmh calmh commented Mar 2, 2025

Since this is the number one complaint. Not tested, just looks reasonable.

Fixes #228

Copy link
Member

@xor-gate xor-gate left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks even better then what I had in mind. The use case where a power-user sets a path outside the bundle is still valid with this fix. Not sure what I was thinking when locking the executable in the defaults store into the bundle. For some odd reason (new) people run .app from DMG on macOS, drag to applications and then unmount them. I'm not a fan of this usage, but the app should not break when people use it in this way.

@xor-gate
Copy link
Member

xor-gate commented Mar 2, 2025

Thanks @calmh for your effort digging through my code !

@xor-gate xor-gate changed the title Do not lock in executable path Do not lock default setting value of syncthing daemon executable path in bundle Mar 2, 2025
@xor-gate
Copy link
Member

xor-gate commented Mar 2, 2025

@calmh could you enable edit by maintainers for this PR, the CI seems to be broken. Seems deprecated runner, need to upgrade to macos-14 at least (or do it yourself). Then I can test the DMG before putting it in develop. Thanks!

@calmh
Copy link
Member Author

calmh commented Mar 2, 2025

It's a branch in the repo, you can just push to it :)

@xor-gate
Copy link
Member

xor-gate commented Mar 2, 2025

Was unaware of it, thanks!

@xor-gate xor-gate added this to the v1.29.2-2 milestone Mar 2, 2025
@xor-gate xor-gate merged commit c67c156 into develop Mar 2, 2025
6 of 7 checks passed
@xor-gate xor-gate deleted the jb/execpath branch March 2, 2025 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants