Bug 239737

Summary: factor out hardcoding of "clangXYZ" in devel/flang
Product: Ports & Packages Reporter: Mark Linimon <linimon>
Component: Individual Port(s)Assignee: Johannes M Dieterich <jmd>
Status: Closed FIXED    
Severity: Affects Only Me Flags: bugzilla: maintainer-feedback? (jmd)
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Bug Depends on:    
Bug Blocks: 239736    
Attachments:
Description Flags
patch to devel/flang/Makefile none

Description Mark Linimon freebsd_committer freebsd_triage 2019-08-09 10:39:32 UTC
Created attachment 206389 [details]
patch to devel/flang/Makefile

There are a number of places in the ports tree where the string "clang<version" is called out.  When doing the work to identify stale versions of clang tree-wide, these instances cause extra work.

Refactor this usage into LLVM_VER.

Note: this refactoring causes portlint to complain but IMHO this complication is worth tolerating to help ease maintainability.

Tested on: amd64.
Comment 1 commit-hook freebsd_committer freebsd_triage 2019-10-04 22:13:27 UTC
A commit references this bug:

Author: linimon
Date: Fri Oct  4 22:13:05 UTC 2019
New revision: 513787
URL: https://svnweb.freebsd.org/changeset/ports/513787

Log:
  Factor out hardcoding of the string "clang<version".  When doing the
  work to identify stale versions of clang tree-wide, these instances
  cause extra work.

  Refactor this usage into LLVM_VER.

  Note: this refactoring causes portlint to complain but IMHO this
  annoyance is worth tolerating to help ease maintainability.

  PR:		239737, 239738
  Approved by:	maintainer

Changes:
  head/devel/flang/Makefile
  head/devel/flang-clang/Makefile
Comment 2 Mark Linimon freebsd_committer freebsd_triage 2019-10-04 22:14:36 UTC
Committed, thanks.