Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

libpath argument is not respected during build step #195

Open
pawelru opened this issue Feb 10, 2023 · 2 comments · May be fixed by #196
Open

libpath argument is not respected during build step #195

pawelru opened this issue Feb 10, 2023 · 2 comments · May be fixed by #196
Labels
bug an unexpected problem or unintended behavior

Comments

@pawelru
Copy link

pawelru commented Feb 10, 2023

Use case:

  • package A
  • package B that has package A in its imports

Steps to reproduce:

  • create tempdir (and don't include that into the .libPath)
  • install package A in tempdir
  • rcmdcheck::rcmdcheck(<path to B source code>, libpath = <tempdir with A pre-installed>)

Things are correct if I add my tempdir into .libPath so it's clear that the build step is looking into the global .libPath instead of libpath argument.

Here's my output:

── R CMD build ────────────────────────────────────────────────────────────────────
   OpenBLAS WARNING - could not determine the L2 cache size on this system, assuming 256k
✔  checking for file ‘.../DESCRIPTION’ (388ms)
─  preparing ‘teal.code’: (29.9s)
✔  checking DESCRIPTION meta-information ... OK
─  installing the package to build vignettes
         -----------------------------------
   OpenBLAS WARNING - could not determine the L2 cache size on this system, assuming 256k
   ERROR: dependency ‘teal.widgets’ is not available for package ‘teal.code’
─  removing ‘/tmp/RtmpNUkapZ/Rinste2442094e6ba/teal.code’
         -----------------------------------
   ERROR: package installation failed
Error in proc$get_built_file() : Build process failed

Plus some debug checks:

> "teal.widgets" %in% installed.packages()
[1] FALSE
> "teal.widgets" %in% installed.packages(lib.loc = libpath)
[1] TRUE
@gaborcsardi gaborcsardi added the bug an unexpected problem or unintended behavior label Feb 10, 2023
@gaborcsardi
Copy link
Member

PR is welcome, I think there is a chance that it just needs passing the argument.

@pawelru pawelru linked a pull request Feb 15, 2023 that will close this issue
@pawelru
Copy link
Author

pawelru commented Feb 15, 2023

@gaborcsardi PR ready 🚀 - please have a look!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug an unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants