Update mbed-os-to-arduino script

[bissonex]

Add support for macOS platform to mbed_os_to_arduino script

[facchinm]

@manchoz wasn't the original version already working on macOS without GNU tools? Maybe only a particular version of the OS?

[manchoz]

Yes, there's been introduced a regression at some point due to a text formatting change which is preventing standard MacOS' sed to work properly.

I will PR to revert ASAP.

[manchoz]

Here's the PR #57

[facchinm]

Hi @bissonex , can you test #57 and report if it works on OSX? It would be better to not need gnu sed as a prerequisite

[bissonex]

Hi @facchinm, I can confirm that #57 is fixing the issue with sed. Please note that the check_tools() function is still not working under macOS 10.15.3. And that a check for tac must also be added. On macOS like on many systems (BSDs, Solaris, AIX, IRIX...), the functionality of GNU tac is available in tail with the -r option.

[manchoz]

I think we can substitute tac with a classic sed one-liner: sed '1!G;h;$!d'.

@bissonex what is not working with check_tools()? I also have macOS 10.15.3 and it's working properly AFAICS. Could you, please, attach a log (running the tool with bash -x)? Thanks!

[bissonex]

I am setting up check_tools() to fail by changing tool name from jq to jq_should_not_found:

check_tools () {
	echo -n "Checking for prerequisites..."
	if not hash jq_should_not_found &>/dev/null ; then
		echo "Please, install jq."
		exit 1
	fi

	if not hash rsync &>/dev/null ; then
		echo "Please, install rsync."
		exit 1
	fi
	echo " done."
}

output from tool:

+ echo -n 'Checking for prerequisites...'
Checking for prerequisites...
+ not hash jq_should_not_found
+ not hash rsync
+ echo ' done.'

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}