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

0.8 install fixes #945

Merged
merged 5 commits into from
Mar 12, 2020
Merged

0.8 install fixes #945

merged 5 commits into from
Mar 12, 2020

Conversation

jumaffre
Copy link
Contributor

We had two issues with our 0.8 install:

  • The generated/ folder (containing the EDL definitions) was no longer copied to our install directory but was still referenced as an include directory for all our targets.
  • The new networkencryption.h file (included by our apps via serialization.h) included cryptobox.h, which includes some evercrypt that apps do not included (yet?).

Once this is merged, I will cut 0.8.1.

@jumaffre jumaffre requested a review from a team as a code owner March 12, 2020 13:44
@ghost
Copy link

ghost commented Mar 12, 2020

fix_0.8_install@5978 aka 20200312.18 vs master ewma over 30 builds from 5654 to 5972
images

src/node/networkencryption.h Outdated Show resolved Hide resolved
@codecov-io
Copy link

codecov-io commented Mar 12, 2020

Codecov Report

Merging #945 into master will increase coverage by 0.09%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #945      +/-   ##
==========================================
+ Coverage   68.46%   68.55%   +0.09%     
==========================================
  Files         103      103              
  Lines        8076     8073       -3     
==========================================
+ Hits         5529     5534       +5     
+ Misses       2547     2539       -8
Flag Coverage Δ
#unit_BFT 68.54% <50%> (+0.07%) ⬆️
#unit_CFT 68.54% <50%> (+0.07%) ⬆️
Impacted Files Coverage Δ
src/node/networkencryption.h 66.67% <50%> (+33.33%) ⬆️
src/ds/thread_messaging.h 25.58% <0%> (+11.63%) ⬆️

@jumaffre jumaffre merged commit 49c7800 into microsoft:master Mar 12, 2020
eddyashton pushed a commit to eddyashton/CCF that referenced this pull request Mar 24, 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.

4 participants