BUG: fix second ami client start

pcdshub / engineering_tools

A repository of scripts, configuration useful for the PCDS team

Other

4 stars 26 forks source link

BUG: fix second ami client start #218

Closed vespos closed 2 weeks ago

vespos commented 3 weeks ago

Description

startami fails when trying to get a second client

Motivation and Context

Shell check forcing some changes that do not work. Revert and ignore shell check.

How Has This Been Tested?

xpp-control as xppopr

Where Has This Been Documented?

N/A

vespos commented 3 weeks ago

@ZLLentz shellcheck seems to be an issue here. I feel we have only recently been using it, do you know why it is suggesting something that does not work here?

ZLLentz commented 3 weeks ago

I'll look into this weird command splitting thing in depth, if this works as-is merge it and proceed forward

ZLLentz commented 3 weeks ago

Linking PR to slack thread so I can find slack thread later: https://slac.slack.com/archives/C2PNS4Q74/p1730337117129199