Merge pull request #77 from warptools/ferk-cmd-fixes

fix the --cmd flag for ferk to not segfault on custom step names
warptools · Jul 6, 2023 · 92d2374 · 92d2374
2 parents d0d683a + 0aec67a
commit 92d2374
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/app/enter/ferk_cli.go b/app/enter/ferk_cli.go
@@ -103,7 +103,7 @@ func cmdFerk(c *cli.Context) error {
 
  // set command to execute
  if c.String("cmd") != "" {
- plot.Steps.Values["ferk"].Protoformula.Action = wfapi.Action{
+ plot.Steps.Values[wfapi.StepName(stepName)].Protoformula.Action = wfapi.Action{
  Exec: &wfapi.Action_Exec{
  Command: strings.Split(c.String("cmd"), " "),
  },
@@ -116,8 +116,8 @@ func cmdFerk(c *cli.Context) error {
  port := wfapi.SandboxPort{
  SandboxPath: &sandboxPath,
  }
- plot.Steps.Values["ferk"].Protoformula.Inputs.Keys = append(plot.Steps.Values["ferk"].Protoformula.Inputs.Keys, port)
- plot.Steps.Values["ferk"].Protoformula.Inputs.Values[port] = wfapi.PlotInput{
+ plot.Steps.Values[wfapi.StepName(stepName)].Protoformula.Inputs.Keys = append(plot.Steps.Values[wfapi.StepName(stepName)].Protoformula.Inputs.Keys, port)
+ plot.Steps.Values[wfapi.StepName(stepName)].Protoformula.Inputs.Values[port] = wfapi.PlotInput{
  PlotInputSimple: &wfapi.PlotInputSimple{
  Mount: &wfapi.Mount{
  Mode: "rw",