Skip to content

Commit

Permalink
Remove use of absolute for local_app_dir
Browse files Browse the repository at this point in the history
Signed-off-by: Ryan Levick <[email protected]>
  • Loading branch information
rylev committed Aug 27, 2024
1 parent c7e310d commit 822a454
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
6 changes: 1 addition & 5 deletions crates/trigger/src/cli.rs
Original file line number Diff line number Diff line change
Expand Up @@ -309,11 +309,7 @@ impl<T: Trigger> TriggerAppBuilder<T> {
self.trigger.add_to_linker(core_engine_builder.linker())?;

let runtime_config_path = options.runtime_config_file;
let local_app_dir = options
.local_app_dir
.map(std::path::absolute)
.transpose()
.context("failed to resolve local app directory path to an absolute path")?;
let local_app_dir = options.local_app_dir.map(PathBuf::from);
let state_dir = match options.state_dir {
// Make sure `--state-dir=""` unsets the state dir
Some("") => UserProvidedPath::Unset,
Expand Down
2 changes: 1 addition & 1 deletion crates/trigger/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ use spin_factors_executor::{FactorsExecutorApp, FactorsInstanceBuilder};

pub use spin_app::App;

/// Type alias for a [`FactorsConfiguredApp`] specialized to a [`Trigger`].
/// Type alias for a [`FactorsExecutorApp`] specialized to a [`Trigger`].
pub type TriggerApp<T> = FactorsExecutorApp<TriggerFactors, <T as Trigger>::InstanceState>;

pub type TriggerInstanceBuilder<'a, T> =
Expand Down

0 comments on commit 822a454

Please sign in to comment.