[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