Bug 331973

Summary:	app-crypt/mcrypt-2.6.8 fails to on OSX prefix due to inclusion of <malloc.h>
Product:	Gentoo Linux	Reporter:	Mark Kowarsky (RETIRED) <mark_alec>
Component:	New packages	Assignee:	Robin Johnson <robbat2>
Status:	RESOLVED FIXED
Severity:	normal	CC:	prefix
Priority:	High
Version:	unspecified
Hardware:	x86
OS:	OS X
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	315803
Attachments:	Patch to correct malloc.h -> stdlib.h

Description Mark Kowarsky (RETIRED) gentoo-dev

2010-08-10 03:58:20 UTC

In attempting to install mcrypt on a prefixed Gentoo system, it failed due to malloc.h not being found. malloc.h is not included in OSX (and I believe other BSDs) and it should not be used (as it is not a POSIX header). Instead stdlib.h should be used.

Attached is a patch to fix the package.

Reproducible: Always

Steps to Reproduce:

Comment 1 Mark Kowarsky (RETIRED) gentoo-dev

2010-08-10 03:59:56 UTC

Created attachment 242191 [details, diff]
Patch to correct malloc.h -> stdlib.h

Added patch to fix src/rfc2440.c in mcrypt.

Comment 2 Jeremy Olexa (darkside) (RETIRED) archtester

2010-08-10 15:22:17 UTC

If work is being done here for Gentoo Prefix stuff.. then let's move libmcrypt to gx86 too :)

for libmcrypt: s/elibtoolize/eautoreconf/ and add keywords: http://overlays.gentoo.org/proj/alt/browser/trunk/prefix-overlay/dev-libs/libmcrypt/libmcrypt-2.5.8-r1.ebuild
for mcrypt: No changes but this patch addition. add keywords

Neither need EAPI bumps, unless desired for ebuild writing simplicity.

I can do it if you would like, revbumps?

Comment 3 Robin Johnson archtester

2010-08-10 15:31:00 UTC

darkside: +1 on commit.

Comment 4 Jeremy Olexa (darkside) (RETIRED) archtester

2010-08-10 18:01:05 UTC

done, thx Mark