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

knitr seems to be required, not optional #2

Closed
barracuda156 opened this issue Nov 9, 2023 · 3 comments
Closed

knitr seems to be required, not optional #2

barracuda156 opened this issue Nov 9, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@barracuda156
Copy link

Is knitr required, despite a claim of DESCRIPTION?

  /opt/local/bin/clang++-mp-15 -std=gnu++17 -dynamiclib -Wl,-headerpad_max_install_names -undefined dynamic_lookup -L/opt/local/Library/Frameworks/R.framework/Resources/lib -L/opt/local/lib -Wl,-headerpad_max_install_names -Wl,-rpath,/opt/local/lib/libgcc -Wl,-syslibroot,/Library/Developer/CommandLineTools/SDKs/MacOSX13.sdk -arch x86_64 -o bvhar.so RcppExports.o design.o estimate-bvar.o estimate-hierarchical.o estimate-ssvs.o estimate-sv.o estimate-var.o estimate-vhar.o forecast-bvar.o forecast-bvhar.o forecast-expand.o forecast-roll.o forecast-var.o forecast-vhar.o generate-distn.o is-stable.o mcmc-draw.o misc-eigen.o simulate-data.o tuning-models.o -fopenmp -L/opt/local/Library/Frameworks/R.framework/Resources/lib -lRlapack -L/opt/local/Library/Frameworks/R.framework/Resources/lib -lRblas -L/opt/local/lib/gcc12/gcc/x86_64-apple-darwin22/12.3.0 -L/opt/local/lib/gcc12 -lgfortran -lquadmath -F/opt/local/Library/Frameworks/R.framework/.. -framework R -Wl,-framework -Wl,CoreFoundation
  installing to /opt/local/var/macports/build/_Users_runner_work_macports-ports_macports-ports_ports_R_R-bvhar/R-bvhar/work/destroot/opt/local/Library/Frameworks/R.framework/Versions/4.3/Resources/library/00LOCK-bvhar-1.0.0/00new/bvhar/libs
  ** R
  ** data
  *** moving datasets to lazyload DB
  ** inst
  ** tests
  ** byte-compile and prepare package for lazy loading
  Error in loadNamespace(name) : there is no package called ‘knitr’
  Error: unable to load R code in package ‘bvhar’
  Execution halted
  ERROR: lazy loading failed for package ‘bvhar’
  * removing ‘/opt/local/var/macports/build/_Users_runner_work_macports-ports_macports-ports_ports_R_R-bvhar/R-bvhar/work/destroot/opt/local/Library/Frameworks/R.framework/Versions/4.3/Resources/library/bvhar’
  Command failed:  cd "/opt/local/var/macports/build/_Users_runner_work_macports-ports_macports-ports_ports_R_R-bvhar/R-bvhar/work/bvhar-1.0.0" && /opt/local/bin/R CMD INSTALL . --library=/opt/local/var/macports/build/_Users_runner_work_macports-ports_macports-ports_ports_R_R-bvhar/R-bvhar/work/destroot/opt/local/Library/Frameworks/R.framework/Versions/4.3/Resources/library --install-tests 
  Error: Failed to destroot R-bvhar: command execution failed
@ygeunkim
Copy link
Owner

It might be because of using S3method for knit_print - should have been in NAMESPACE - in a wrong way. I will fix it ASAP using @exportS3Method of roxygen2.

Thanks!

ygeunkim added a commit that referenced this issue Nov 10, 2023
ygeunkim added a commit that referenced this issue Nov 10, 2023
ygeunkim added a commit that referenced this issue Nov 10, 2023
@ygeunkim
Copy link
Owner

Commit cd9301b fix this issue. Also I have submitted v1.0.1 to CRAN. I will create the corresponding release after CRAN update.

@barracuda156
Copy link
Author

Thank you!

ygeunkim added a commit that referenced this issue Nov 11, 2023
@ygeunkim ygeunkim added the bug Something isn't working label Mar 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants