Bug 197385 - [MAINTAINER] math/openblas: Add Clang support
Summary: [MAINTAINER] math/openblas: Add Clang support
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: Thomas Zander
URL:
Keywords: patch, patch-ready
Depends on:
Blocks:
 
Reported: 2015-02-07 03:52 UTC by Eijiro Shibusawa
Modified: 2015-02-23 00:06 UTC (History)
2 users (show)

See Also:


Attachments
diff (1.50 KB, patch)
2015-02-07 03:52 UTC, Eijiro Shibusawa
koobs: maintainer-approval+
Details | Diff
poudriere log (compressed) (256.67 KB, application/octet-stream)
2015-02-07 03:57 UTC, Eijiro Shibusawa
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Eijiro Shibusawa 2015-02-07 03:52:32 UTC
Created attachment 152652 [details]
diff

Attached patch adds Clang support.

NOTE:
- This port was tested with portlint.
- The maintainer has carried out only a minimal testing by using poudriere.
Comment 1 Eijiro Shibusawa 2015-02-07 03:57:54 UTC
Created attachment 152655 [details]
poudriere log (compressed)
Comment 2 commit-hook freebsd_committer 2015-02-21 13:41:14 UTC
A commit references this bug:

Author: riggs
Date: Sat Feb 21 13:40:20 UTC 2015
New revision: 379512
URL: https://svnweb.freebsd.org/changeset/ports/379512

Log:
  Add support for clang

  PR:		197385
  Submitted by:	phd_kimberlite@yahoo.co.jp (maintainer)

Changes:
  head/math/openblas/Makefile
  head/math/openblas/files/patch-Makefile.system
Comment 3 Thomas Zander freebsd_committer 2015-02-21 13:52:54 UTC
For the next update to the port, could you double-check the OPTIONs? For instance, clang does not feature OpenMP support yet. It is not a showstopper for this commit because the default config builds, but please have a look next time. Thank you!
Comment 4 commit-hook freebsd_committer 2015-02-23 00:06:05 UTC
A commit references this bug:

Author: gerald
Date: Mon Feb 23 00:05:41 UTC 2015
New revision: 379635
URL: https://svnweb.freebsd.org/changeset/ports/379635

Log:
  Replace using _GCC_VER which is essentially internal to Mk/bsd.gcc.mk
  (and replaced usage of _GCC_RUNTIME) by LDFLAGS which is a cleaner and
  more robust approach.

  PR:		197385
  Submitted by:	phd_kimberlite@yahoo.co.jp (maintainer)

Changes:
  head/math/openblas/Makefile
  head/math/openblas/files/patch-Makefile.system