search

r-lib / pkgdepends

R Package Dependency Resolution
https://r-lib.github.io/pkgdepends/
Other
102 stars 30 forks source link

Sending "multi_ack" in git-v1 #333

Closed jameslairdsmith closed 1 year ago

jameslairdsmith commented 1 year ago

I've been spending a little while trying to get pak to work with the Bank of England's locally hosted version of Azure DevOps, but I've been getting an HTTP error when pak (or more accurately pkgdepends) tries to get the target package's DESCRIPTION file. The various error messages point towards pak not advertising it's "multi_ack" support in git protocol V1.

I've tested the fix in this PR internally for the Bank's R packages and it works for us. I unfortunately can't provide a reprex, but there might be a public instance of Azure DevOps that I don't know about which could help with that.

gaborcsardi commented 1 year ago

Thanks! It does not break the Bioconductor git repo, so it is probably fine everywhere.