{"action":"opened","issue":{"url":"https://api.github.com/repos/hadley/devtools/issues/1254","repository_url":"https://api.github.com/repos/hadley/devtools","labels_url":"https://api.github.com/repos/hadley/devtools/issues/1254/labels{/name}","comments_url":"https://api.github.com/repos/hadley/devtools/issues/1254/comments","events_url":"https://api.github.com/repos/hadley/devtools/issues/1254/events","html_url":"https://github.com/hadley/devtools/issues/1254","id":163929608,"number":1254,"title":"biocInstaller necessary if biocViews in DESCRIPTION but not a bioc Package","user":{"login":"muschellij2","id":1075118,"avatar_url":"https://avatars.githubusercontent.com/u/1075118?v=3","gravatar_id":"","url":"https://api.github.com/users/muschellij2","html_url":"https://github.com/muschellij2","followers_url":"https://api.github.com/users/muschellij2/followers","following_url":"https://api.github.com/users/muschellij2/following{/other_user}","gists_url":"https://api.github.com/users/muschellij2/gists{/gist_id}","starred_url":"https://api.github.com/users/muschellij2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/muschellij2/subscriptions","organizations_url":"https://api.github.com/users/muschellij2/orgs","repos_url":"https://api.github.com/users/muschellij2/repos","events_url":"https://api.github.com/users/muschellij2/events{/privacy}","received_events_url":"https://api.github.com/users/muschellij2/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2016-07-05T19:41:30Z","updated_at":"2016-07-05T19:41:30Z","closed_at":null,"body":"This issue arose in the `matrixStats` package (https://github.com/HenrikBengtsson/matrixStats), with associated issue opened there @HenrikBengtsson:\r\nhttps://github.com/HenrikBengtsson/matrixStats/issues/88\r\n\r\nOverall, the \"issue\" is that in utils https://github.com/hadley/devtools/blob/master/R/utils.r#L155:\r\n```r\r\nis_bioconductor <- function(x) {\r\n  x$package != \"BiocInstaller\" && !is.null(x$biocviews)\r\n}\r\n```\r\nImplies that if biocViews exists and the package is not `BiocInstaller`, then the package is assumed bioconductor package.  This is probably a good assumption or one of the only ways to distinguish the CRAN/bioC package distinction.  Thus, though not an `Imports` or `Suggests`, `BiocInstaller` seems to be a hidden/implicit dependency when installing using `devtools`.  This affects Travis builds.  \r\n\r\nThis may not be an \"issue\" exactly, and I may just add an install of `BiocInstaller` to the Travis to make it work, but figured I'd bring up the point here."}}
{"actor":{"display_login":"muschellij2"}}