Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

CI: Update bash on Travis osx for hash support #99

Merged

Conversation

jodh-intel
Copy link
Contributor

The version of bash(1) provided in the Travis osx environment does
not support typeset -A, so update it using homebrew.

Fixes #98.

Signed-off-by: James O. D. Hunt james.o.hunt@intel.com

The version of `bash(1)` provided in the Travis osx environment does
not support `typeset -A`, so update it using homebrew.

Fixes kata-containers#98.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess if we are happy using homebrew...
lgtm

@caoruidong
Copy link
Member

caoruidong commented Jul 5, 2018

lgtm

Approved with PullApprove

@jodh-intel
Copy link
Contributor Author

@grahamwhaley - are you aware of issues with it?

@grahamwhaley
Copy link
Contributor

No - it's just with it being the 'unofficial package manager', and not a base part of OSX, that's all. I expect we are fine with it, and we should merge :-)

@codecov
Copy link

codecov bot commented Jul 5, 2018

Codecov Report

Merging #99 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #99   +/-   ##
=======================================
  Coverage   53.42%   53.42%           
=======================================
  Files           7        7           
  Lines         277      277           
=======================================
  Hits          148      148           
  Misses        116      116           
  Partials       13       13

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 de2d2a6...392d63c. Read the comment docs.

@jodh-intel
Copy link
Contributor Author

I came across it as its part of the standard Travis osx environment, so that might give us some level of confidence:

@jodh-intel jodh-intel merged commit d8da593 into kata-containers:master Jul 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants