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

do.call(docker, x) : second argument must be a list #65

Closed
nanxstats opened this issue Apr 14, 2018 · 1 comment
Closed

do.call(docker, x) : second argument must be a list #65

nanxstats opened this issue Apr 14, 2018 · 1 comment
Assignees

Comments

@nanxstats
Copy link
Contributor

There are some issues for using do.call in R 3.5.0 and devel branch of bioc (as of April 14, 2018) that caused devel branch build to fail. To reproduce:

> library("sevenbridges")
> requirements(docker(pull = "rocker/r-base"))

It gives

Error in do.call(docker, x) : second argument must be a list 
5. stop("second argument must be a list") 
4. do.call(docker, x) at class-cwl.R#2677
3. FUN(X[[i]], ...) 
2. lapply(listData[!idx.fd], function(x) {
    if ("class" %in% names(x)) {
        cls <- x$class
        switch(cls, DockerRequirement = { ... at class-cwl.R#2671
1. requirements(docker(pull = "rocker/r-base")) 

which should give

[[1]]
class: DockerRequirement
dockerPull: rocker/r-base

As a workaround to pass the build/check, I've commented out all hints = requirements(...) in d40b22f.

To fix this:

  1. Fix the issue in requirements(), most likely happened in its do.call() statements (x is not a list and cannot be used as args now, not sure why it could be used before).
  2. Restore all commented code blocks in commit d40b22f.
nanxstats added a commit that referenced this issue Oct 10, 2018
nanxstats added a commit that referenced this issue Oct 10, 2018
@nanxstats nanxstats assigned nanxstats and unassigned tengfei Oct 10, 2018
@nanxstats
Copy link
Contributor Author

Solved by 6371faa and 7ec80ff.

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