Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[R-package] Fallback to MinGW when VS build fails #664

Merged
merged 15 commits into from
Jul 13, 2017
Merged

[R-package] Fallback to MinGW when VS build fails #664

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
70b1bc6
Change VS -> MinGW for CRAN
Laurae2 Jun 30, 2017
7570715
Documentation to switch to MinGW by default
Laurae2 Jun 30, 2017
e2aead2
Merge branch 'master' into patch-2
Laurae2 Jul 6, 2017
2c5ea8e
Force cmake to run twice
Laurae2 Jul 6, 2017
4098fc1
Try again dual build for Rtools
Laurae2 Jul 6, 2017
48f8f44
Switch to cmake for building twice
Laurae2 Jul 6, 2017
ed00f7b
Try with Visual Studio as default, MinGW as fallback
Laurae2 Jul 13, 2017
2914c1f
Try to remove VS appropriately when failing
Laurae2 Jul 13, 2017
52276e6
Attempt to get rid of build folder first
Laurae2 Jul 13, 2017
9557b61
Switch to unlink from rm rf
Laurae2 Jul 13, 2017
6a4c23c
Change lib_folder correctly when VS fails
Laurae2 Jul 13, 2017
31bbe84
Add README updates from Microsoft/LightGBM@1c8355c
Laurae2 Jul 13, 2017
6bcc4b0
Merge branch 'master' into patch-2
Laurae2 Jul 13, 2017
91bcdb3
Update default compiler doc and add GPU online install doc
Laurae2 Jul 13, 2017
9ccac03
Better GPU doc
Laurae2 Jul 13, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions R-package/src/install.libs.R
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ if (!use_precompile) {
cmake_cmd <- paste0(cmake_base, " -DCMAKE_GENERATOR_PLATFORM=x64 ")
tryVS <- system(paste0(cmake_cmd, " .."))
if (tryVS == 1) {
system("rm -rf .")
Copy link
Collaborator

Choose a reason for hiding this comment

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

not sure the rm -rf can work in windows or not. you can use unlink("./*", recursive = TRUE)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It works with your method.

rm rf works, but I was denied access to remove . or .. apparently.

cmake_cmd <- paste0(cmake_base, " -G \"MinGW Makefiles\" ") # Switch to MinGW on failure, try build once
Copy link
Collaborator

Choose a reason for hiding this comment

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

It is better to remove all cmake caches if last camke is fail. you can delete all contents in build folder.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do you know where is the build folder in all the folders? I can't seem to get rid of it.

Objective would be:

  • try VS build
  • if VS build fails, try MinGW build but delete build folder first (I don't know where is the build folder in all the folders)

system(paste0(cmake_cmd, " ..")) # Must build twice for Windows due sh.exe in Rtools
}
Expand Down