Skip to content

Commit

Permalink
Do not lock default setting value of syncthing daemon executable path…
Browse files Browse the repository at this point in the history
… in bundle (#242)

Do not lock executable path at the application bundle location. Fixes #228
  • Loading branch information
calmh authored Mar 2, 2025
1 parent 902f327 commit c67c156
Show file tree
Hide file tree
Showing 5 changed files with 19 additions and 9 deletions.
16 changes: 12 additions & 4 deletions .github/workflows/build-syncthing-macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,21 +17,29 @@ jobs:
build-debug:
name: Build debug
if: github.event_name == 'push' && github.ref != 'refs/heads/release' # Debug not necessary on release branch (develop is intermediate branch)
runs-on: macos-latest
runs-on: macos-13
steps:
- uses: actions/checkout@v4
with:
fetch-depth: 0

- name: Build debug target
run: |
make debug
run: make debug

- name: Prepare debug build for artifact
run: make debug-dist

- name: Archive artifacts
uses: actions/upload-artifact@v4
with:
name: syncthing-macos-debug
path: Build/Products/Debug/dist

build-release:
name: Build release
if: github.event_name == 'push' && github.ref == 'refs/heads/release'
environment: signing
runs-on: macos-latest
runs-on: macos-13
steps:
- uses: actions/checkout@v4
with:
Expand Down
3 changes: 3 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ all: debug
debug:
xcodebuild -workspace "syncthing.xcworkspace" -derivedDataPath $(PWD) -configuration Debug -scheme Pods-syncthing
xcodebuild -workspace "syncthing.xcworkspace" -derivedDataPath $(PWD) -configuration Debug -scheme syncthing
debug-dist:
mkdir -p Build/Products/Debug/dist
cp -r Build/Products/Debug/Syncthing.app Build/Products/Debug/dist
debug-dmg:
xcodebuild -workspace "syncthing.xcworkspace" -derivedDataPath $(PWD) -configuration Debug -scheme Pods-syncthing
xcodebuild -workspace "syncthing.xcworkspace" -derivedDataPath $(PWD) -configuration Debug -scheme syncthing-dmg
Expand Down
2 changes: 1 addition & 1 deletion cmd/update-release.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@
from urllib.request import urlopen
from string import Template

distVersion = 1
distVersion = 2
latest_url = "https://api.github.com/repos/syncthing/syncthing/releases/latest"
infoPlist = 'syncthing/Info.plist'
infoPlistTmpl = 'syncthing/Info.plist.tmpl'
Expand Down
4 changes: 2 additions & 2 deletions syncthing/Info.plist
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@
<key>CFBundlePackageType</key>
<string>APPL</string>
<key>CFBundleShortVersionString</key>
<string>1.29.2-1</string>
<string>1.29.2-2</string>
<key>CFBundleVersion</key>
<string>102900201</string>
<string>102900202</string>
<key>LSApplicationCategoryType</key>
<string>public.app-category.utilities</string>
<key>LSMinimumSystemVersion</key>
Expand Down
3 changes: 1 addition & 2 deletions syncthing/STApplication.m
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ - (void)applicationLoadConfiguration {
_executable = [NSString stringWithFormat:@"%@/%@",
[[NSBundle mainBundle] resourcePath],
@"syncthing/syncthing"];
[defaults setValue:_executable forKey:@"Executable"];
}

_syncthing.URI = [defaults stringForKey:@"URI"];
Expand Down Expand Up @@ -203,7 +202,7 @@ - (void)refreshDevices {
self.toggleAllDevicesItem.title = @"Pause All Devices";
[[self statusMonitor] setCurrentStatus:SyncthingStatusIdle];
}

self.devicesPaused = allPaused;
}

Expand Down

0 comments on commit c67c156

Please sign in to comment.