Bug 336096

Summary:	dev-libs/libmcal does not respect LDFLAGS
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	Robin Johnson <robbat2>
Status:	RESOLVED FIXED
Severity:	QA	CC:	esigra, mmk
Priority:	High	Flags:	robbat2: Bugday+
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	331933
Attachments:	Build log libmcal-0.7-r6.ebuild libmcal-0.7-flex-2.5.37.patch libmcal-0.7-gcc4.patch libmcal-0.7-fpic.patch

Description Diego Elio Pettenò (RETIRED) gentoo-dev

2010-09-05 14:07:25 UTC

See attached build log.

(Please do not complain about the need for looking at the build log until you can provide an easy way to open bugs with the correct data picked out of a tinderbox log. Thanks.)

Comment 1 Diego Elio Pettenò (RETIRED) gentoo-dev

2010-09-05 14:07:39 UTC

Created attachment 246135 [details]
Build log

Comment 2 Michael Mair-Keimberger (iamnr3) 2013-03-23 16:32:12 UTC

Created attachment 342998 [details]
libmcal-0.7-r6.ebuild

I've updated the ebuild (please review)
* fixes LDFLAGS issue
* fixes bug 312951
* fixes bug 440848 (found a patch for that)
* fixes bug 243924
* updated ebuild to eapi=5

However repoman still complains because econf is called in in src_compile(), but i couldn't find a way to give it in src_configure() since mstore/icap backend has to be build first. What's the correct solution for this?

Comment 3 Michael Mair-Keimberger (iamnr3) 2013-03-23 16:32:59 UTC

Created attachment 343000 [details, diff]
libmcal-0.7-flex-2.5.37.patch

patch for bug 440848

Comment 4 Michael Mair-Keimberger (iamnr3) 2013-03-23 16:33:46 UTC

Created attachment 343002 [details, diff]
libmcal-0.7-gcc4.patch

updated patch to silence QA warnings

Comment 5 Michael Mair-Keimberger (iamnr3) 2013-03-23 16:35:11 UTC

Created attachment 343004 [details, diff]
libmcal-0.7-fpic.patch

updated patch to fix LDFLAGS/CFLAGS/CC/SONAME issues

Comment 6 Robin Johnson archtester

2013-11-03 20:37:09 UTC

InCVS; thanks for the fixes and sorry for missing this package for so long.