Bug 232598 - devel/cquery: Allow overriding LLVM version
Summary: devel/cquery: Allow overriding LLVM version
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Tobias Kortkamp
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2018-10-24 09:04 UTC by Tobias Kortkamp
Modified: 2018-11-08 12:08 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (henry.hu.sh)

Attachments
cquery.diff (943 bytes, patch)
2018-10-24 09:04 UTC, Tobias Kortkamp 		tobik: maintainer-approval? (henry.hu.sh)
 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 Tobias Kortkamp freebsd_committer freebsd_triage 2018-10-24 09:04:40 UTC 
Created attachment 198538 [details]
cquery.diff

Cquery works fine with LLVM 7 and I'd like to be able to override it for myself
when necessary.  This adds an LLVM_DEFAULT variable so that users can override
it in their make.conf.  It's similar to what the Gecko ports already use.
Comment 1 commit-hook freebsd_committer freebsd_triage 2018-11-08 12:08:19 UTC 
A commit references this bug:

Author: tobik
Date: Thu Nov  8 12:07:54 UTC 2018
New revision: 484446
URL: https://svnweb.freebsd.org/changeset/ports/484446

Log:
  devel/cquery: Allow overriding the LLVM version via LLVM_DEFAULT

  - Cquery builds and works fine with LLVM 7 for example
  - Make the libclang.so spec more specific to prevent the framework
    from picking the wrong library version from llvm60 with LLVM_DEFAULT=70
    when both llvm60 and llvm70 are installed in the build environment

  PR:		232598
  Approved by:	henry.hu.sh@gmail.com (maintainer timeout, 2 weeks)

Changes:
  head/devel/cquery/Makefile