Summary: | math/metis: update IDXTYPEWIDTH to 64 on 64 bit platforms. | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Ports & Packages | Reporter: | Pedro F. Giffuni <pfg> | ||||||||||
Component: | Individual Port(s) | Assignee: | Steve Wills <swills> | ||||||||||
Status: | Closed FIXED | ||||||||||||
Severity: | Affects Only Me | Keywords: | patch | ||||||||||
Priority: | --- | ||||||||||||
Version: | Latest | ||||||||||||
Hardware: | Any | ||||||||||||
OS: | Any | ||||||||||||
Attachments: |
|
Created attachment 183176 [details]
Metis Port update patch
Some more change while here:
- Use OpenMP by default but define compiler:openmp which is more generic than just depending on GCC.
- Regenerate the patch with "makepatch".
There are currently no ports depending on this version of metis so the changes won't have any impact.
Created attachment 183189 [details]
Metis Port update patch
KISS: leave out openmp as a default for now. Upstream has a special version for that.
The rest of the patch has a higher priority anyways.
Created attachment 201012 [details]
math/metis update patch
Add some extra portlint
A commit references this bug: Author: swills Date: Tue Jan 15 12:52:47 UTC 2019 New revision: 490369 URL: https://svnweb.freebsd.org/changeset/ports/490369 Log: math/metis: update IDXTYPEWIDTH to 64 on 64 bit platforms While here, pet portlint PR: 219749 Submitted by: pfg Changes: head/math/metis/Makefile head/math/metis/files/patch-programs_CMakeLists.txt Committed, thanks! |
Created attachment 183172 [details] Port update patch This appears documented in the INSTALL.txt file: ... 3. Edit the file include/metis.h and specify the width (32 or 64 bits) of the elementary data type used in METIS. This is controled by the IDXTYPEWIDTH constant. For now, on a 32 bit architecture you can only specify a width of 32, whereas for a 64 bit architecture you can specify a width of either 32 or 64 bits. ... It affects the case where there is a huge graph. I decided not to touch REALTYPEWIDTH for now, since it is unclear what side effects it may have.