Bug 242966 - math/clp: fix build on non-x86 non-ARM
Summary: math/clp: fix build on non-x86 non-ARM
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: Piotr Kubaj
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-12-29 18:14 UTC by Piotr Kubaj
Modified: 2020-01-12 17:28 UTC (History)
1 user (show)

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

Attachments
patch (1.88 KB, patch)
2019-12-29 18:14 UTC, Piotr Kubaj 		pkubaj: maintainer-approval? (co9co9)
 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 Piotr Kubaj freebsd_committer freebsd_triage 2019-12-29 18:14:56 UTC 
Created attachment 210309 [details]
patch

Only amd64 and i386 have immintrin.h. There's a check for ARM, but not for other architectures.

Building without immintrin.h works fine on amd64.
Comment 1 commit-hook freebsd_committer freebsd_triage 2020-01-12 17:27:06 UTC 
A commit references this bug:

Author: pkubaj
Date: Sun Jan 12 17:26:25 UTC 2020
New revision: 522821
URL: https://svnweb.freebsd.org/changeset/ports/522821

Log:
  math/clp: fix build on non-x86 non-ARM

  Only amd64 and i386 have immintrin.h. There's a check for ARM, but not for other architectures.

  Building without immintrin.h works fine on amd64.

  PR:		242966
  Approved by:	co9co9@gmail.com (maintainer timeout)
  MFH:		2020Q1 (fix build blanket)

Changes:
  head/math/clp/files/patch-Clp_src_ClpPackedMatrix.cpp
  head/math/clp/files/patch-Clp_src_ClpSimplexDual.cpp
Comment 2 commit-hook freebsd_committer freebsd_triage 2020-01-12 17:28:09 UTC 
A commit references this bug:

Author: pkubaj
Date: Sun Jan 12 17:28:03 UTC 2020
New revision: 522823
URL: https://svnweb.freebsd.org/changeset/ports/522823

Log:
  MFH: r522821

  math/clp: fix build on non-x86 non-ARM

  Only amd64 and i386 have immintrin.h. There's a check for ARM, but not for other architectures.

  Building without immintrin.h works fine on amd64.

  PR:		242966
  Approved by:	co9co9@gmail.com (maintainer timeout)
  Approved by:	portmgr (fix build blanket)

Changes:
_U  branches/2020Q1/
  branches/2020Q1/math/clp/files/patch-Clp_src_ClpPackedMatrix.cpp
  branches/2020Q1/math/clp/files/patch-Clp_src_ClpSimplexDual.cpp