Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 117256 - cdk has incorrect PIC handling
Summary: cdk has incorrect PIC handling
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Aron Griffis (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-31 01:28 UTC by SpanKY
Modified: 2006-03-24 22:37 UTC (History)
4 users (show)

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description SpanKY gentoo-dev 2005-12-31 01:28:50 UTC
the ebuild does 'append-flags -fPIC' ... this is clearly wrong
Comment 1 solar (RETIRED) gentoo-dev 2005-12-31 06:40:40 UTC
I'm pretty sure it appends -fPIC to CFLAGS due to other packages linking 
to the current stable as a shared object and the current cdk is an foo.a
only. But the next release of cdk is supposed to be a lib.a and lib.so so
things that attempt to link shared will get proper shared behavior.
I think this was done for another bug that was being marked stable that was 
regressing. 
Comment 2 SpanKY gentoo-dev 2005-12-31 13:06:02 UTC
cdk-5 produces a .so and a .a but the ebuild still does 'append-flags -fPIC'
Comment 3 solar (RETIRED) gentoo-dev 2005-12-31 13:26:33 UTC
I think licq is the testcase for removing the extra call to append-flags on x86
Comment 4 Marcelo Goes (RETIRED) gentoo-dev 2006-02-19 14:41:18 UTC
I just tested removing ``append-flags -fPIC`` from cdk-5.0.20051230, licq compiled and ran fine. Is there anything else to test?
Comment 5 Marcelo Goes (RETIRED) gentoo-dev 2006-02-19 14:43:04 UTC
By the way, from cdk's Makefile.in:

.c.os:
@RULE_CC@
        @ECHO_CC@$(CC) $(CFLAGS) $(CPPFLAGS) -c -fPIC $< -o $@

I guess this should cover all shared libs, right?
Comment 6 Aron Griffis (RETIRED) gentoo-dev 2006-03-24 22:37:50 UTC
ok, fixed in cdk-5.0.20060220.ebuild since that's the current testing