Skip to content

Commit 94b3b04

Browse files
committed
Fix build error messaging
1 parent 13e5ce7 commit 94b3b04

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

nimterop/build/getheader.nim

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,11 +208,12 @@ proc buildLibrary(lname, outdir, conFlags, cmakeFlags, makeFlags: string, buildT
208208
var
209209
lpath = findFile(lname, outdir, regex = true)
210210
makePath = outdir
211+
buildStatus: BuildStatus
212+
errors: seq[string]
211213

212214
if lpath.nBl:
213215
return lpath
214216

215-
var buildStatus: BuildStatus
216217

217218
for buildType in buildTypes:
218219
case buildType
@@ -223,6 +224,8 @@ proc buildLibrary(lname, outdir, conFlags, cmakeFlags, makeFlags: string, buildT
223224

224225
if buildStatus.built:
225226
break
227+
elif buildStatus.error.nBl:
228+
errors.add buildStatus.error
226229

227230
if buildStatus.buildPath.len > 0:
228231
let libraryExists = findFile(lname, buildStatus.buildPath, regex = true).len > 0
@@ -231,7 +234,7 @@ proc buildLibrary(lname, outdir, conFlags, cmakeFlags, makeFlags: string, buildT
231234
make(buildStatus.buildPath, lname, makeFlags, regex = true)
232235
buildStatus.built = true
233236

234-
let error = if buildStatus.error.len > 0: buildStatus.error else: "No build files found in " & outdir
237+
let error = if errors.len > 0: errors.join("\n") else: "No build files found in " & outdir
235238
doAssert buildStatus.built, &"\nBuild configuration failed - {error}\n"
236239

237240
result = findFile(lname, outdir, regex = true)

0 commit comments

Comments
 (0)