devtools::install_github remote_sha() issue

[tbillman]

Hello all,

I made a post about an issue I was having regarding installing a package with install_github() on Stack Overflow and was directed here. Post with details here:

https://stackoverflow.com/questions/54768969/r-install-github-issue-with-remote-sha

Essentially I was trying to install the CBPS package with the repo 'kosukeimai/CBPS' and option dependencies = T and I get the following error:

Error in class(xx) <- cl : attempt to set an attribute on NULL

After doing some debugging it looks like the issue occurs in the internal function remote_sha(). After that it goes into UseMethod, which I don't know how to get into.

Any help would be appreciated.

[BarkleyBG]

I think this may relate to r-lib/devtools#1979 ?

## install
install_github(MY_ORG,  username = MY_ID, host = MY_HOST,
auth_token = my_github_pat, dependencies = T)

which returns

##Error in class(xx) <- cl : attempt to set an attribute on NULL

Here's a traceback:

traceback()
10: .POSIXct(res_headers$x-ratelimit-reset, tz = "UTC")
9: github_error(res)
8: stop(github_error(res))
7: github_DESCRIPTION(username = remote$username, repo = remote$repo,
subdir = remote$subdir, host = remote$host, ref = remote$ref,
pat = remote$auth_token %||% github_pat(), use_curl = use_curl)
6: remote_package_name.github_remote(remote)
5: remote_package_name(remote)
4: FUN(X[[i]], ...)
3: vapply(remotes, install_remote, ..., FUN.VALUE = character(1))
2: install_remotes(remotes, auth_token = auth_token, host = host,
dependencies = dependencies, upgrade = upgrade, force = force,
quiet = quiet, build = build, build_opts = build_opts, repos = repos,
type = type, ...)
1: install_github( MY_ORG,  username = MY_ID, host = MY_HOST,
auth_token = my_github_pat, dependencies = T)

[antoine-sachet]

Running into the same issue here. Here is an example of res that makes github_error fail:

