Skip to content

Commit

Permalink
set default values for ARGS
Browse files Browse the repository at this point in the history
  • Loading branch information
nbw committed Jan 5, 2025
1 parent 0fcf298 commit ada3548
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/nixpacks/builder/docker/dockerfile_generation.rs
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@ impl DockerfileGenerator for BuildPlan {
// Pull the variables in from docker `--build-arg`
variables
.iter()
.map(|var| var.0.to_string())
.map(|var| format!("{}=\"{}\"", var.0.trim(), var.1.trim()))
.collect::<Vec<_>>()
.join(" "),
// Make the variables available at runtime
Expand Down
6 changes: 3 additions & 3 deletions src/providers/elixir.rs
Original file line number Diff line number Diff line change
Expand Up @@ -33,14 +33,14 @@ impl Provider for ElixirProvider {

// Install Phase
let mut install_phase =
Phase::install(Some("MIX_ENV=prod mix local.hex --force".to_string()));
Phase::install(Some("mix local.hex --force".to_string()));
install_phase.only_include_files = Some(vec![
"config/config.exs".to_string(),
"config/prod.exs".to_string(),
"config/${MIX_ENV}.exs".to_string(),
"mix.exs".to_string(),
"mix.lock".to_string(),
]);
install_phase.add_cmd("MIX_ENV=prod mix local.rebar --force");
install_phase.add_cmd("mix local.rebar --force");
install_phase.add_cmd("mix deps.get --only prod");
plan.add_phase(install_phase);

Expand Down

0 comments on commit ada3548

Please sign in to comment.