Skip to content

Conversation

townsend2010
Copy link
Contributor

Fixes #1368

@multipass-ci-bot

This comment has been minimized.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from 19aa11c to e1ab4f2 Compare July 27, 2020 14:07
@multipass-ci-bot

This comment has been minimized.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from e1ab4f2 to 5e1512d Compare July 28, 2020 15:58
@multipass-ci-bot

This comment has been minimized.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from 5e1512d to ea3b0eb Compare July 29, 2020 16:11
@multipass-ci-bot

This comment has been minimized.

@codecov
Copy link

codecov bot commented Jul 29, 2020

Codecov Report

Merging #1659 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1659   +/-   ##
=======================================
  Coverage   76.23%   76.23%           
=======================================
  Files         227      227           
  Lines        8328     8328           
=======================================
  Hits         6349     6349           
  Misses       1979     1979           
Impacted Files Coverage Δ
src/cert/biomem.h 100.00% <ø> (ø)
src/cert/biomem.cpp 90.90% <100.00%> (ø)

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 5c951a2...530756c. Read the comment docs.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from ea3b0eb to 3ccad79 Compare July 31, 2020 16:50
@multipass-ci-bot

This comment has been minimized.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from 3ccad79 to a36cfea Compare July 31, 2020 17:59
@townsend2010 townsend2010 marked this pull request as ready for review July 31, 2020 18:32
@multipass-ci-bot

This comment has been minimized.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from a36cfea to 071648a Compare September 1, 2020 13:15
@multipass-ci-bot

This comment has been minimized.

@Saviq
Copy link
Collaborator

Saviq commented Sep 10, 2020

You can remove the architectures clause from snapcraft.yaml:

https://code.launchpad.net/~saviq/+snap/multipass-all
image

@townsend2010
Copy link
Contributor Author

You can remove the architectures clause from snapcraft.yaml:

Ah, right, cool!

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from 071648a to 413c68f Compare September 10, 2020 13:59
@multipass-ci-bot

This comment has been minimized.

@townsend2010 townsend2010 force-pushed the use-openssl-not-boringssl branch from 413c68f to 530756c Compare September 11, 2020 15:36
@multipass-ci-bot
Copy link
Collaborator

macOS build available: multipass-1.5.0-dev.173.pr1659+g995f13ae.mac-Darwin.pkg

@Saviq
Copy link
Collaborator

Saviq commented Sep 14, 2020

Noice!

bors merge

bors bot added a commit that referenced this pull request Sep 14, 2020
1659: [build] Use OpenSSL instead of gRPC's boringssl r=Saviq a=townsend2010

Fixes #1368 

Co-authored-by: Chris Townsend <[email protected]>
@bors
Copy link
Contributor

bors bot commented Sep 14, 2020

Build failed:

@Saviq
Copy link
Collaborator

Saviq commented Sep 14, 2020

Hum. Try again?

bors merge

@bors
Copy link
Contributor

bors bot commented Sep 14, 2020

Build failed:

@Saviq Saviq merged commit 684fb4b into master Sep 14, 2020
@bors bors bot deleted the use-openssl-not-boringssl branch September 14, 2020 15:41
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.

[snap] use Ubuntu's OpenSSL packages
3 participants