Navigation Menu

Skip to content

Commit

Permalink
Fix a bug in the last commit, that could cause configure errors
Browse files Browse the repository at this point in the history
Previous commit added a line to print out errors that wouldn't
get printed out otherwise if you pass in --ignore-errors.
Doing this though would cause builds without --ignore-errors to
fail as often the messages may not be fatal messages. Correct this.
  • Loading branch information
samcv committed Aug 26, 2017
1 parent c368470 commit e5e5a3f
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion Configure.pl
Expand Up @@ -183,7 +183,9 @@
unshift @errors, @moar_errors if @moar_errors;
}
sorry($options{'ignore-errors'}, @errors) if @errors;
sorry($options{'ignore-errors'}, @moar_errors) if @moar_errors;
# If we ignore errors, normally we'd print out the @moar_errors elsewhere
# so make sure to print them out now. Don't print unless errors are ignored
print join("\n", '', @moar_errors, "\n") if @moar_errors and $options{'ignore-errors'};
$config{'make'} = `$moar_path --libpath="src/vm/moar/stage0" "src/vm/moar/stage0/nqp.moarvm" -e "print(nqp::backendconfig()<make>)"`
|| 'make';
$config{moar} = $moar_path;
Expand Down

0 comments on commit e5e5a3f

Please sign in to comment.