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

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

Closed
maelle opened this issue Apr 26, 2022 · 13 comments
Closed

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

maelle opened this issue Apr 26, 2022 · 13 comments
Assignees

Comments

@maelle
Copy link

maelle commented Apr 26, 2022

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
Copy link
Owner

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

@eblondel
Copy link
Owner

eblondel commented Apr 26, 2022

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
Copy link
Owner

@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
Copy link
Author

maelle commented Apr 26, 2022

Thanks, will do!

@maelle maelle closed this as completed Apr 26, 2022
@eblondel
Copy link
Owner

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
Copy link
Author

maelle commented May 3, 2022

@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 🙏

@eblondel
Copy link
Owner

eblondel commented May 3, 2022

@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
Copy link
Author

maelle commented May 3, 2022

Thank you!

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

@eblondel
Copy link
Owner

eblondel commented May 3, 2022

@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
Copy link
Owner

eblondel commented May 3, 2022

reverting at 6eac341

@eblondel
Copy link
Owner

@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

@eblondel eblondel changed the title Another note on {XML} Issue with XML package in Github r-lib v2 actions May 18, 2022
@maelle
Copy link
Author

maelle commented May 20, 2022

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

Thank you!!

@eblondel
Copy link
Owner

Wonderful!

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

No branches or pull requests

2 participants