Skip to content

Conversation

@mmmorris1975
Copy link
Contributor

Update apt-get command to perform cleanup, shrinks existing image by about 20MB

Use ARG instead of ENV for LIQUIBASE_VERSION. This enables you to pass the Liquibase
version at image build time and create tagged builds easily. Also avoids setting
an extra env variable during docker run. Example tagged build command:
docker build . --build-arg LIQUIBASE_VERSION=3.8.5 -t liquibase/docker:3.8.5

Simplify download logic and avoid the intermediate file

Use chmod safely (755 vs 777)

Update apt-get command to perform cleanup, shrinks existing image by about 20MB

Use ARG instead of ENV for LIQUIBASE_VERSION.  This enables you to pass the Liquibase
  version at image build time and create tagged builds easily.  Also avoids setting
  an extra env variable during docker run.  Example tagged build command:
docker build . --build-arg LIQUIBASE_VERSION=3.8.5 -t liquibase/docker:3.8.5

Simplify download logic and avoid the intermediate file

Use chmod safely (755 vs 777)
Copy link
Contributor

@robertericreeves robertericreeves left a comment

Choose a reason for hiding this comment

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

Awesome!!!

@robertericreeves robertericreeves merged commit c464d3f into liquibase:master Feb 21, 2020
@robertericreeves
Copy link
Contributor

DM me your address and we'll send you a Liquibase T-shirt!!! You can also DM @liquibase on Twitter.

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