stat -c does not work for osx 10.10 after upgrade to 1.17.0

[markwu]

See 1fe60b0#commitcomment-8636924

stat -c %d . does not work for osx 10.10. But stat -f %d . works.

I am not sure it is the right fix or not.

[markwu]

Original code will cause the following errors:

stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]
-bash: [: too many arguments
stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]
-bash: [: too many arguments
stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]

[marcioAlmada]

@markwu what version of stat are you using? I'm not using Yosemite yet, heard about too much bugs. @c9s, do you have this issue?

[sam452]

On a new install of yosemite 10.1 I get the following after pasting in
[[ -e ~/.phpbrew/bashrc ]] && source ~/.phpbrew/bashrc

I get this on a new terminal. So +1 to the above, sam

stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]
-bash: [: too many arguments
stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]
-bash: [: too many arguments
stat: illegal option -- c
usage: stat [-FlLnqrsx] [-f format] [-t timefmt] [file ...]

[markwu]

I tihnk it is BSD stat ... come with yosemete 10.10, not GNU stat. I didn't see any version information from man stat.

[markwu]

You can see detail bsd stat manual here http://www.freebsd.org/cgi/man.cgi?query=stat&sektion=1

[racklin]

@markwu @marcioAlmada @c9s
Hey guys, we are talking about shell script?

Why not just replace

curr_fs=$(stat -c %d .)

to

curr_fs=$(stat -c %d . 2>/dev/null)
if [ $? -ne 0 ]; then 
    curr_fs=$(stat -f %d . 2>/dev/null)
fi

That works both GNU and workaround for BSD style stat .

Should i sent a PR for that?

[sam452]

I'm still coming up with this output when I open a new terminal. I have run phpbrew self-update. Am I misunderstanding something?

[marcioAlmada]

@sam452 we had no releases including this merge yet. Soon we'll have a release with the new features and this fix.