Skip to content

Fix warning from jemalloc configure script#7790

Merged
oranagra merged 1 commit intoredis:unstablefrom
yoonghm:redis-7784-b
Sep 21, 2020
Merged

Fix warning from jemalloc configure script#7790
oranagra merged 1 commit intoredis:unstablefrom
yoonghm:redis-7784-b

Conversation

@yoonghm
Copy link
Contributor

@yoonghm yoonghm commented Sep 11, 2020

--enable-cc-silence was already removed from configure.ac file in the newer version of jemalloc library. It is time to update Makefile.

`--enable-cc-silence` was already removed from `configure.ac` file in the newer version of jemalloc library. It is time to update Makefile.
@yoonghm
Copy link
Contributor Author

yoonghm commented Sep 21, 2020

This is also removed in https://github.com/jemalloc/jemalloc/blob/dev/configure.ac and not used already in the jemalloc version within the latest redis code.

@oranagra oranagra changed the title Fix warning in redis/issues/7784 Fix warning from jemalloc configure script Sep 21, 2020
@oranagra oranagra merged commit d3faed8 into redis:unstable Sep 21, 2020
JackieXie168 pushed a commit to JackieXie168/redis that referenced this pull request Nov 4, 2020
jemalloc configure shows this:
    configure: WARNING: unrecognized options: --enable-cc-silence

The changelog of jemalloc 4.0 has:
  - Replace --enable-cc-silence with --disable-cc-silence to suppress spurious
    warnings by default.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants