Page MenuHomeFreeBSD
Differential D20997

science/rdkit: fix build on GCC architectures
ClosedPublic
Actions

Authored by pkubaj on Jul 19 2019, 4:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 18, 4:44 AM
Unknown Object (File)
Wed, Oct 15, 4:10 AM
Unknown Object (File)
Sun, Oct 12, 3:57 PM
Unknown Object (File)
Fri, Sep 26, 8:04 AM
Unknown Object (File)
Fri, Sep 26, 7:27 AM
Unknown Object (File)
Thu, Sep 25, 9:15 PM
Unknown Object (File)
Thu, Sep 25, 1:30 PM
Unknown Object (File)
Sep 24 2025, 3:44 AM
View All Files
Subscribers
None

Details

Reviewers
linimon
mat
tcberner
Commits
rP509389: science/rdkit: fix build on GCC architectures
Summary

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.

PR: 239299

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

pkubaj created this revision.Jul 19 2019, 4:05 PM
Harbormaster completed remote builds in B25423: Diff 59930.Jul 19 2019, 4:05 PM
pkubaj updated this revision to Diff 60292.Jul 30 2019, 4:13 PM

Fix packaging.

Harbormaster completed remote builds in B25596: Diff 60292.Jul 30 2019, 4:13 PM
pkubaj updated this revision to Diff 60293.Jul 30 2019, 4:14 PM

Correct PLIST_SUB.

Harbormaster completed remote builds in B25597: Diff 60293.Jul 30 2019, 4:14 PM
tcberner added inline comments.Aug 4 2019, 6:02 PM
science/rdkit/pkg-plist
298 ↗(On Diff #60293)

wouldn't it make more sense to use NON_PPC64 here? as you are not installing them when the architecture is ppc64.

mat added inline comments.Aug 13 2019, 1:19 PM
science/rdkit/Makefile
52–56 ↗(On Diff #60293)

ARCH is defined after bsd.port.options.mk, no need to do the pre/post thing.

pkubaj updated this revision to Diff 60738.Aug 13 2019, 4:16 PM
pkubaj marked an inline comment as done.

Don't include bsd.port.pre.mk, bsd.port.options.mk is enough.

Also the .if is not exclusive to powerpc64, but to all big-endian architectures. Since there's no ports variable for endianness, we need to list all architectures.

Harbormaster completed remote builds in B25841: Diff 60738.Aug 13 2019, 4:16 PM
pkubaj marked an inline comment as done.Aug 13 2019, 4:17 PM
pkubaj added inline comments.
science/rdkit/pkg-plist
298 ↗(On Diff #60293)

Changed to LE, since the issue is about endianness.

mat accepted this revision.Aug 19 2019, 11:52 AM

"LE" is pretty generic, LITTLE_ENDIAN would probably be better.

This revision is now accepted and ready to land.Aug 19 2019, 11:52 AM
Closed by commit rP509389: science/rdkit: fix build on GCC architectures (authored by pkubaj). · Explain WhyAug 20 2019, 12:09 AM
This revision was automatically updated to reflect the committed changes.
pkubaj added a commit: rP509389: science/rdkit: fix build on GCC architectures.

Revision Contents
Changeset List

PathSize
head/
science/
rdkit/
13 lines
22 lines

Diff 61021

View Options

head/science/rdkit/Makefile

Loading...
View Options

head/science/rdkit/pkg-plist

Loading...