Skip to content

Make the build file slightly easier#7

Open
nsaripalli wants to merge 7 commits intoPRL-PRG:masterfrom
nsaripalli:patch-1
Open

Make the build file slightly easier#7
nsaripalli wants to merge 7 commits intoPRL-PRG:masterfrom
nsaripalli:patch-1

Conversation

@nsaripalli
Copy link
Copy Markdown

for compiling with an alternate R interpreter

for compiling with an alternate R interpreter
@@ -1,24 +1,26 @@
R := R
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mainly to help with alternate R installations one may want to install this on

Makefile Outdated
document: clean
Rscript -e 'devtools::document()'
Rscript -e 'rmarkdown::render("README.Rmd")'
$(R) --no-echo -e 'devtools::document()'
Copy link
Copy Markdown
Author

@nsaripalli nsaripalli Feb 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tried running with the -s and it seems to also do the same thing? But this is where I do not have a ton of experience. What is the difference between running Rscript and an R interpreter with the args above to run a command? Are there any differences?

Makefile Outdated
document: clean
Rscript -e 'devtools::document()'
Rscript -e 'rmarkdown::render("README.Rmd")'
$(R) --no-echo -e 'devtools::document()'
Copy link
Copy Markdown
Author

@nsaripalli nsaripalli Feb 17, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using -s (no-echo) here since --slave seems to be deprecated
wch/r-source@f1ff49e

@nsaripalli nsaripalli marked this pull request as draft February 17, 2021 01:01
@nsaripalli nsaripalli marked this pull request as ready for review February 17, 2021 01:59
removing document step on build, since it seems like it requires a gui
@nsaripalli nsaripalli marked this pull request as draft February 17, 2021 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant