Simplify github workflows

.github/workflows/rust.yml

@@ -108,6 +108,7 @@ jobs:
             os: macOS-latest
             features: auto-splitting
             cross: skip
+            install_target: true
 
           - label: macOS x86_64-v3
             target: x86_64-apple-darwin
@@ -116,6 +117,7 @@ jobs:
             os: macOS-latest
             features: auto-splitting
             cross: skip
+            install_target: true
 
     steps:
       - name: Checkout Commit
@@ -124,7 +126,7 @@ jobs:
       - name: Install Rust
         uses: hecrj/setup-rust-action@v2
         with:
-          rust-version: ${{ matrix.toolchain || 'stable' }}
+          rust-version: 'stable'
 
       - name: Install Target
         if: matrix.install_target != ''
@@ -157,20 +159,20 @@ jobs:
         env:
           TARGET: ${{ matrix.target }}
           SKIP_CROSS: ${{ matrix.cross }}
-          IS_DEPLOY: ${{ startsWith(github.ref, 'refs/tags/') && (matrix.release_anyway != '' || !(startsWith(matrix.toolchain, 'nightly') || startsWith(matrix.toolchain, 'beta'))) }}
+          IS_DEPLOY: ${{ startsWith(github.ref, 'refs/tags/') }}
           FEATURES: ${{ matrix.features }}
           RUSTFLAGS: ${{ matrix.rust_flags }}
 
       - name: Prepare Release
-        if: startsWith(github.ref, 'refs/tags/') && matrix.release == ''
+        if: startsWith(github.ref, 'refs/tags/')
         run: bash .github/workflows/before_deploy.sh
         env:
           OS_NAME: ${{ matrix.os }}
           TARGET: ${{ matrix.target }}
           RELEASE_TARGET: ${{ matrix.target_rename || matrix.target }}
 
       - name: Release
-        if: startsWith(github.ref, 'refs/tags/') && matrix.release == ''
+        if: startsWith(github.ref, 'refs/tags/')
         uses: softprops/action-gh-release@v2
         with:
           files: livesplit-one-*.*

src/config.rs

@@ -45,7 +45,7 @@
     #[serde(skip)]
     can_save: bool,
     #[serde(default)]
     history: BTreeMap<Arc<str>, BTreeMap<Arc<str>, BTreeSet<Arc<Path>>>>,
 }
 
 impl Splits {
@@ -149,7 +149,7 @@
         fs::write(CONFIG_PATH.as_path(), buf).ok()
     }
 
     pub fn splits_history(&self) -> &BTreeMap<Arc<str>, BTreeMap<Arc<str>, BTreeSet<Arc<Path>>>> {
         &self.splits.history
     }
 
@@ -248,9 +248,10 @@
     }
 
     pub fn open_splits(
         &mut self,
         shared_timer: &SharedTimer,
         layout_data: &mut LayoutData,
+        #[cfg(feature = "auto-splitting")]
         auto_splitter: &livesplit_core::auto_splitting::Runtime,
         path: PathBuf,
     ) -> Result<()> {
@@ -454,7 +455,7 @@
             .show_titlebar(false)
             .transparent(true)
             .set_always_on_top(true)
     }
 
     #[cfg(feature = "auto-splitting")]
     pub fn maybe_load_auto_splitter(&self, runtime: &livesplit_core::auto_splitting::Runtime, timer: SharedTimer) {
@@ -464,7 +465,7 @@
                 log::error!("Auto Splitter failed to load: {}", e);
             }
         }
     }
 
     #[cfg(feature = "auto-splitting")]
     pub fn maybe_replace_auto_splitter(&self, runtime: &livesplit_core::auto_splitting::Runtime, timer: SharedTimer) {
@@ -484,7 +485,7 @@
     let mut run = Run::new();
     run.push_segment(Segment::new("Time"));
     run
 }
 
 pub fn show_error(error: anyhow::Error) {
     // this MessageDialog is for displaying errors, 

src/timer_form.rs

@@ -351,6 +351,7 @@
                     let result = data.config.borrow_mut().open_splits(
                         &data.timer,
                         &mut data.layout_data.borrow_mut(),
+                        #[cfg(feature = "auto-splitting")]
                         &data.auto_splitter,
                         file_info.path().to_path_buf(),
                     );
@@ -865,7 +866,7 @@
                     data.config.borrow_mut().set_hotkeys(hotkey_config);
                 }
                 data.settings_editor = None;
                 let _ = HOTKEY_SYSTEM.write().unwrap().as_mut().unwrap().activate();
                 return;
             }
         }