doesn't work on windows

[Mavic129]

tar on windows doesn't support symlink
spawn is not available on windows

tested on windows 10

[MAVProxyUser]

https://github.com/MAVProxyUser/P0VsRedHerring/tree/RedHerring4Win will be where this lives.

Current problems:

1. Symlink support on windows.
2. Host file write does not work properly.

[hugheaves]

I got this script working under Windows last night using Cygwin. I changed some of the hard coded paths (i.e. DJI Assistant binary) to be variables, and then configured the variables based on the detected OS.

I've never actually used a private repo on GitHub before. If I fork it and submit a PR, does that make a public copy? Would it be best to just push a branch to this repo containing the changes?

[MAVProxyUser]

I believe you can do either Hugh... I plan to flip the Repo to public midnight tonight if all goes as planned. Just trying to hammer some bugs out with beta testers before the gates open.

[MAVProxyUser]

feel free to attach a .diff file to this post too...

[HFMan]

hugh.. work from a fork. When you are done, you can just do a pull request and this will notify MAVProxyUser he can pull your changes into master.

[MAVProxyUser]

https://github.com/MAVProxyUser/P0VsRedHerring/commits/cygwin-compatibility

Is this your commit @hugheaves ?

[hugheaves]

@MAVProxyUser, yes, that branch contains your script with some conditional logic to handle Cygwin. I'm guessing there's a native (non-Cygwin) ruby for Windows as well, but as the script made use of the typical unix commands, Cygwin seemed like the "path of least resistance".

P.S. Sorry for the late reply ... I just got back into town after a few days "on the road".

[MAVProxyUser]

I need to do some heavy testing, but I believe this patch cleans up windows.
ac04a11

[hugheaves]

Looks good. I think you can pretty much ignore my branch at this point, as I see that you've got the required cygwin binaries in master, (I was running using a full cygwin install, but that's overkill for this use case)