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

[Makefile] Support docker-like OCI engines #431

Merged
merged 2 commits into from
Aug 27, 2022

Conversation

signed-log
Copy link
Contributor

@signed-log signed-log commented Aug 27, 2022

What does this PR do?

Remove the hard-coding for docker and allow other OCI engines to be used for testing, but keeping a hard-coded default of docker

Previous Behavior

Short of installing docker or symlinking podman (for ex.) to docker, you can't use something like podman.

New Behavior

  • Create OCI variable to override the docker engine
  • Refactor the test-docker error message to expose this possibility

Have tests been written for this change?

No

Have these commits been signed with GnuPG?

Yes


Please review yadm's Contributing Guide for best practices.

Support setting a different OCI engine than docker

- Create OCI variable to override the docker engine
- Refactor the test-docker error message to expose this possibility

Signed-off-by: Nicolas signed-log FORMICHELLA <[email protected]>
@signed-log signed-log marked this pull request as draft August 27, 2022 09:49
Document the OCI override switch

Signed-off-by: Nicolas signed-log FORMICHELLA <[email protected]>
@signed-log signed-log marked this pull request as ready for review August 27, 2022 10:05
@TheLocehiliosan TheLocehiliosan self-assigned this Aug 27, 2022
@TheLocehiliosan TheLocehiliosan merged commit a89e5ce into yadm-dev:develop Aug 27, 2022
@signed-log signed-log deleted the makefile-oci-patch branch September 4, 2022 11:25
TheLocehiliosan added a commit that referenced this pull request Jan 23, 2023
* Support spaces in distro/distro-family (#432)
* Fix zsh hanging when tab completing add/checkout (#417)
* Add yadm-untracked script to contributed files (#418)
* Fix documentation typos (#425)
* Support docker-like OCI engines for dev testing (#431)
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.

2 participants