redis: Update to support macOS 11

[stuartcarnie]

• Have you followed the guidelines for contributing?
• Have you checked that there aren't other open pull requests for the same formula update/change?
• Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
• Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
• Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

---

redis will fail on arm64 due to different ucontext_t structs, which the approved pull request to the redis project resolves

[stuartcarnie]

👋 @claui – I've pushed up your suggested feedback; thanks for the review!

[SMillerDev]

Thanks @stuartcarnie ! Without contributions like yours it'd be impossible to keep homebrew going with the high standards that users have come to expect from the project. You can feel good knowing that you've made the world a tiny bit better for homebrew users around the world! 👍 🎉

[claui]

All set!

[jclusso]

Does this patch resolve a separate issue from this? I'm getting this error.

Already downloaded: /Users/jarrett/Library/Caches/Homebrew/downloads/911a7607e9ed1e64aa0eb93186736f996adbe830e11164c5eecddc0e6c2d9a05--redis-6.0.9.tar.gz
==> make install PREFIX=/opt/homebrew/Cellar/redis/6.0.9 CC=clang BUILD_TLS=yes
Last 15 lines from /Users/jarrett/Library/Logs/Homebrew/redis/01.make:
  "_SSL_set_verify", referenced from:
      _connCreateAcceptedTLS in tls.o
  "_SSL_write", referenced from:
      _connTLSWrite in tls.o
      _connTLSSyncWrite in tls.o
      _redisSSLWrite in libhiredis_ssl.a(ssl.o)
  "_TLS_client_method", referenced from:
      _redisSecureConnection in libhiredis_ssl.a(ssl.o)
  "_TLS_method", referenced from:
      _tlsConfigure in tls.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make[1]: *** [redis-server] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [install] Error 2```

[koriym]

I got a same error.

[thomasdarde]

Thanks for the patch !

I have brew up to date. I'm in iTerm, launched with rosetta enabled.
brew reinstall redis -s

Redis is not able to run via launchd and this fails :
redis-server /usr/local/etc/redis.conf

It works with sudo however :
sudo redis-server /usr/local/etc/redis.conf

Also see: redis/redis#8062

[gabrielcechet]

Thanks for the patch !

I have brew up to date. I'm in iTerm, launched with rosetta enabled.
brew reinstall redis -s

Redis is not able to run via launchd and this fails :
redis-server /usr/local/etc/redis.conf

It works with sudo however :
sudo redis-server /usr/local/etc/redis.conf

Also see: redis/redis#8062

sudo redis-server /usr/local/etc/redis.conf

is it possible to put this setting on the brew automatically?

[jclusso]

I was wondering the same thing. That would at least be a temporary fix maybe.

…

On Wed, Dec 2, 2020 at 5:16 PM Gabriel Cechet ***@***.***> wrote: Thanks for the patch ! I have brew up to date. I'm in iTerm, launched with rosetta enabled. brew reinstall redis -s Redis is not able to run via launchd and this fails : redis-server /usr/local/etc/redis.conf It works with sudo however : sudo redis-server /usr/local/etc/redis.conf Also see: redis/redis#8062 <redis/redis#8062> sudo redis-server /usr/local/etc/redis.conf is it possible to put this setting on the brew automatically? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#57664 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAEX7NF6PLFDK4QXF7TBETLSS24DTANCNFSM4OUSX5UA> .

-- Regards, Jarrett Lusso