Bug 229724 - math/cryptominisat: unbreak on aarch64
Summary: math/cryptominisat: unbreak on aarch64
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: arm64 Any
: --- Affects Only Me
Assignee: Gleb Popov
URL:
Keywords:
Depends on:
Blocks: 201763
  Show dependency treegraph
 
Reported: 2018-07-12 12:04 UTC by Greg V
Modified: 2018-08-10 10:21 UTC (History)
2 users (show)

See Also:
arrowd: maintainer-feedback+


Attachments
cryptominisat.patch (887 bytes, patch)
2018-07-12 12:04 UTC, Greg V
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Greg V 2018-07-12 12:04:07 UTC
Created attachment 195075 [details]
cryptominisat.patch

http://thunderx1.nyi.freebsd.org/data/head-arm64-default/p472730_s335353/logs/errors/cryptominisat-5.6.1.log

-mtune=native is not supported on aarch64. We actually probably don't want it anywhere: why tune for the build box's processor in particular.
Comment 1 Gleb Popov freebsd_committer 2018-07-12 16:29:26 UTC
The patch looks fine.
Comment 2 commit-hook freebsd_committer 2018-08-10 10:20:26 UTC
A commit references this bug:

Author: arrowd
Date: Fri Aug 10 10:20:16 UTC 2018
New revision: 476827
URL: https://svnweb.freebsd.org/changeset/ports/476827

Log:
  math/cryptominisat: Unbreak on aarch64.

  PR:		229724
  Submitted by:	Greg V <greg@unrelenting.technology>
  Approved by:	tcberner (mentor)

Changes:
  head/math/cryptominisat/Makefile
  head/math/cryptominisat/files/
  head/math/cryptominisat/files/patch-CMakeLists.txt
Comment 3 Gleb Popov freebsd_committer 2018-08-10 10:21:39 UTC
Thanks for your contribution.