Page MenuHomeFreeBSD

math/R: Fix build with clang/openmp 13
ClosedPublic

Authored by jrm on Oct 3 2021, 2:27 PM.
Tags
None
Referenced Files
F102996056: D32280.id.diff
Tue, Nov 19, 3:13 PM
F102927121: D32280.diff
Mon, Nov 18, 8:28 PM
Unknown Object (File)
Fri, Nov 8, 1:18 AM
Unknown Object (File)
Tue, Nov 5, 6:56 PM
Unknown Object (File)
Tue, Nov 5, 7:40 AM
Unknown Object (File)
Oct 18 2024, 3:45 AM
Unknown Object (File)
Oct 17 2024, 6:43 AM
Unknown Object (File)
Oct 16 2024, 5:47 PM
Subscribers

Details

Summary

The mgcv package is one of the recommended packages to be installed with
R. A problem with mgcv and clang/openmp 13 is that mgcv versions prior
to 1.8-37 include omp.h after including R.h. This causes a conflict
between "#define match" and an openmp pragma line, which uses match().
This problem was fixed in mgcv 1.8-37, so switch to this newer version.

PR: 258887
Reported by: dim

Diff Detail

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

Event Timeline

jrm requested review of this revision.Oct 3 2021, 2:27 PM
  • Fix plist issue when the NLS knob is on
  • Bump PORTREVISION

Builds OK for me, also installs, and seems to run (at least very rudimentarily :) ):

% R

R version 4.1.1 (2021-08-10) -- "Kick Things"
Copyright (C) 2021 The R Foundation for Statistical Computing
Platform: amd64-portbld-freebsd14.0 (64-bit)

R is free software and comes with ABSOLUTELY NO WARRANTY.
You are welcome to redistribute it under certain conditions.
Type 'license()' or 'licence()' for distribution details.

R is a collaborative project with many contributors.
Type 'contributors()' for more information and
'citation()' on how to cite R or R packages in publications.

Type 'demo()' for some demos, 'help()' for on-line help, or
'help.start()' for an HTML browser interface to help.
Type 'q()' to quit R.

> print("Hello World")
[1] "Hello World"
> sin(pi)
[1] 1.224647e-16
> quit()
Save workspace image? [y/n/c]: n
This revision is now accepted and ready to land.Oct 3 2021, 7:00 PM
This revision was automatically updated to reflect the committed changes.