Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 349784 - net-libs/nativebiginteger TFU ebuild
Summary: net-libs/nativebiginteger TFU ebuild
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High QA (vote)
Assignee: Thomas Sachau
Depends on:
Blocks: as-needed ldflags
  Show dependency tree
Reported: 2010-12-26 15:29 UTC by Diego Elio Pettenò (RETIRED)
Modified: 2010-12-28 12:35 UTC (History)
3 users (show)

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

Diff from ebuild in tree (nativebiginteger.ebuild.diff,432 bytes, text/plain)
2010-12-27 14:51 UTC, Dane Smith (RETIRED)
New patch for some magic assembly (nativebiginteger-0.6.4-asmfix.patch,568 bytes, text/plain)
2010-12-27 14:52 UTC, Dane Smith (RETIRED)
as-needed fix (nativebiginteger-0.6.4-as-needed.patch,412 bytes, text/plain)
2010-12-27 14:56 UTC, Dane Smith (RETIRED)
Improved assembly fix. (nativebiginteger-0.6.4-asmfix.patch,579 bytes, text/plain)
2010-12-27 15:01 UTC, Dane Smith (RETIRED)

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-12-26 15:29:27 UTC
Let's see..

 - on x86 it builds a TEXTREl object, filtering PIE; why at all?
 - it fails to properly account for ldflags;
 - it filters --as-needed rather than PASSING THE LIBRARIES IN THE CORRECT ORDER;

I'm tempted to kill it with fire, let's instead consider fixing it properly, shall we?
Comment 1 Dane Smith (RETIRED) gentoo-dev 2010-12-27 14:51:20 UTC
Created attachment 258174 [details]
Diff from ebuild in tree

Diff from the build in the tree. I tested this (with the patch to follow) on x86. _PLEASE_ test on amd64. I had I to use some assembly magic to make it play nice on x86. It should not affect the amd64 version, however, I don't trust any code I write.

New patch to follow.
Comment 2 Dane Smith (RETIRED) gentoo-dev 2010-12-27 14:52:51 UTC
Created attachment 258175 [details]
New patch for some magic assembly

My assembly is decent, but it's by no means the best. I'm fairly sure this fixes things. I know it compiles on x86. Hope it helps.
Comment 3 Dane Smith (RETIRED) gentoo-dev 2010-12-27 14:56:14 UTC
Created attachment 258177 [details]
as-needed fix

Pretty sure you already have this under control Tommy. But this will add the missing library into the mix.
Comment 4 Dane Smith (RETIRED) gentoo-dev 2010-12-27 15:01:15 UTC
Created attachment 258178 [details]
Improved assembly fix.

Changed #else to #elseif __i386__. Thanks for the catch Diego.
Comment 5 Thomas Sachau gentoo-dev 2010-12-28 12:35:27 UTC
Fixed with 0.6.4-r3