Bug 200475 - [exp-run] with the "CC" alias removed (see Phabricator review D1932)
Summary: [exp-run] with the "CC" alias removed (see Phabricator review D1932)
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Dimitry Andric
Depends on: 200522 200523 200524 200525 200527
  Show dependency treegraph
Reported: 2015-05-26 20:22 UTC by Dimitry Andric
Modified: 2015-05-29 21:52 UTC (History)
3 users (show)

See Also:
dim: exp-run?


Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer 2015-05-26 20:22:04 UTC
Please perform an exp-run with the patch from https://reviews.freebsd.org/D1932 applied.  This patch removes the "CC" alias for "c++".

I know that at least CMake seems to prefer "CC" over "c++", but if the former does not exist at all, it should choose the latter instead.  There may be other programs that change their configure or build behavior due to this change.
Comment 1 Antoine Brodin freebsd_committer 2015-05-27 13:32:21 UTC
Take for exp-run
Comment 2 Antoine Brodin freebsd_committer 2015-05-28 21:53:49 UTC
Exp-run results:


5 new failures:

+ {"origin"=>"converters/tuc", "pkgname"=>"tuc-1.10", "phase"=>"build", "errortype"=>"???"}
+ {"origin"=>"math/rapid", "pkgname"=>"rapid-2.01", "phase"=>"build", "errortype"=>"???"}
+ {"origin"=>"misc/felis", "pkgname"=>"felis-1.0", "phase"=>"build", "errortype"=>"???"}
+ {"origin"=>"print/pfbtopfa", "pkgname"=>"pfbtopfa-1.0", "phase"=>"build", "errortype"=>"???"}
+ {"origin"=>"textproc/kenlm", "pkgname"=>"kenlm-20140830", "phase"=>"build", "errortype"=>"???"}

Failure logs:


For converters/tuc, misc/felis and print/pfbtopfa,  CC is used where it seems ${CC} is wanted.

math/rapid and textproc/kenlm may be able to use ${CXX} instead of CC
Comment 3 Dimitry Andric freebsd_committer 2015-05-29 20:53:06 UTC
Submitted bug 200522 for converters/tuc.
Comment 4 Dimitry Andric freebsd_committer 2015-05-29 20:56:51 UTC
Submitted bug 200523 for misc/felis.
Comment 5 Dimitry Andric freebsd_committer 2015-05-29 21:00:10 UTC
Submitted bug 200524 for print/pfbtopfa.
Comment 6 Dimitry Andric freebsd_committer 2015-05-29 21:05:17 UTC
Submitted bug 200525 for math/rapid.
Comment 7 Dimitry Andric freebsd_committer 2015-05-29 21:17:30 UTC
Submitted bug 200527 for textproc/kenlm.
Comment 8 commit-hook freebsd_committer 2015-05-29 21:20:36 UTC
A commit references this bug:

Author: dim
Date: Fri May 29 21:19:56 UTC 2015
New revision: 387891
URL: https://svnweb.freebsd.org/changeset/ports/387891

  In PR200475, it turned out math/rapid is using "CC" to compile C++
  files. Make it use ${CXX} instead.

  Approved by:	jwbacon@tds.net (maintainer)
  PR:		200475, 200525

Comment 9 commit-hook freebsd_committer 2015-05-29 21:33:41 UTC
A commit references this bug:

Author: dim
Date: Fri May 29 21:32:44 UTC 2015
New revision: 387893
URL: https://svnweb.freebsd.org/changeset/ports/387893

  In PR200475, it turned out textproc/kenlm is using "CC" to do a sanity
  check at bootstrap time.  Make it use ${CXX} instead.

  Approved by:	demon (maintainer)
  PR:		200475, 200527

Comment 10 Dimitry Andric freebsd_committer 2015-05-29 21:52:17 UTC
Closing this bug now, as all sub-bugs have been fixed.  Now /usr/bin/CC is no longer necessary! :-)