Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 332729 - app-editors/easyedit does not respect LDFLAGS
Summary: app-editors/easyedit does not respect LDFLAGS
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High QA (vote)
Assignee: Akinori Hattori
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: ldflags
  Show dependency tree
 
Reported: 2010-08-14 12:04 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2011-05-08 12:21 UTC (History)
1 user (show)

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


Attachments
Build log (easyedit-1.4.7:20100814-034241.log,2.50 KB, text/plain)
2010-08-14 12:04 UTC, Diego Elio Pettenò (RETIRED)
Details
Patch to easyedit-1.4.7.ebuild so generated Makefile includes $LDFLAGS (easyedit-1.4.7.ebuild.patch,353 bytes, patch)
2010-08-14 17:29 UTC, Kevin Pyle
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
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.