include the OS in the app name

marph91 · Apr 27, 2024 · d0a34df · d0a34df
1 parent 496055f
commit d0a34df
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -28,8 +28,8 @@ jobs:
         run: python test/test_app.py
       - name: Smoke test
         run: |
-          ./dist/jimmy .cache/arbitrary_folder/arbitrary_folder --dry-run
-          ./dist/jimmy .cache/obsidian/obsidian_vault --app obsidian --dry-run
+          ./dist/jimmy* .cache/arbitrary_folder/arbitrary_folder --dry-run
+          ./dist/jimmy* .cache/obsidian/obsidian_vault --app obsidian --dry-run
       - name: Release
         uses: softprops/action-gh-release@v2
         # release only if there is a release tag

diff --git a/jimmy.spec b/jimmy.spec
@@ -22,6 +22,11 @@ def list_python_files(folder):
 hiddenimports = list_python_files(Path("src/apps"))
 
 
+# Generate the executable name based on OS.
+import platform
+executable_name = f"jimmy-{platform.system().lower()}"
+
+
 a = Analysis(
     ['src/jimmy.py'],
     pathex=[],
@@ -42,7 +47,7 @@ exe = EXE(
     a.binaries,
     a.datas,
     [],
-    name='jimmy',
+    name=executable_name,
     debug=False,
     bootloader_ignore_signals=False,
     strip=False,