Re: scripts/build/scan-mf-deps: script to generate MF dependencies (issue 553700043 by address@hidden)

[jonas . hahnfeld]

On 2020/03/17 07:54:13, dak wrote:
>
https://codereview.appspot.com/553700043/diff/567370043/mf/invoke-mf2pt.sh
> File mf/invoke-mf2pt.sh (right):
> 
>
https://codereview.appspot.com/553700043/diff/567370043/mf/invoke-mf2pt.sh#newcode4
> mf/invoke-mf2pt.sh:4: realpath() {
> On 2020/03/17 07:41:25, hahnjo wrote:
> > AFAIK sh syntax would be
> > realpath()
> > (
> >   ...
> > )
> > 
> > Curly braces are not part of POSIX
> Are you sure?  I have never seen that.  What isn't POSIX is bash's
"function"
> keyword, but Han-Wen does not use it here.

Ah pardon me: { } are part of POSIX and create a compound statement. I
confused this with the portable way of scoping variables, which is done
via a subshell enclosed by ( )

https://codereview.appspot.com/553700043/