Neil Lathwood
Neil Lathwood
@MajesticFalcon I'd be happy to review. What's the general overview of why this PR was submitted in the first place?
@ytti so is this something you'd be happy to merge? Or is it something you'd want as an entire rewrite?
@jackson-tim Any feedback on the requested changes for this?
@STetzel Can you take a look at the failed tests please. You will also need to add this model to https://github.com/ytti/oxidized/blob/master/docs/Supported-OS-Types.md and also update the changelog as per the top...
@STetzel Can you add this model to the docs as per my last comment as well please?
Yeah it's never worked as far as I know. Most likely a simple fix but I've never taken a look.
Just change `copy run start` for `write mem`? Alternatively have you tried to add a newline at the end of `copy run start\n`
What user do you run oxidized as? Have you tried running oxs as the same user?
You should show what you are sending across.
I can confirm the changes proposed here work absolutely fine.