[ARVADOS] updated: 1.1.3-270-gbe671e4

Git user git at public.curoverse.com
Thu Apr 19 14:26:49 EDT 2018


Summary of changes:
 build/run-tests.sh                  |  8 ++++----
 doc/Rakefile                        |  4 ++--
 doc/sdk/R/index.html.textile.liquid |  2 +-
 sdk/R/install_deps.R                | 11 ++++-------
 4 files changed, 11 insertions(+), 14 deletions(-)

       via  be671e455b11f50af965923b980977264a8d7364 (commit)
      from  274124d082f7268fbe3b3d5e0608a3f4b6ae68ac (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit be671e455b11f50af965923b980977264a8d7364
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Thu Apr 19 14:24:11 2018 -0400

    13076: Get rid of pkgdown, just call knitr
    
    Converts README.Rmd to README.html for documentation.
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/build/run-tests.sh b/build/run-tests.sh
index b89c8d9..f567cf4 100755
--- a/build/run-tests.sh
+++ b/build/run-tests.sh
@@ -246,9 +246,9 @@ sanity_checks() {
     if [[ "$NEED_SDK_R" = true ]]; then
       # R SDK stuff
       echo -n 'R: '
-      which R || fatal "No R. Try: apt-get install r-base"
+      which Rscript || fatal "No Rscript. Try: apt-get install r-base"
       echo -n 'testthat: '
-      R -q -e "library('testthat')" || fatal "No testthat. Try: apt-get install r-cran-testthat"
+      Rscript -e "library('testthat')" || fatal "No testthat. Try: apt-get install r-cran-testthat"
       # needed for roxygen2, needed for devtools, needed for R sdk
       pkg-config --exists libxml-2.0 || fatal "No libxml2. Try: apt-get install libxml2-dev"
       # needed for pkgdown, builds R SDK doc pages
@@ -802,7 +802,7 @@ do_install sdk/ruby ruby_sdk
 install_R_sdk() {
   if [[ "$NEED_SDK_R" = true ]]; then
     cd "$WORKSPACE/sdk/R" \
-       && R --quiet --vanilla --file=install_deps.R
+       && Rscript --vanilla install_deps.R
   fi
 }
 do_install sdk/R R_sdk
@@ -981,7 +981,7 @@ do_test sdk/ruby ruby_sdk
 test_R_sdk() {
   if [[ "$NEED_SDK_R" = true ]]; then
     cd "$WORKSPACE/sdk/R" \
-        && R --quiet --file=run_test.R
+        && Rscript --vanilla run_test.R
   fi
 }
 
diff --git a/doc/Rakefile b/doc/Rakefile
index eaa5410..4870eb1 100644
--- a/doc/Rakefile
+++ b/doc/Rakefile
@@ -29,10 +29,10 @@ file "sdk/R/arvados/index.html" do |t|
   `which R`
   if $? == 0
     Dir.chdir("../sdk/R/") do
-      STDERR.puts `R --quiet --vanilla -e 'pkgdown::build_site()' 2>&1`
+      STDERR.puts `Rscript createDoc.R README.Rmd index.html 2>&1`
     end
     raise if $? != 0
-    cp_r("../sdk/R/docs", "sdk/R/arvados")
+    cp_r("../sdk/R/README.html", "sdk/R/")
   else
     puts "Warning: R not found, R documentation will not be generated".colorize(:light_red)
   end
diff --git a/doc/sdk/R/index.html.textile.liquid b/doc/sdk/R/index.html.textile.liquid
index 7b788a1..1d7803a 100644
--- a/doc/sdk/R/index.html.textile.liquid
+++ b/doc/sdk/R/index.html.textile.liquid
@@ -12,4 +12,4 @@ Copyright (C) The Arvados Authors. All rights reserved.
 SPDX-License-Identifier: CC-BY-SA-3.0
 {% endcomment %}
 
-notextile. <iframe src="arvados/" style="width:100%; height:100%; border:none" />
+notextile. <iframe src="README.html" style="width:100%; height:100%; border:none" />
diff --git a/sdk/R/install_deps.R b/sdk/R/install_deps.R
index a54a9a2..5314c86 100644
--- a/sdk/R/install_deps.R
+++ b/sdk/R/install_deps.R
@@ -5,14 +5,11 @@ if (!requireNamespace("devtools")) {
 if (!requireNamespace("roxygen2")) {
   install.packages("roxygen2")
 }
-
-# These install from github so install known-good versions instead of
-# letting any push to master break our build.
-if (!requireNamespace("pkgload")) {
-  devtools::install_github("r-lib/pkgload", ref="7a97de62adf1793c03e73095937e4655baad79c9")
+if (!requireNamespace("knitr")) {
+  install.packages("knitr")
 }
-if (!requireNamespace("pkgdown")) {
-  devtools::install_github("r-lib/pkgdown", ref="897ffbc016549c11c4263cb5d1f6e9f5c99efb45")
+if (!requireNamespace("markdown")) {
+  install.packages("markdown")
 }
 
 devtools::install_dev_deps()

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list