make terraform-fmt.sh work without gsed

[pcrock-thmdo]

not all macOS users have gsed installed, so we need to just do the BSD-flavored sed incantation

UPDATE: or maybe awk gets rid of some cross-platform jank...

❗ needs testing on macOS!

[sbresin]

i am using macOS, but my sed command comes from nix and is the GNU variant.
so this would break for me in this case.

i guess you would need to check something like sed --help | grep GNU to either be a 0 or error out to be sure which one you have.

In this case i wonder though if this should not work for both variants

sed -i -E '/^$/N;/^\n$/D' the lowercase -i works the same on both variants and the capital -E should as well

[pcrock-thmdo]

lowercase -i works the same on both variants

it exists in both variants, but behaves differently in both variants. see https://stackoverflow.com/a/4247319/138757.

i am considering moving to perl then, but you're right, the --help or --version approach might work better than the uname approach.

also... never thought about nix! thanks for adding that perspective @sbresin

[sbresin]

One more thing. I will move to uutils-coreutils soon and ubuntu for example is planning to do as well. So theres even one player more, even though this one ist GNU compatible.
So you would need to detect just the BSD variant and treat it different.

[sbresin]

If you are writing to the file using redirection you could have just used sed without the -i flag. But i am fine with it either way

[syphar]

general note here:

in the future we just have to balance when it's ok to tell people "you need gsed on a mac", compared to how many hoops we want to jump through to make it work with BSD sed

[pcrock-thmdo]

If you are writing to the file using redirection you could have just used sed without the -i flag.

oh yeah. duh. 🙄 🤦