Summary: | net-analyzer/ntopng-2.4 - Ignores CC/CXX and calls /usr/bin/clang++ directly | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Thomas Deutschmann (RETIRED) <whissi> |
Component: | Current packages | Assignee: | Sławek Lis (RETIRED) <slis> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | cwprogram, slawomir.nizio, slis |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Bug Depends on: | |||
Bug Blocks: | 243502 | ||
Attachments: |
build.log
Properly utilize user CXX and CXXFLAGS |
Description
Thomas Deutschmann (RETIRED)
2016-11-19 20:10:13 UTC
Created attachment 453806 [details, diff]
Properly utilize user CXX and CXXFLAGS
So, it does actually have something for setting the compiler it's just in a GPP variable. I adjusted the build system to use CXX instead, and also made sure CXX compiles were actually using user CFLAGS. Confirmed compilation with both clang and gcc
Thank you for your support. I've pushed your patch. The make system is still calling g++ directly, see the tracker. I am suggesting the following changes: diff --git a/net-analyzer/ntopng/ntopng-2.4.ebuild b/net-analyzer/ntopng/ntopng-2.4.ebuild index f6c9071b71..91b680bec4 100644 --- a/net-analyzer/ntopng/ntopng-2.4.ebuild +++ b/net-analyzer/ntopng/ntopng-2.4.ebuild @@ -3,7 +3,7 @@ # $Id$ EAPI=5 -inherit autotools eutils user +inherit autotools eutils user toolchain-funcs DESCRIPTION="Network traffic analyzer with web interface" HOMEPAGE="http://www.ntop.org/" @@ -20,7 +20,7 @@ DEPEND="dev-db/sqlite:3 dev-libs/geoip dev-libs/glib:2 dev-libs/libxml2 - net-analyzer/rrdtool + net-analyzer/rrdtool:= net-libs/libpcap net-misc/curl virtual/libmysqlclient" @@ -42,6 +42,7 @@ src_prepare() { } src_configure() { + tc-export CC CXX LD NM OBJDUMP PKG_CONFIG cd "${S}/nDPI" econf cd "${S}" In addition I see that ntopng links to libzmq.so.5, libjson-c.so.2 and libhiredis.so.0.13 on my system. So it looks like you didn't catch all the dependencies. Please verify |