-
Notifications
You must be signed in to change notification settings - Fork 59
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
pkg_deps gets wrong dependencies (for past versions) #663
Comments
Seems correct to me: ❯ pak::pkg_deps("[email protected]")
# A data frame: 3 × 32
ref type direct directpkg status package version license needscompilation
<chr> <chr> <lgl> <lgl> <chr> <chr> <chr> <chr> <lgl>
1 Balanc… stan… TRUE TRUE OK Balanc… 1.6.3 GPL (>… TRUE
2 Rcpp stan… FALSE FALSE OK Rcpp 1.0.12 GPL (>… FALSE
3 Sampli… stan… FALSE FALSE OK Sampli… 1.0.0 GPL (>… FALSE
# ℹ 23 more variables: priority <chr>, md5sum <chr>, sha256 <chr>,
# filesize <int>, built <chr>, platform <chr>, rversion <chr>,
# repotype <chr>, repodir <chr>, target <chr>, deps <list>, mirror <chr>,
# sources <list>, remote <list>, error <list>, metadata <list>,
# dep_types <list>, params <list>, sysreqs <chr>, cache_status <chr>,
# lib_status <chr>, old_version <chr>, new_version <chr> ❯ pak::pkg_deps("[email protected]")$deps[[1]]
# A data frame: 3 × 5
ref type package op version
<chr> <chr> <chr> <chr> <chr>
1 Rcpp Imports Rcpp ">=" "0.11.1"
2 SamplingBigData Imports SamplingBigData "" ""
3 Rcpp LinkingTo Rcpp "" "" |
That's not a minimum requirement, |
So, the reported version in the column labelled "version" in the returned The documentation is ambiguous about this:
Why would it return "dependency data" in the |
|
Struggling with several issues in
pak
recently. This one seems like a simple bug to recreate. The code below requests specific versions of the packageBalancedSampling
. It showsRcpp
has a minimum version, which appears to be incorrect based on the DESCRIPTION file in the respective archives. The version1.6.3
does not haveRcpp >= 1.0.12
butpkg_dep
says it does.The text was updated successfully, but these errors were encountered: