Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 333811 - net-misc/ipsorcery does not respect LDFLAGS
Summary: net-misc/ipsorcery 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: Daniel Black (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: ldflags
  Show dependency tree
 
Reported: 2010-08-21 15:28 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2010-08-21 20:45 UTC (History)
0 users

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


Attachments
Build log (ipsorcery-2.0.9:20100821-152028.log,1.85 KB, text/plain)
2010-08-21 15:35 UTC, Diego Elio Pettenò (RETIRED)
Details
Patch to ipsorcery-2.0.9.ebuild to address various QA issues (ipsorcery-2.0.9.ebuild.patch,1.15 KB, patch)
2010-08-21 20:43 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-21 15:28:58 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-21 15:35:02 UTC
Created attachment 243877 [details]
Build log
Comment 2 Markos Chandras (RETIRED) gentoo-dev 2010-08-21 19:13:43 UTC
+*ipsorcery-2.0.9-r1 (21 Aug 2010)
+
+  21 Aug 2010; Markos Chandras <hwoarang@gentoo.org>
+  +ipsorcery-2.0.9-r1.ebuild:
+  Respect CC,CFLAGS,LDFLAGS. Bug #333811. Thanks to Diego E. 'Flameeyes'
+  Pettenò <flameeyes@gentoo.org> for reporting. Keyworded for amd64
+
Comment 3 Kevin Pyle 2010-08-21 20:43:59 UTC
Created attachment 243957 [details, diff]
Patch to ipsorcery-2.0.9.ebuild to address various QA issues

This package had several problems:

- CC=cc
- Ignored CPPFLAGS (minor)
- Ignored LDFLAGS
- Non-GTK build used make, rather than emake.
- Use of dosbin without ||die

This patch addresses these problems by:
- Deleting the CC= line, inheriting toolchain-funcs, and exporting CC
- Adding CPPFLAGS and LDFLAGS alongside the CFLAGS already added by the ebuild
- Switching to emake
- Adding ||die to dosbin calls

Also, of minor interest, the package is not keyworded for amd64, but it built correctly for me on a stable amd64 system.  However, it has no self tests, and I am not in a position to test using the program in a live environment.
Comment 4 Kevin Pyle 2010-08-21 20:45:39 UTC
Weird.  I did not get a collision alert when I posted my patch, and comment #2 was not visible when I started work on this.  Oh well.