Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 607468 - dev-lang/crystal-0.20.5: does not respect CC/CXX
Summary: dev-lang/crystal-0.20.5: does not respect CC/CXX
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Lixiao Chen
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: tc-directly
  Show dependency tree
 
Reported: 2017-01-28 11:36 UTC by Michał Górny
Modified: 2017-01-28 12:49 UTC (History)
2 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 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-01-28 11:36:44 UTC
From early start of build log:

g++ -c -O2 -pipe -march=native  -o src/llvm/ext/llvm_ext.o src/llvm/ext/llvm_ext.cc `/usr/bin/llvm-config --cxxflags`
cc -O2 -pipe -march=native -fPIC    -c -o src/ext/sigfault.o src/ext/sigfault.c
ar -rcs src/ext/libcrystal.a src/ext/sigfault.o

This really should be $(tc-getCXX)/$(tc-getCC)/$(tc-getAR)
Comment 1 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2017-01-28 11:38:30 UTC
As a side note, threads=4 seems a wrong idea to. I'd suggest, from multiprocessing:

  $(makeopts_jobs "${MAKEOPTS}" "$(get_nproc)"
Comment 2 Sergei Trofimovich gentoo-dev 2017-01-28 12:49:51 UTC
Pushed as:
    https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=ef80935bdf6325a68ed25e5a94ae602227402cd2

Proposed AR override upstream as:
    https://github.com/crystal-lang/crystal/pull/3950

Thanks for the report!