Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 334759 - sys-block/smp_utils does not respect LDFLAGS
Summary: sys-block/smp_utils 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: Gentoo's Team for Core System packages
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: ldflags
  Show dependency tree
 
Reported: 2010-08-27 09:01 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2011-08-30 09:32 UTC (History)
1 user (show)

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


Attachments
Build log (smp_utils-0.94:20100827-035108.log,15.24 KB, text/plain)
2010-08-27 09:01 UTC, Diego Elio Pettenò (RETIRED)
Details
Patch to smp_utils-0.94.ebuild to respect LDFLAGS, and make other improvements (smp_utils-0.94.ebuild.patch,1.80 KB, patch)
2010-09-04 22:17 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-27 09:01:33 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-27 09:01:45 UTC
Created attachment 244879 [details]
Build log
Comment 2 Kevin Pyle 2010-09-04 22:17:26 UTC
Created attachment 246050 [details, diff]
Patch to smp_utils-0.94.ebuild to respect LDFLAGS, and make other improvements

This patch makes the following changes:

- Switch to a single sed that hits all the Makefiles, instead of running sed individually on each one
- Delete CFLAGS and LDFLAGS assignments so that user values are respected.  Upstream had assigned LDFLAGS to be empty, triggering the original bug report.
- Add $(CFLAGS) to link lines, to handle any cases where -f options need to be specified at both compile and link phases.
- Switch to builtin rule for compiling .o files.  This required moving upstream -I and -D lines into $CPPFLAGS.
- Rework recursive descent so that libsmp.a is not rebuilt during 'make install', since rebuilding it caused Make to then relink one of the outputs.
- Use CHOST-qualified values for CC and AR.
- Use emake for install, since it works fine with MAKEOPTS=-j.
Comment 3 Tim Harder gentoo-dev 2011-08-30 09:32:23 UTC
Fixed in 0.96 in CVS.