Bug 234179

Summary: science/py-dlib: fix build with GCC-based architectures
Product: Ports & Packages Reporter: Piotr Kubaj <pkubaj>
Component: Individual Port(s)Assignee: Gleb Popov <arrowd>
Status: Closed FIXED    
Severity: Affects Only Me CC: linimon, pkubaj
Priority: --- Flags: arrowd: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch none

Description Piotr Kubaj freebsd_committer freebsd_triage 2018-12-19 19:39:54 UTC
Created attachment 200283 [details]
patch

This port needs new GCC to build, so add USES=compiler:c++11-lang to fix build on GCC architectures.

Additionally, it tests for AVX support even if AVX option is off, which breaks build.

Explicitly disable AVX when AVX option is off. Since avxargs is used by BLASLAPACK option, rename their avxags to blaslapackargs.

Tested on powerpc64 and amd64.

Hardware sponsored by IntegriCloud.
Comment 1 Gleb Popov freebsd_committer freebsd_triage 2018-12-20 07:05:37 UTC
The change looks fine, thanks for fixing that.
Comment 2 commit-hook freebsd_committer freebsd_triage 2018-12-22 01:19:20 UTC
A commit references this bug:

Author: linimon
Date: Sat Dec 22 01:18:32 UTC 2018
New revision: 488038
URL: https://svnweb.freebsd.org/changeset/ports/488038

Log:
  This port needs a newer GCC to build, so add USES=compiler:c++11-lang
  to fix build on GCC-based architectures.

  Additionally, it tests for AVX support even if AVX option is off,
  which breaks the build.

  Explicitly disable AVX when AVX option is off.  Since avxargs is used
  by BLASLAPACK option, rename their avxags to blaslapackargs.

  While here, pet portlint (whitespace).

  PR:		234179
  Submitted by:	Piotr Kubaj
  Approved by:	maintainer

Changes:
  head/science/py-dlib/Makefile