Closed MikeKSmith closed 8 years ago
Hi Mike,
Thanks. I’ll take a look.
Best regards, Andy
On 03 Feb 2016, at 5:39 , Mike K Smith notifications@github.com wrote:
Used .libPaths instead of R.home(). Add code to check xpose.ini exists in each .libPath entry and use the one nearest the top of the path. Using file.path instead of paste( ... ) since it's a little more robust. Using file.exists(...) to check existence of the xpose.ini in each libPath. Then picking the one that comes first in the libPath (if there's more than one). Assumes that R looks in libPaths in order they appear in .libPath(). Usually user library location(s) first, then finally installation directory.
You can view, comment on, or merge this pull request online at:
https://github.com/andrewhooker/xpose4/pull/4 https://github.com/andrewhooker/xpose4/pull/4 Commit Summary
Change behaviour of search for xpose.ini. Use .libPaths instead of R.home(). Add code to check xpose.ini exists in each .libPath entry and use the one nearest the top of the path. File Changes
M R/xpose.data.R https://github.com/andrewhooker/xpose4/pull/4/files#diff-0 (20) Patch Links:
https://github.com/andrewhooker/xpose4/pull/4.patch https://github.com/andrewhooker/xpose4/pull/4.patch https://github.com/andrewhooker/xpose4/pull/4.diff https://github.com/andrewhooker/xpose4/pull/4.diff — Reply to this email directly or view it on GitHub https://github.com/andrewhooker/xpose4/pull/4.
Used .libPaths instead of R.home(). Add code to check xpose.ini exists in each .libPath entry and use the one nearest the top of the path. Using file.path instead of paste( ... ) since it's a little more robust. Using file.exists(...) to check existence of the xpose.ini in each libPath. Then picking the one that comes first in the libPath (if there's more than one). Assumes that R looks in libPaths in order they appear in .libPath(). Usually user library location(s) first, then finally installation directory.