.tmux: sed: invalid option -- E

[blackb1rd]

$ sed --version
GNU sed version 4.1.5
$ cat /etc/redhat-release
CentOS release 5.8 (Final)

$ cut -c3- ~/.tmux.conf | sh -sx _apply_configuration
...
$ sed: invalid option -- E
...

when using the old versions sed < 4.2, sed is not supported -E yet.

According to http://blog.dmitryleskov.com/small-hacks/mysterious-gnu-sed-option-e/ , it seems like the option -r == -E

There was similar issue reported:
https://github.com/fisherman/getopts/issues/12
https://github.com/rvm/rvm/issues/1410
https://github.com/c9/core/issues/346

[gpakosz]

Thanks, will look into it

[gpakosz]

Hey @blackb1rd 👋

Honestly, I don't know if it's worth the hassle supporting CentOS < 6 by now 🤷‍♂️

[blackb1rd]

Hi @gpakosz :wave:,

I think, it may not worth to support CentOS < 6 but I'm not sure if it's worth to make the compatibility with old versions sed < 4.2.

[gpakosz]

macOS stock's BSD sed doesn't support -r, so... supporting both will add more tests everywhere

[gpakosz]

Are there many OS distributions that currently use GNU sed < 4.2? In any case those won't have a recent tmux version either and I'm about to raise the minimum required version to 2.4

[blackb1rd]

Are there many OS distributions that currently use GNU sed < 4.2?

I think, not many OS distributions is using GNU sed < 4.2 and there is a Red Hat Enterprise Linux 6.

[gpakosz]

Hey @blackb1rd 👋

Can you please try the gh-150 branch if you're still running OS distributions for which GNU sed < 4.2 by chance?

[blackb1rd]

Hi @gpakosz
I tried the gh-150 branch and it works fine. :thumbsup:

[gpakosz]

So on CentOS 5.8?

[blackb1rd]

Yes.

[gpakosz]

@blackb1rd awesome, thanks testing 🙏

I'll merge the branch soon.