Bug 332729

Summary:	app-editors/easyedit does not respect LDFLAGS
Product:	Gentoo Linux	Reporter:	Diego Elio Pettenò (RETIRED) <flameeyes>
Component:	New packages	Assignee:	Akinori Hattori <hattya>
Status:	RESOLVED FIXED
Severity:	QA	CC:	esigra
Priority:	High
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	331933
Attachments:	Build log Patch to easyedit-1.4.7.ebuild so generated Makefile includes $LDFLAGS

Description Diego Elio Pettenò (RETIRED) gentoo-dev

2010-08-14 12:04:17 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-08-14 12:04:32 UTC

Created attachment 242943 [details]
Build log

Comment 2 Kevin Pyle 2010-08-14 17:29:23 UTC

Created attachment 242965 [details, diff]
Patch to easyedit-1.4.7.ebuild so generated Makefile includes $LDFLAGS

Upstream ships a custom script 'create.make' that appears to be an attempt at writing a configure script by hand.  Unfortunately, create.make also generates the Makefile and does so without any reference to LDFLAGS.  The attached patch amends easyedit-1.4.7.ebuild so that the expression that converts cc to $(CC) also inserts a reference to $(LDFLAGS).  With this patch, I see my LDFLAGS used in the compilation of the output file.

Comment 3 Akinori Hattori gentoo-dev

2011-05-07 16:38:40 UTC

Fixed in CVS.

Comment 4 Ulrich Müller gentoo-dev

2011-05-07 21:25:26 UTC

When doing a package move, could you please update reverse dependencies too?
I've fixed virtual/editor and app-editors/ersatz-emacs (blocker).

Comment 5 Akinori Hattori gentoo-dev

2011-05-08 12:21:46 UTC

Sorry for my careless mistake and thank you for fixing that.