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

GEODE-8183: Update dockerfile versions #604

Merged
merged 1 commit into from May 28, 2020

Conversation

moleske
Copy link
Member

@moleske moleske commented May 24, 2020

  • Geode from 1.9 to 1.12
  • CMake from 3.12.2 to 3.12.4
  • Set to noninteractive so that building doesn't require interaction
  • Make key=value for LABEL clearer

I'm not really sure how the Travis dockerfile has stayed up to date. I'm assuming it is part of the release process to update it, but changes never get ported here since this was trying to install 1.9 which you can't download anymore.

- Geode from 1.9 to 1.12
- CMake from 3.12.2 to 3.12.4
- Set to noninteractive so that building doesn't require interaction
- Make key=value for LABEL clearer
Copy link
Contributor

@jake-at-work jake-at-work left a comment

Choose a reason for hiding this comment

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

It would be nice to fix this to use the images coming out of geode. I think there is a docker hub apache/geode:latest. We could copy geode from that image into this image rather than extracting and downloading. Heck might even be able to just layer on the clang bits to the geode base. I can't recall what its base OS is.

docker/Dockerfile Show resolved Hide resolved
@pdxcodemonkey pdxcodemonkey merged commit b309e96 into apache:develop May 28, 2020
@moleske moleske deleted the update-dockerfile branch May 28, 2020 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants