Bug 239299

Summary: science/rdkit: fix build on non-x86 architectures
Product: Ports & Packages Reporter: Piotr Kubaj <pkubaj>
Component: Individual Port(s)Assignee: Piotr Kubaj <pkubaj>
Status: Closed FIXED    
Severity: Affects Only Me CC: yuri
Priority: --- Flags: yuri: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
patch
yuri: maintainer-approval+
v2 yuri: maintainer-approval+

Description Piotr Kubaj freebsd_committer freebsd_triage 2019-07-19 08:45:32 UTC
Created attachment 205885 [details]
patch

This port needs new GCC to build.

It also adds -mpopcnt to *FLAGS, which breaks build on non-x86. Since it optimizes for the host CPU, don't enable it.
Comment 1 Yuri Victorovich freebsd_committer freebsd_triage 2019-07-19 08:48:17 UTC
Comment on attachment 205885 [details]
patch

Approved.
Comment 2 Piotr Kubaj freebsd_committer freebsd_triage 2019-07-30 14:45:43 UTC
Created attachment 206157 [details]
v2

Fix to pkg-plist is necessary. Those files are not installed on powerpc64, I'm not sure about other architectures.
Comment 3 commit-hook freebsd_committer freebsd_triage 2019-08-20 00:09:27 UTC
A commit references this bug:

Author: pkubaj
Date: Tue Aug 20 00:08:48 UTC 2019
New revision: 509389
URL: https://svnweb.freebsd.org/changeset/ports/509389

Log:
  science/rdkit: fix build on GCC architectures

  This port needs new GCC to build.

  It adds -mpopcnt to *FLAGS, which breaks build on non-x86. Since it optimizes for the host CPU, don't enable it.

  Some libraries are not installed on big-endian systems.

  PR:             239299
  Approved by:    yuri (maintainer), mat (mentor)
  Differential Revision:  https://reviews.freebsd.org/D20997

Changes:
  head/science/rdkit/Makefile
  head/science/rdkit/pkg-plist