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.

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

feat: add manual generation of abi, runtime-capi #69

Merged
merged 1 commit into from
Jan 3, 2020

Conversation

baszalmstra
Copy link
Collaborator

When using build.rs files several files get regenerated every build
which causes rebuilds even when nothing really changed. This commit
changes this behavior to make the user manually run a command when the
files should be regenerated.

@codecov
Copy link

codecov bot commented Dec 29, 2019

Codecov Report

Merging #69 into master will increase coverage by 0.09%.
The diff coverage is 90.74%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
+ Coverage   71.83%   71.92%   +0.09%     
==========================================
  Files         106      109       +3     
  Lines        7228     7270      +42     
==========================================
+ Hits         5192     5229      +37     
- Misses       2036     2041       +5
Impacted Files Coverage Δ
crates/mun_abi/src/autogen.rs 71.57% <ø> (ø) ⬆️
crates/tools/src/abi.rs 95.83% <100%> (ø)
crates/tools/src/syntax.rs 100% <100%> (ø)
crates/tools/src/runtime_capi.rs 100% <100%> (ø)
crates/tools/src/lib.rs 75% <70.58%> (-12.76%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a3d6c70...71476b7. Read the comment docs.

@baszalmstra baszalmstra force-pushed the manual_generation branch 2 times, most recently from 63a49e4 to 5e1d322 Compare December 29, 2019 13:49
@Wodann
Copy link
Collaborator

Wodann commented Dec 29, 2019

Is it necessary to do this for the CAPI too?

@baszalmstra
Copy link
Collaborator Author

baszalmstra commented Dec 29, 2019

Is it necessary to do this for the CAPI too?

Appears to be the case yes. Without this, cargo always detects changes.

@baszalmstra baszalmstra self-assigned this Dec 29, 2019
.github/workflows/ci.yml Outdated Show resolved Hide resolved
crates/tools/src/abi.rs Outdated Show resolved Hide resolved
@baszalmstra baszalmstra force-pushed the manual_generation branch 2 times, most recently from cff4356 to 77b8e19 Compare December 29, 2019 19:58
@baszalmstra baszalmstra requested a review from Wodann December 29, 2019 19:59
@baszalmstra baszalmstra force-pushed the manual_generation branch 2 times, most recently from 686178c to e5a2cf8 Compare December 30, 2019 18:19
@Wodann
Copy link
Collaborator

Wodann commented Dec 31, 2019

Do you have any idea why the gen-runtime-capi check is failing just for ubuntu beta?

@baszalmstra
Copy link
Collaborator Author

Do you have any idea why the gen-runtime-capi check is failing just for ubuntu beta?

No, I cant reproduce it either. But Im working on it. :)

When using build.rs files several files get regenerated every build
which causes rebuilds even when nothing really changed. This commit
changes this behavior to make the user manually run a command when the
files should be regenerated.
@baszalmstra
Copy link
Collaborator Author

@Wodann Wodann added this to the Mun v0.2 milestone May 14, 2020
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.

2 participants