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 treegraph
 
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

Note You need to log in before you can comment on or make changes to this bug.
Description Dimitry Andric freebsd_committer 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 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 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