Skip to content

Call get_compiler_cmd() at the beginning of create_sysimage and create_app - #770

Merged
DilumAluthge merged 3 commits into
masterfrom
dpa/compiler
Jan 27, 2023
Merged

Call get_compiler_cmd() at the beginning of create_sysimage and create_app#770
DilumAluthge merged 3 commits into
masterfrom
dpa/compiler

Conversation

@DilumAluthge

@DilumAluthge DilumAluthge commented Jan 26, 2023

Copy link
Copy Markdown
Member

That way, users will immediately get an error if a compiler cannot be found, instead of having to wait a long time before the error is thrown.

Replaces #769
Closes #769
Closes #677
Closes #727

@codecov

codecov Bot commented Jan 26, 2023

Copy link
Copy Markdown

Codecov Report

Merging #770 (89caaef) into master (05f6854) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #770      +/-   ##
==========================================
+ Coverage   81.54%   81.59%   +0.05%     
==========================================
  Files           3        3              
  Lines         661      663       +2     
==========================================
+ Hits          539      541       +2     
  Misses        122      122              
Impacted Files Coverage Δ
src/PackageCompiler.jl 91.36% <100.00%> (+0.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@DilumAluthge

Copy link
Copy Markdown
Member Author

I'm not sure what the current state of CI is.

@IanButterworth Are these CI failures expected?

Comment thread src/PackageCompiler.jl
@IanButterworth

Copy link
Copy Markdown
Member

1.6 won't pass. The rest should but there are gremlins

Comment thread src/PackageCompiler.jl Outdated
Comment thread src/PackageCompiler.jl Outdated
Co-authored-by: Ian Butterworth <i.r.butterworth@gmail.com>
@DilumAluthge DilumAluthge added bug Reproducible incorrect behavior in PackageCompiler merge me and removed bug Reproducible incorrect behavior in PackageCompiler labels Jan 27, 2023
@DilumAluthge
DilumAluthge merged commit 4ffa03a into master Jan 27, 2023
@DilumAluthge
DilumAluthge deleted the dpa/compiler branch January 27, 2023 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check c++ compiler earlier Fail early if no compiler can be found?

2 participants