Bug 200527 - Make textproc/kenlm use the CXX variable
Summary: Make textproc/kenlm use the CXX variable
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Dmitry Sivachenko
URL:
Keywords:
Depends on:
Blocks: 200475
  Show dependency tree / graph
 
Reported: 2015-05-29 21:17 UTC by Dimitry Andric
Modified: 2015-05-29 21:42 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (demon)

Attachments
Fix textproc/kenlm sanity Jamfile to use the CXX variable (1.35 KB, patch)
2015-05-29 21:17 UTC, Dimitry Andric 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer freebsd_triage 2015-05-29 21:17:08 UTC 
Created attachment 157258 [details]
Fix textproc/kenlm sanity Jamfile to use the CXX variable

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

Note that either with or without this patch, the port does not build to completion anyway, with link errors like these:

clang-linux.link lm/bin/model_test.test/clang-linux-3.6.1/release/link-static/threading-multi/model_test
/usr/lib/crt1.o: In function `_start1':
/usr/src/lib/csu/i386/crt1_c.c:(.text+0x156): undefined reference to `main'
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
Comment 1 Dmitry Sivachenko freebsd_committer freebsd_triage 2015-05-29 21:25:24 UTC 
(In reply to Dimitry Andric from comment #0)

Looks fine, please commit.

PS: it does compile at least on stable/10 (amd64)
Comment 2 commit-hook freebsd_committer freebsd_triage 2015-05-29 21:33:42 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

Log:
  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

Changes:
  head/textproc/kenlm/files/patch-jam-files-sanity.jam