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

Registry fails to update with 'unpacking the sent packfile failed on the remote' #152

Closed
Bisaloo opened this issue May 9, 2022 · 3 comments

Comments

@Bisaloo
Copy link

Bisaloo commented May 9, 2022

https://github.com/r-universe/epiforecasts/actions/workflows/sync.yml

ERROR updating forecast.vocs from https://github.com/epiforecasts/forecast.vocs (unpacking the sent packfile failed on the remote)

ERROR updating socialmixr from https://github.com/epiforecasts/socialmixr (unpacking the sent packfile failed on the remote)

ERROR updating cmdstanr from https://github.com/stan-dev/cmdstanr (unpacking the sent packfile failed on the remote)

Error in sync::sync_from_registry("https://github.com/r-universe/epiforecasts") :
Failed to update packages: forecast.vocs, socialmixr, cmdstanr

@jeroen
Copy link
Member

jeroen commented May 11, 2022

I thought maybe the repo is corrupted somehow, so I deleted it and started from scratch, but the problem persists...

@jeroen
Copy link
Member

jeroen commented May 13, 2022

OK I found the problem. The root cause is a syntax error in the Authors@R field from the description file in forecast.vocs.

Your last commit of the description file added an author, however you have mixed up the parenthesis and therefore instead of two person() objects, you have one person() which is nested into another. Somehow R accepts this and we get a malformed maintainer signature.

I have added a step to the universe sync function that tries to validate the maintainer sig before using it in the commit: r-universe-org/sync@42e61e5

Bisaloo added a commit to epiforecasts/forecast.vocs that referenced this issue May 13, 2022
@Bisaloo
Copy link
Author

Bisaloo commented May 13, 2022

Ah yes, good catch. And socialmixr had a similar issue that I fixed a couple of days ago: epiforecasts/socialmixr@541d170 (missing closing parenthesis on L13).

Thanks for your help!

This issue was closed.
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

No branches or pull requests

2 participants