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

Double load_all() output #1877

Closed
lionel- opened this issue Sep 26, 2018 · 5 comments
Closed

Double load_all() output #1877

lionel- opened this issue Sep 26, 2018 · 5 comments

Comments

@lionel-
Copy link
Member

lionel- commented Sep 26, 2018

I just updated to last github devtools and it seems load_all() double installs packages:

Loading tidyselect
Re-compiling tidyselect
Running /Library/Frameworks/R.framework/Resources/bin/R CMD INSTALL \
  /Users/lionel/Dropbox/Projects/R/tidyverse/tidyselect \
  '--library=/var/folders/b9/1vbq6rn93_1fk71sn95dqb8r0000gn/T//RtmpePst0L/devtools_install_f0c24439234' \
  --no-R --no-data --no-help --no-demo --no-inst --no-docs --no-exec \
  --no-multiarch --no-test-load
* installing *source* package ‘tidyselect’ ...
   * installing *source* package ‘tidyselect’ ...
** libs
   ** libs
g++-8 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG  -I"/Users/lionel/R/library/3.5/Rcpp/include" -I/usr/local/include   -fPIC  -arch x86_64 -ftemplate-depth-256 -Wall -pedantic -O0 -c RcppExports.cpp -o RcppExports.o
   g++-8 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG  -I"/Users/lionel/R/library/3.5/Rcpp/include" -I/usr/local/include   -fPIC  -arch x86_64 -ftemplate-depth-256 -Wall -pedantic -O0 -c RcppExports.cpp -o RcppExports.o
g++-8 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG  -I"/Users/lionel/R/library/3.5/Rcpp/include" -I/usr/local/include   -fPIC  -arch x86_64 -ftemplate-depth-256 -Wall -pedantic -O0 -c combine_variables.cpp -o combine_variables.o
   g++-8 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG  -I"/Users/lionel/R/library/3.5/Rcpp/include" -I/usr/local/include   -fPIC  -arch x86_64 -ftemplate-depth-256 -Wall -pedantic -O0 -c combine_variables.cpp -o combine_variables.o
g++-8 -dynamiclib -Wl,-headerpad_max_install_names -undefined dynamic_lookup -single_module -multiply_defined suppress -L/Library/Frameworks/R.framework/Resources/lib -L/usr/local/lib -o tidyselect.so RcppExports.o combine_variables.o -F/Library/Frameworks/R.framework/.. -framework R -Wl,-framework -Wl,CoreFoundation
   g++-8 -dynamiclib -Wl,-headerpad_max_install_names -undefined dynamic_lookup -single_module -multiply_defined suppress -L/Library/Frameworks/R.framework/Resources/lib -L/usr/local/lib -o tidyselect.so RcppExports.o combine_variables.o -F/Library/Frameworks/R.framework/.. -framework R -Wl,-framework -Wl,CoreFoundation
installing to /private/var/folders/b9/1vbq6rn93_1fk71sn95dqb8r0000gn/T/RtmpePst0L/devtools_install_f0c24439234/tidyselect/libs
   installing to /private/var/folders/b9/1vbq6rn93_1fk71sn95dqb8r0000gn/T/RtmpePst0L/devtools_install_f0c24439234/tidyselect/libs
* DONE (tidyselect)
   * DONE (tidyselect)

I got similar results with devtools::install(). In this case it went away after installing github r-lib/xopen and reinstalling github devtools. It didn't fix load_all().

@lionel-
Copy link
Member Author

lionel- commented Sep 26, 2018

Not sure why but devtools::install_github("r-lib/devtools) didn't upgrade rcmdcheck even though the Imports depends on >= 1.3.0. devtools::check() didn't work properly until I manually installed the new version. It is possible my devtools installation is entirely borked.

@jimhester
Copy link
Member

So it is not being double-installed, but the output is being doubled, likely from recent changes to pkgbuild, I will track it down.

@jimhester
Copy link
Member

Thanks, should now be fixed.

@lionel-
Copy link
Member Author

lionel- commented Sep 27, 2018

Thanks, confirmed. The new output looks great.

@lock
Copy link

lock bot commented Mar 26, 2019

This old issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with reprex) and link to this issue. https://reprex.tidyverse.org/

@lock lock bot locked and limited conversation to collaborators Mar 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants