Skip to content

Update containerd/continuity to fix ARM 32-bit builds - #36339

Merged
AkihiroSuda merged 1 commit into
moby:masterfrom
thaJeztah:bump-continuity
Feb 17, 2018
Merged

AkihiroSuda merged 1 commit into
moby:masterfrom
thaJeztah:bump-continuity

Conversation

@thaJeztah

@thaJeztah thaJeztah commented Feb 16, 2018 •

Copy link
Copy Markdown
Member

This updates the containerd/continuity package to containerd/continuity@d876b2b,
which fixes builds failing on ARM 32-bit, after this dependency was added in
b3aab5e (#36290)

Full diff: containerd/continuity@992a5f1...d8fb858

Changes:

ping @seemethere @cpuguy83 @dnephin

@cpuguy83 cpuguy83 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah

Copy link
Copy Markdown
Member Author

hold on, updating with one more commit per @tiborvass's request 😄

This updates the containerd/continuity package to d8fb8589b0e8e85b8c8bbaa8840226d0dfeb7371
which fixes builds failing on ARM 32-bit, after this dependency was added in
b3aab5e

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah

Copy link
Copy Markdown
Member Author

and updated

@seemethere seemethere left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@seemethere seemethere left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@codecov

codecov Bot commented Feb 17, 2018 •

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (master@747c163). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #36339   +/-   ##
=========================================
  Coverage          ?   34.27%           
=========================================
  Files             ?      609           
  Lines             ?    45287           
  Branches          ?        0           
=========================================
  Hits              ?    15523           
  Misses            ?    27816           
  Partials          ?     1948

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants