Skip to content

Commit 8342c27

Browse files
committed
getting better logs
1 parent 7bb6baf commit 8342c27

File tree

3 files changed

+10
-4
lines changed

3 files changed

+10
-4
lines changed

.ci/test_r_package_windows.ps1

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,8 @@ Write-Output "Done installing Rtools"
6767

6868
# MiKTeX and pandoc can be skipped on non-MINGW builds, since we don't
6969
# build the package documentation for those
70-
if ($env:COMPILER -eq "MINGW") {
70+
#if ($env:COMPILER -eq "MINGW") {
71+
if ($false) {
7172
Write-Output "Downloading MiKTeX"
7273
Download-File-With-Retries -url "https://miktex.org/download/win/miktexsetup-x64.zip" -destfile "miktexsetup-x64.zip"
7374
Add-Type -AssemblyName System.IO.Compression.FileSystem
@@ -86,6 +87,9 @@ Write-Output "Installing dependencies"
8687
$packages = "c('data.table', 'jsonlite', 'Matrix', 'processx', 'R6', 'testthat'), dependencies = c('Imports', 'Depends', 'LinkingTo')"
8788
Rscript --vanilla -e "options(install.packages.check.source = 'no'); install.packages($packages, repos = '$env:CRAN_MIRROR', type = 'binary', lib = '$env:R_LIB_PATH')" ; Check-Output $?
8889

90+
Rscript build_r.R
91+
Check-Output $false
92+
8993
Write-Output "Building R package"
9094

9195
# R CMD check is not used for MSVC builds

R-package/inst/make-r-def.R

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ IN_DLL_FILE <- args[[1L]]
1414
OUT_DEF_FILE <- args[[2L]]
1515
DLL_BASE_NAME <- basename(IN_DLL_FILE)
1616

17-
print(sprintf("Creating '%s' from '%s'", OUT_DEF_FILE, IN_DLL_FILE))
17+
message(sprintf("Creating '%s' from '%s'", OUT_DEF_FILE, IN_DLL_FILE))
1818

1919
# use objdump to dump all the symbols
2020
OBJDUMP_FILE <- "objdump-out.txt"
@@ -58,3 +58,4 @@ writeLines(
5858
, con = OUT_DEF_FILE
5959
, sep = "\n"
6060
)
61+
message(sprintf("Successfully created '%s'", OUT_DEF_FILE))

R-package/src/install.libs.R

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ if (!use_precompile) {
157157
# Must build twice for Windows due sh.exe in Rtools
158158
cmake_args <- c(cmake_args, "-G", shQuote("MinGW Makefiles"))
159159
.run_shell_command("cmake", c(cmake_args, ".."), strict = FALSE)
160-
build_cmd <- "make.exe"
160+
build_cmd <- "make"
161161
build_args <- "_lightgbm"
162162
} else {
163163
visual_studio_succeeded <- .generate_vs_makefiles(cmake_args)
@@ -166,7 +166,7 @@ if (!use_precompile) {
166166
# Must build twice for Windows due sh.exe in Rtools
167167
cmake_args <- c(cmake_args, "-G", shQuote("MinGW Makefiles"))
168168
.run_shell_command("cmake", c(cmake_args, ".."), strict = FALSE)
169-
build_cmd <- "make.exe"
169+
build_cmd <- "make"
170170
build_args <- "_lightgbm"
171171
} else {
172172
build_cmd <- "cmake"
@@ -209,6 +209,7 @@ if (!use_precompile) {
209209
}
210210

211211
# build the library
212+
message("building lib_lightgbm")
212213
.run_shell_command(build_cmd, build_args)
213214
src <- file.path(lib_folder, paste0("lib_lightgbm", SHLIB_EXT), fsep = "/")
214215

0 commit comments

Comments
 (0)