res <- 
list(url = "https://api.bitbucket.org/2.0/repos/cloudyr/aws.comprehend/commits/master", 
    status_code = 404L, type = "application/json; charset=utf-8", 
    headers = as.raw(c(0x48, 0x54, 0x54, 0x50, 0x2f, 0x32, 0x20, 
    0x34, 0x30, 0x34, 0x20, 0x0d, 0x0a, 0x73, 0x65, 0x72, 0x76, 
    0x65, 0x72, 0x3a, 0x20, 0x6e, 0x67, 0x69, 0x6e, 0x78, 0x0d, 
    0x0a, 0x76, 0x61, 0x72, 0x79, 0x3a, 0x20, 0x41, 0x75, 0x74, 
    0x68, 0x6f, 0x72, 0x69, 0x7a, 0x61, 0x74, 0x69, 0x6f, 0x6e, 
    0x0d, 0x0a, 0x63, 0x61, 0x63, 0x68, 0x65, 0x2d, 0x63, 0x6f, 
    0x6e, 0x74, 0x72, 0x6f, 0x6c, 0x3a, 0x20, 0x6d, 0x61, 0x78, 
    0x2d, 0x61, 0x67, 0x65, 0x3d, 0x39, 0x30, 0x30, 0x0d, 0x0a, 
    0x63, 0x6f, 0x6e, 0x74, 0x65, 0x6e, 0x74, 0x2d, 0x74, 0x79, 
    0x70, 0x65, 0x3a, 0x20, 0x61, 0x70, 0x70, 0x6c, 0x69, 0x63, 
    0x61, 0x74, 0x69, 0x6f, 0x6e, 0x2f, 0x6a, 0x73, 0x6f, 0x6e, 
    0x3b, 0x20, 0x63, 0x68, 0x61, 0x72, 0x73, 0x65, 0x74, 0x3d, 
    0x75, 0x74, 0x66, 0x2d, 0x38, 0x0d, 0x0a, 0x73, 0x74, 0x72, 
    0x69, 0x63, 0x74, 0x2d, 0x74, 0x72, 0x61, 0x6e, 0x73, 0x70, 
    0x6f, 0x72, 0x74, 0x2d, 0x73, 0x65, 0x63, 0x75, 0x72, 0x69, 
    0x74, 0x79, 0x3a, 0x20, 0x6d, 0x61, 0x78, 0x2d, 0x61, 0x67, 
    0x65, 0x3d, 0x33, 0x31, 0x35, 0x33, 0x36, 0x30, 0x30, 0x30, 
    0x3b, 0x20, 0x69, 0x6e, 0x63, 0x6c, 0x75, 0x64, 0x65, 0x53, 
    0x75, 0x62, 0x44, 0x6f, 0x6d, 0x61, 0x69, 0x6e, 0x73, 0x3b, 
    0x20, 0x70, 0x72, 0x65, 0x6c, 0x6f, 0x61, 0x64, 0x0d, 0x0a, 
    0x64, 0x61, 0x74, 0x65, 0x3a, 0x20, 0x54, 0x75, 0x65, 0x2c, 
    0x20, 0x30, 0x39, 0x20, 0x41, 0x70, 0x72, 0x20, 0x32, 0x30, 
    0x31, 0x39, 0x20, 0x31, 0x33, 0x3a, 0x34, 0x36, 0x3a, 0x34, 
    0x35, 0x20, 0x47, 0x4d, 0x54, 0x0d, 0x0a, 0x78, 0x2d, 0x73, 
    0x65, 0x72, 0x76, 0x65, 0x64, 0x2d, 0x62, 0x79, 0x3a, 0x20, 
    0x61, 0x70, 0x70, 0x2d, 0x31, 0x34, 0x32, 0x0d, 0x0a, 0x78, 
    0x2d, 0x73, 0x74, 0x61, 0x74, 0x69, 0x63, 0x2d, 0x76, 0x65, 
    0x72, 0x73, 0x69, 0x6f, 0x6e, 0x3a, 0x20, 0x65, 0x33, 0x32, 
    0x32, 0x35, 0x31, 0x33, 0x35, 0x34, 0x36, 0x66, 0x62, 0x0d, 
    0x0a, 0x65, 0x74, 0x61, 0x67, 0x3a, 0x20, 0x22, 0x32, 0x62, 
    0x36, 0x32, 0x66, 0x37, 0x33, 0x31, 0x33, 0x38, 0x62, 0x62, 
    0x37, 0x34, 0x35, 0x63, 0x30, 0x61, 0x36, 0x35, 0x64, 0x33, 
    0x30, 0x64, 0x31, 0x35, 0x33, 0x36, 0x31, 0x61, 0x32, 0x32, 
    0x22, 0x0d, 0x0a, 0x78, 0x2d, 0x72, 0x65, 0x6e, 0x64, 0x65, 
    0x72, 0x2d, 0x74, 0x69, 0x6d, 0x65, 0x3a, 0x20, 0x30, 0x2e, 
    0x30, 0x30, 0x39, 0x37, 0x34, 0x38, 0x32, 0x32, 0x30, 0x34, 
    0x34, 0x33, 0x37, 0x33, 0x0d, 0x0a, 0x78, 0x2d, 0x76, 0x65, 
    0x72, 0x73, 0x69, 0x6f, 0x6e, 0x3a, 0x20, 0x65, 0x33, 0x32, 
    0x32, 0x35, 0x31, 0x33, 0x35, 0x34, 0x36, 0x66, 0x62, 0x0d, 
    0x0a, 0x78, 0x2d, 0x72, 0x65, 0x71, 0x75, 0x65, 0x73, 0x74, 
    0x2d, 0x63, 0x6f, 0x75, 0x6e, 0x74, 0x3a, 0x20, 0x34, 0x35, 
    0x33, 0x0d, 0x0a, 0x78, 0x2d, 0x66, 0x72, 0x61, 0x6d, 0x65, 
    0x2d, 0x6f, 0x70, 0x74, 0x69, 0x6f, 0x6e, 0x73, 0x3a, 0x20, 
    0x53, 0x41, 0x4d, 0x45, 0x4f, 0x52, 0x49, 0x47, 0x49, 0x4e, 
    0x0d, 0x0a, 0x78, 0x2d, 0x63, 0x61, 0x63, 0x68, 0x65, 0x2d, 
    0x69, 0x6e, 0x66, 0x6f, 0x3a, 0x20, 0x63, 0x61, 0x63, 0x68, 
    0x65, 0x64, 0x0d, 0x0a, 0x63, 0x6f, 0x6e, 0x74, 0x65, 0x6e, 
    0x74, 0x2d, 0x6c, 0x65, 0x6e, 0x67, 0x74, 0x68, 0x3a, 0x20, 
    0x32, 0x34, 0x33, 0x0d, 0x0a, 0x0d, 0x0a)), modified = structure(NA_integer_, class = c("POSIXct", 
    "POSIXt")), times = c(redirect = 0, namelookup = 8.9e-05, 
    connect = 9.4e-05, pretransfer = 3e-04, starttransfer = 0.096757, 
    total = 0.096813), content = as.raw(c(0x7b, 0x22, 0x74, 0x79, 
    0x70, 0x65, 0x22, 0x3a, 0x20, 0x22, 0x65, 0x72, 0x72, 0x6f, 
    0x72, 0x22, 0x2c, 0x20, 0x22, 0x65, 0x72, 0x72, 0x6f, 0x72, 
    0x22, 0x3a, 0x20, 0x7b, 0x22, 0x6d, 0x65, 0x73, 0x73, 0x61, 
    0x67, 0x65, 0x22, 0x3a, 0x20, 0x22, 0x52, 0x65, 0x73, 0x6f, 
    0x75, 0x72, 0x63, 0x65, 0x20, 0x6e, 0x6f, 0x74, 0x20, 0x66, 
    0x6f, 0x75, 0x6e, 0x64, 0x22, 0x2c, 0x20, 0x22, 0x64, 0x65, 
    0x74, 0x61, 0x69, 0x6c, 0x22, 0x3a, 0x20, 0x22, 0x54, 0x68, 
    0x65, 0x72, 0x65, 0x20, 0x69, 0x73, 0x20, 0x6e, 0x6f, 0x20, 
    0x41, 0x50, 0x49, 0x20, 0x68, 0x6f, 0x73, 0x74, 0x65, 0x64, 
    0x20, 0x61, 0x74, 0x20, 0x74, 0x68, 0x69, 0x73, 0x20, 0x55, 
    0x52, 0x4c, 0x2e, 0x5c, 0x6e, 0x5c, 0x6e, 0x46, 0x6f, 0x72, 
    0x20, 0x69, 0x6e, 0x66, 0x6f, 0x72, 0x6d, 0x61, 0x74, 0x69, 
    0x6f, 0x6e, 0x20, 0x61, 0x62, 0x6f, 0x75, 0x74, 0x20, 0x6f, 
    0x75, 0x72, 0x20, 0x41, 0x50, 0x49, 0x27, 0x73, 0x2c, 0x20, 
    0x70, 0x6c, 0x65, 0x61, 0x73, 0x65, 0x20, 0x72, 0x65, 0x66, 
    0x65, 0x72, 0x20, 0x74, 0x6f, 0x20, 0x74, 0x68, 0x65, 0x20, 
    0x64, 0x6f, 0x63, 0x75, 0x6d, 0x65, 0x6e, 0x74, 0x61, 0x74, 
    0x69, 0x6f, 0x6e, 0x20, 0x61, 0x74, 0x3a, 0x20, 0x68, 0x74, 
    0x74, 0x70, 0x73, 0x3a, 0x2f, 0x2f, 0x64, 0x65, 0x76, 0x65, 
    0x6c, 0x6f, 0x70, 0x65, 0x72, 0x2e, 0x61, 0x74, 0x6c, 0x61, 
    0x73, 0x73, 0x69, 0x61, 0x6e, 0x2e, 0x63, 0x6f, 0x6d, 0x2f, 
    0x62, 0x69, 0x74, 0x62, 0x75, 0x63, 0x6b, 0x65, 0x74, 0x2f, 
    0x61, 0x70, 0x69, 0x2f, 0x32, 0x2f, 0x72, 0x65, 0x66, 0x65, 
    0x72, 0x65, 0x6e, 0x63, 0x65, 0x2f, 0x22, 0x7d, 0x7d)))

remotes:::github_error(res)
#> Error in class(xx) <- cl : attempt to set an attribute on NULL

As an aside, this actually a symptom of another bug. I was installing a package from bitbucket which has aws.comprehend as a dependency with cloudyr/aws.comprehend as a remote. For some reason, it tried to install "cloudyr/aws.comprehend" from bitbucket when it is actually a github package. Github is supposed to be the default but the behaviour is actually to re-use the existing host? This causes the 404 status and the subsequent error above.

[jimhester]

The original issue seems to be fixed in the devel version now.