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

Use /usr/bin/env to make the scripts more portable. #21

Merged
merged 1 commit into from Jun 16, 2016

Conversation

Projects
None yet
2 participants
@cperl82
Contributor

cperl82 commented Jun 16, 2016

In addition, remove --script, as I believe that is not necessary when
using Rscript instead of R. In my tests, they produced the exact same
output, but with these changes I was able to run the R scripts directly
instead of needing to pass them as an argument to Rscript, as my Rscript
is at /usr/bin/Rscript.

Use /usr/bin/env to make the scripts more portable.
In addition, remove --script, as I believe that is not necessary when
using Rscript instead of R.  In my tests, they produced the exact same
output, but with these changes I was able to run the R scripts directly
instead of needing to pass them as an argument to Rscript, as my Rscript
is at /usr/bin/Rscript.

@gvnn3 gvnn3 merged commit cb5968c into ptpd:master Jun 16, 2016

@cperl82 cperl82 deleted the cperl82:env-for-rscript branch Jun 16, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment