wjones127 commented on a change in pull request #11644:
URL: https://github.com/apache/arrow/pull/11644#discussion_r750602681
##########
File path: dev/tasks/r/github.devdocs.yml
##########
@@ -59,10 +61,22 @@ jobs:
env:
LIBARROW_BINARY: FALSE
ARROW_R_DEV: TRUE
- run: bash arrow/r/vignettes/script.sh
+ DEVDOCS_WINDOWS: {{ "${{contains(matrix.os, 'windows')}}" }}
+ run: |
+ if [ $DEVDOCS_WINDOWS == "true" ]; then
+ # This starts a special mingw64 Git Bash shell
+ $RTOOLS40_HOME/msys2_shell.cmd -here -mingw64 -no-start -defterm
-full-path arrow/r/vignettes/script.sh
+ else
+ bash arrow/r/vignettes/script.sh
+ fi
shell: bash
- name: Ensure that the Arrow package is loadable and we have the
correct one
+ env:
+ DEVDOCS_WINDOWS: {{ "${{contains(matrix.os, 'windows')}}" }}
run: |
+ if [ $DEVDOCS_WINDOWS == "true" ]; then
+ export PATH=/c/R/bin:$PATH
+ fi
Review comment:
Okay yeah, I just did the cygwin conversion there. Also needed to wrap
the path in quotes because of the spaces in Windows paths.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
