Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 581512 - dev-libs/librdkafka ignores CC/CXX
Summary: dev-libs/librdkafka ignores CC/CXX
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Thomas Deutschmann (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-04-29 06:48 UTC by Jeroen Roovers (RETIRED)
Modified: 2016-05-10 12:42 UTC (History)
2 users (show)

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


Attachments
Proposed patch (bugfix_581512.diff,669 bytes, patch)
2016-05-08 16:53 UTC, Thomas Deutschmann (RETIRED)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jeroen Roovers (RETIRED) gentoo-dev 2016-04-29 06:48:00 UTC
src_configure():
  CC                       gcc
  CXX                      g++
src_compile/test():
>>> Compiling source in /var/tmp/portage/dev-libs/librdkafka-0.9.0.99/work/librdkafka-0.9.0.99 ...
make -j3
make[1]: Entering directory '/var/tmp/portage/dev-libs/librdkafka-0.9.0.99/work/librdkafka-0.9.0.99/src'
gcc -MD -MP -g -O2 -fPIC -Wall -Wsign-compare -Wfloat-equal -Wpointer-arith -g -O2 -fPIC -Wall -Wsign-compare -Wfloat-equal -Wpointer-arith -mschedule=8000 -ma
rch=2.0 -ggdb -Wall -O2 -pipe -Wno-comment -mschedule=8000 -march=2.0 -ggdb -Wall -O2 -pipe -Wno-comment       -mschedule=8000 -march=2.0 -ggdb -Wall -O2 -pipe
 -Wno-comment       -mschedule=8000 -march=2.0 -ggdb -Wall -O2 -pipe -Wno-comment       -c rdkafka.c -o rdkafka.o

and so on.
Comment 1 Thomas Deutschmann (RETIRED) gentoo-dev 2016-05-03 11:51:02 UTC
Thank you for the report!

The following fix is scheduled for getting merged:

https://github.com/gentoo/gentoo/compare/master...Whissi:bug_581512?expand=1


--- /usr/portage/dev-libs/librdkafka/librdkafka-0.9.0.99.ebuild 2016-05-02 22:27:00.000000000 +0200
+++ /usr/local/portage/dev-libs/librdkafka/librdkafka-0.9.0.99-r1.ebuild        2016-05-03 13:27:36.647132709 +0200
@@ -4,6 +4,8 @@

 EAPI=6

+inherit toolchain-funcs
+
 DESCRIPTION="Apache Kafka C/C++ client library"
 HOMEPAGE="https://github.com/edenhill/librdkafka"

@@ -38,6 +40,8 @@
 "

 src_configure() {
+       tc-export CC CXX
+
        local myeconf=(
                --no-cache
                --no-download
Comment 2 Thomas Deutschmann (RETIRED) gentoo-dev 2016-05-04 14:10:57 UTC
Jeroen, we are unsure if we have fully understood your bug report.

Was it about that your are reporting the package ignores CC/CXX, i.e.

> # CC=foo emerge dev-libs/librdkafka

didn't used "foo" (which I cannot reproduce)

or that it was directly calling gcc instead of something like x86_64-pc-linux-gnu-gcc (which the patch should fix, well we could also export LD, NM, OBJDUMP, STRIP...)? Or something else?
Comment 3 Jeroen Roovers (RETIRED) gentoo-dev 2016-05-04 23:19:44 UTC
The build system ignores Gentoo's CC/CXX.
Comment 4 Thomas Deutschmann (RETIRED) gentoo-dev 2016-05-08 16:53:01 UTC
Created attachment 433678 [details, diff]
Proposed patch

Diff against current tree; Don't forget to revbump.
Comment 5 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2016-05-10 12:42:41 UTC
commit f318caf29ae5bc7722447481538875bc48aad4e5
Author: Thomas Deutschmann <whissi@whissi.de>
Commit: Lars Wendler <polynomial-c@gentoo.org>

    dev-libs/librdkafka: Respect CC/CXX variables (bug #581512).
    
    Package-Manager: portage-2.2.28
    Signed-off-by: Lars Wendler <polynomial-c@gentoo.org>