eblondel / ows4R

R Interface for OGC Web-Services (OWS)
https://eblondel.github.io/ows4R/
Other
36 stars 8 forks source link

Issue with XML package in Github r-lib v2 actions #71

Closed maelle closed 2 years ago

maelle commented 2 years ago

Related to #59

Just for info we're seeing such compilation errors on Ubuntu with XML: EMODnet/EMODnetWFS/runs/6176000380?check_suite_focus=true

Cc @annakrystalli

eblondel commented 2 years ago

it's not really related to #59 by the way :-)

eblondel commented 2 years ago

ows4R is not yet tested on ubuntu-latest. XML package is maintained by CRAN team, if there is some issue related to code compilation it will be probably fixed soon.

eblondel commented 2 years ago

@maelle i've configured the github CI with ubuntu-latest for both R release/devel, and it works. Not sure why you got a compilation issue, maybe it's unexpected error, and trying to re-run the job will work.

maelle commented 2 years ago

Thanks, will do!

eblondel commented 2 years ago

Ok, fyi i dont configure macOS and Windows CI tests because I do need to run docker images and docker doesn't work there, I will look further how I can build a more complete CI action later that would test over these OS.

maelle commented 2 years ago

@eblondel do you know of examples of GHA workflow without Docker that'd work for XML? Our standard workflows still don't work (which worries me for standard users that will try to install the package outside of Docker too). See https://github.com/EMODnet/EMODnetWFS/actions/runs/2257266720

Thank you :pray:

eblondel commented 2 years ago

@maelle just a clarification, I'm not containerizing the tests within docker, but Docker is used to launch some applications WITHIN the VM that is launched with ubuntu for tests. Most of the packages i authored used XML behind, such as geometa, geosapi, atom4R. I'm going to add more configs to them to test them deeply, and to have a look to your config above to see what you could do.

maelle commented 2 years ago

Thank you!

For the record, I use Ubuntu locally, and was able to install XML without any particular issue.

eblondel commented 2 years ago

@maelle looking at your action config, I see that you use v2 actions from r-lib, while in ows4R I used 'v1' for r-setup and r-setup-dependencies; and 'check-r-package' for check-r-package action https://github.com/eblondel/ows4R/blob/master/.github/workflows/r-cmd-check.yml#L46 This may be related in particular to https://github.com/r-lib/actions/tree/v2-branch/setup-r-dependencies#installing-the-latest-dependencies I'm going to give a try over v2 r-lib/actions with ows4R and let you know

eblondel commented 2 years ago

reverting at https://github.com/eblondel/ows4R/commit/6eac34136f29a2677f2ea707bff39cd05b8ab8db

eblondel commented 2 years ago

@maelle this is what I could investigate related to Github CI failures in building XML package:

Let me know if this fixes your build for https://github.com/EMODnet/EMODnetWFS

maelle commented 2 years ago

Success https://github.com/EMODnet/EMODnetWFS/runs/6521287978?check_suite_focus=true :tada:

Thank you!!

eblondel commented 2 years ago

Wonderful!