Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 378085 - Version bump dev-db/redis-2.4.4
Summary: Version bump dev-db/redis-2.4.4
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal enhancement (vote)
Assignee: Dirkjan Ochtman (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-08-07 13:53 UTC by Dirkjan Ochtman (RETIRED)
Modified: 2011-11-30 12:14 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
redis-2.4.1.ebuild (redis-2.4.1.ebuild,3.31 KB, text/plain)
2011-10-19 19:33 UTC, Johan Bergström
Details
redis-2.4.1-shared.patch (redis-2.4.1-shared.patch,708 bytes, patch)
2011-10-19 19:33 UTC, Johan Bergström
Details | Diff
Build log with linenoise failure (build.log,7.01 KB, text/plain)
2011-11-29 18:27 UTC, Dirkjan Ochtman (RETIRED)
Details
redis-2.4.4.ebuild (redis-2.4.4.ebuild,3.48 KB, text/plain)
2011-11-29 19:08 UTC, Johan Bergström
Details
redis-2.4.4-tcmalloc.patch (redis-2.4.4-tcmalloc.patch,442 bytes, patch)
2011-11-29 19:08 UTC, Johan Bergström
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Dirkjan Ochtman (RETIRED) gentoo-dev 2011-08-07 13:53:43 UTC
It hasn't been released yet, but this will help to keep track of things.

 * fix src/Makefile so we don't use bundled jemalloc (we have the same version in tree)
 * possibly build hiredis (the c api) as a shared extension
 * fix static/shared linking for above
 * USE: lua for libraries?

 * SRC_URI can nowadays stick to ${P}
 * remove einfo
Comment 1 remoe 2011-10-19 19:03:40 UTC
Hi

Version 2.4.1 was released:

http://redis.io/

I hope someone can update it :)

Thanks
Comment 2 Johan Bergström 2011-10-19 19:33:26 UTC
Created attachment 290267 [details]
redis-2.4.1.ebuild

Fixes most stuff mentioned above, besides lua part since it didn't land in 2.4. Also remove lots of docs since they aren't bundled anymore (?). Most importantly, jemalloc builds against shared libraries. Our main concern forward will be that upstream has ruled against using shared libraries. Hopefully it will Just Work.

Upstream info here: https://github.com/antirez/redis/pull/137
Comment 3 Johan Bergström 2011-10-19 19:33:55 UTC
Created attachment 290269 [details, diff]
redis-2.4.1-shared.patch
Comment 4 remoe 2011-10-19 20:41:56 UTC
Thanks,

I've done a quick test and it works on my system (~amd64).
Comment 5 Priit Laes (IRC: plaes) 2011-10-25 09:04:21 UTC
When bumping, please change HOMEPAGE to "http://redis.io"
Comment 6 Johan Bergström 2011-11-21 18:08:47 UTC
2.4.3 is now out. Renaming attachments works for me.
Comment 7 Johan Bergström 2011-11-21 18:13:03 UTC
I'm thinking we should add USE=jemalloc, enabled by default and make sure we negate tcmalloc when jemalloc is enabled (and the other way around).  The "fallback" memory allocator would then be libc malloc.
Comment 8 Dirkjan Ochtman (RETIRED) gentoo-dev 2011-11-24 13:02:23 UTC
I tried the attached ebuild and patch with 2.4.3, and it failed:

MAKE linenoise
make[2]: Entering directory `/var/tmp/portage/dev-db/redis-2.4.3/work/redis-2.4.3/deps/linenoise'
x86_64-pc-linux-gnu-gcc  -c -Wall -W -Os -g linenoise.c
x86_64-pc-linux-gnu-gcc  -c -Wall -W -Os -g example.c
x86_64-pc-linux-gnu-gcc  -Wall -W -Os -g -o linenoise_example linenoise.o example.o
make[2]: Leaving directory `/var/tmp/portage/dev-db/redis-2.4.3/work/redis-2.4.3/deps/linenoise'
make[1]: Leaving directory `/var/tmp/portage/dev-db/redis-2.4.3/work/redis-2.4.3/src'
make: *** [all] Error 2
emake failed
 * ERROR: dev-db/redis-2.4.3 failed (compile phase):
 *   emake failed
 *
 * Call stack:
 *          ebuild.sh, line   84:  Called src_compile
 *        environment, line 3065:  Called _eapi2_src_compile
 *   phase-helpers.sh, line  570:  Called die
 * The specific snippet of code:
 *              emake || die "emake failed"
 *
 * If you need support, post the output of 'emerge --info =dev-db/redis-2.4.3',
 * the complete build log and the output of 'emerge -pqv =dev-db/redis-2.4.3'.
 * The complete build log is located at '/var/tmp/portage/dev-db/redis-2.4.3/temp/build.log'.
 * The ebuild environment file is located at '/var/tmp/portage/dev-db/redis-2.4.3/temp/environment'.
 * S: '/var/tmp/portage/dev-db/redis-2.4.3/work/redis-2.4.3'
Comment 9 Luca Barbato gentoo-dev 2011-11-26 09:47:20 UTC
The patch works for me now...
Comment 10 Johan Bergström 2011-11-26 09:55:25 UTC
djc: how did you build it (what use flags), and can i have the entire build log?

thanks!
Comment 11 Johan Bergström 2011-11-29 17:58:38 UTC
2.4.4 yesterday. Patch still applies and works nicely at my place. GCC 4.5.3 fwiw.
Comment 12 Dirkjan Ochtman (RETIRED) gentoo-dev 2011-11-29 18:27:04 UTC
Created attachment 294221 [details]
Build log with linenoise failure
Comment 13 Johan Bergström 2011-11-29 19:08:26 UTC
Created attachment 294227 [details]
redis-2.4.4.ebuild

So. Here's a new ebuild. Changes:
 - bump to eapi=4
 - add jemalloc use-flag (enabled by default, upstream choice)
   USE=jemalloc and USE=tcmalloc cannot be used simultaneously,
   USE="-jemalloc -tcmalloc" will choose the standard libc malloc.
 - remove Changelog docs, no longer included
 - add patch for an upstream bug related to tcmalloc build

Pretty please review this. I have most likely missed stuff related to the eapi 4-bump.
Comment 14 Johan Bergström 2011-11-29 19:08:51 UTC
Created attachment 294229 [details, diff]
redis-2.4.4-tcmalloc.patch

Above mentioned patch
Comment 15 Dirkjan Ochtman (RETIRED) gentoo-dev 2011-11-30 12:14:22 UTC
I've put 2.4.4 in the tree, thanks!