Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop #96

Merged
merged 4 commits into from
Sep 2, 2024
Merged

Develop #96

merged 4 commits into from
Sep 2, 2024

Conversation

auzaheta
Copy link
Collaborator

@auzaheta auzaheta commented Sep 2, 2024

Description

The CRAN team reported that when the package is checked --as-cran finds

* checking use of SHLIB_OPENMP_*FLAGS in Makefiles ... NOTE
  src/Makevars: SHLIB_OPENMP_CXXFLAGS is included in PKG_CXXFLAGS but not in PKG
_LIBS
  src/Makevars: SHLIB_OPENMP_CFLAGS is included in PKG_LIBS but linking is by C+
+

Use of these macros is discussed in sect 1.2.1.1 of ‘Writing R
Extensions’. The macros for different languages may differ so the
matching macro must be used in PKG_CXXFLAGS (etc) and match that used
in PKG_LIBS (except for Fortran: see the manual).

(not seen in any of the regular checks).

In src/Makevars.in you should change

PKG_LIBS= @OPENMP_CFLAG@ $(LAPACK_LIBS) $(BLAS_LIBS) $(FLIBS)

to

PKG_LIBS= @OPENMP_FLAG@ $(LAPACK_LIBS) $(BLAS_LIBS) $(FLIBS)

Checklist:

  • I have added tests that prove my fix is effective or that my feature works
  • The package builds on my OS without issues (please add workstation details)
  • My changes generate no new warnings
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (.R, NEWS.md)
  • I have bumped the version by the appropriate increment in the DESCRIPTION file (major, minor, patch)

@auzaheta auzaheta merged commit 0fcddaf into main Sep 2, 2024
9 checks passed
@jhollway
Copy link
Collaborator

jhollway commented Sep 2, 2024

Super, thanks @auzaheta!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants