Skip to content

Add Dockerfile #3455

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

Merged
merged 2 commits into from
Dec 9, 2020
Merged

Add Dockerfile #3455

merged 2 commits into from
Dec 9, 2020

Conversation

norio-nomura
Copy link
Contributor

@norio-nomura norio-nomura commented Dec 4, 2020

  • providing a runtime environment on ubuntu that does not include the Swift toolchain.
  • supportintg docker build https://github.com/realm/SwiftLint.git
  • adding libcurl4-openssl-dev as build time dependency that introduced by Add remote, parent & child configuration features #3058
  • move the build-time dependency information from Makefile to Dockerfile.

Provides:
- a runtime environment on ubuntu that does not include the Swift toolchain.
- supportintg `docker build https://github.com/realm/SwiftLint.git`
- add `libcurl4-openssl-dev` as build time dependency that introduced by #3058
Hiding dependencies required at build time in the Dockerfile
@norio-nomura norio-nomura requested a review from jpsim December 4, 2020 15:30
@norio-nomura
Copy link
Contributor Author

Might need a CI to build Dockerfile?

@jpsim
Copy link
Collaborator

jpsim commented Dec 4, 2020

Might need a CI to build Dockerfile?

Yes I think that'd be worthwhile, would've caught the change from #3058

Copy link
Collaborator

@jpsim jpsim left a comment

Choose a reason for hiding this comment

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

A CI job to validate this would be great. Thanks!

@norio-nomura
Copy link
Contributor Author

Is there any particular reason not to migrate from Azure Pipelines to GitHub Actions?
Seeing that Azure Pipelines VM images for Microsoft-hosted CI/CD has been replaced by GitHub Actions Virtual Environments, there seems to me no reason to continue using Azure Pipelines.

@jpsim
Copy link
Collaborator

jpsim commented Dec 6, 2020

Migrating to GitHub Actions is in my list of things to do, I just haven’t gotten around to it yet.

@jpsim
Copy link
Collaborator

jpsim commented Dec 9, 2020

We can add a CI job in a separate PR.

@jpsim jpsim merged commit 71449e7 into master Dec 9, 2020
@jpsim jpsim deleted the add-dockerfile branch December 9, 2020 14:14
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