-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
[R-package] Fix build_r.R failing in Windows #1694
Conversation
build_r.R
Outdated
@@ -1,4 +1,4 @@ | |||
# for macOS users who have decided to use gcc | |||
# Fr macOS users who have decided to use gcc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For
build_r.R
Outdated
) | ||
result <- file.copy(from = "src/", | ||
to = file.path("lightgbm_r", "src/") | ||
, recursive = TRUE, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two ,
at one line
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL recursive=TRUE can cause issues. Thanks for the fix!
) | ||
result <- file.copy(from = "CMakeLists.txt", | ||
to = file.path("lightgbm_r", "inst", "bin/"), | ||
overwrite = TRUE) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this what was causing the problem? (looks like you removed the recursive = TRUE
)
A lot of these are formatting changes, so trying to find whichever thing was causing the issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had two errors:
- on
devtools::document(pkg = "lightgbm_r/")
(because of the trailing slash) - on the file with recursive (it causes a warning, which causes an error when R is setup to throw an error on warnings)
@jameslamb Do you have the rights to delete the branch fix-#1693 ? It seems I cannot delete it anymore (getting GitHub 404 error). |
@Laurae2 I've removed it: |
Thanks @StrikerRUS ! |
Fixes #1693