search

r-lib / actions

GitHub Actions for the R community
Creative Commons Zero v1.0 Universal
968 stars 216 forks source link

[macOS] udunits library is not found #877

Closed Enchufa2 closed 3 months ago

Enchufa2 commented 3 months ago

Consider this workflow, which used to work fine, but now macOS fails (see here) because libudunits was not found. However, you can see that it was installed in a previous step, i.e. here. As I said, this used to work just fine.

gaborcsardi commented 3 months ago

Why do you think this is a bug in r-lib/actions? In which action specifically?

Enchufa2 commented 3 months ago

Maybe setup-r's toolchain is missing a path? Maybe GA changed something in the base image that setup-r or check-r-package are not considering? I don't know. Not a macOS expert here, and we're following the documentation. The thing is that the workflow didn't change, and it worked, but now it fails, and we use:

gaborcsardi commented 3 months ago

It also fails for me locally, the same way, so this is not an issue with r-lib/actions. 

❯ R CMD INSTALL .
* installing to library ‘/Users/gaborcsardi/Library/R/arm64/4.4/library’
* installing *source* package ‘units’ ...
** using staged installation
configure: units: 0.8-5
...
checking for udunits2.h... no
checking for udunits2/udunits2.h... no
checking for ut_read_xml in -ludunits2... no
configure: error: in `/private/tmp/units':
configure: error:
...
❯ brew info udunits

==> udunits: stable 2.2.28 (bottled), HEAD
Unidata unit conversion library
https://www.unidata.ucar.edu/software/udunits/
Installed
/opt/homebrew/Cellar/udunits/2.2.28 (30 files, 662.1KB) *
  Poured from bottle using the formulae.brew.sh API on 2024-07-11 at 19:21:53
Enchufa2 commented 3 months ago

So the toolchain? Or brew? Nothing changed in units or the workflow.

gaborcsardi commented 3 months ago

Possibly the R package never worked on arm64 macOS with Homebrew. Nevertheless, this is not a problem with r-lib/actions, so I am going to have to close this issue.

github-actions[bot] commented 2 months ago

This issue has been automatically locked. If you believe you have found a related problem, please file a new issue and include a link to this issue