Skip to content

GEODE-6012: Corrected package name in Dockerfile#395

Merged
jake-at-work merged 2 commits intoapache:developfrom
igodwin:GEODE-6012-fix-dockerfile-typo
Nov 9, 2018
Merged

GEODE-6012: Corrected package name in Dockerfile#395
jake-at-work merged 2 commits intoapache:developfrom
igodwin:GEODE-6012-fix-dockerfile-typo

Conversation

@igodwin
Copy link
Copy Markdown

@igodwin igodwin commented Nov 7, 2018

  • libc++api-dev (non-existent) changed to libc++abi-dev

Docker image successfully builds and geode-native is able to be built on resulting image.

- libc++api-dev (non-existent) changed to libc++abi-dev

FROM ubuntu
#FROM ubuntu
FROM ubuntu:16.04
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.

Why lock this down to an older version of ubuntu?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

That's a mistake, leftover from testing on 16.04. I've pushed the change and payed attention, this time.

@jake-at-work jake-at-work merged commit 5cb8443 into apache:develop Nov 9, 2018
@igodwin igodwin deleted the GEODE-6012-fix-dockerfile-typo branch March 13, 2019 17:40
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