Skip to content
This repository was archived by the owner on Jul 18, 2025. It is now read-only.

Implement run command#133

Closed
vdemeester wants to merge 1 commit into
docker-archive-public:masterfrom
vdemeester:carry-pr-45
Closed

Implement run command#133
vdemeester wants to merge 1 commit into
docker-archive-public:masterfrom
vdemeester:carry-pr-45

Conversation

@vdemeester
Copy link
Copy Markdown
Contributor

Carrying #45, add a basic run command (with no flags for now). 🐨

# libcompose run <service> <command>
$ ~docker/libcompose/libcompose-cli run web ls
WARN[0000] Note: This is an experimental alternate implementation of the Compose CLI (https://github.com/docker/compose) 
bin  boot  dev  etc  home  lib  lib64  media  mnt  opt  proc  root  run  sbin  srv  sys  tmp  usr  var
$ ~docker/libcompose/libcompose-cli run web /bin/sh
WARN[0000] Note: This is an experimental alternate implementation of the Compose CLI (https://github.com/docker/compose) 
# ls
bin  boot  dev  etc  home  lib  lib64  media  mnt  opt  proc  root  run  sbin  srv  sys  tmp  usr  var
# echo foo > /bar    
# cat bar
foo

This is the first step to have a run command that is acting the same as docker-compose. I imported go-dockerpty in a non academic way (and clean it up) as it was not compiling under windows and redefining stuff that are already available on docker/docker/pkg/…. I'll probably make a PR to the repos and vendor it if it makes it and we think it's better to do.

  • Need to fix/add some integration tests.

🐸
Signed-off-by: Frederic Branczyk fbranczyk@gmail.com

$ libcompose run <service> <command>

This is the first step to have a run command that is acting the same as docker-compose.

Signed-off-by: Frederic Branczyk <fbranczyk@gmail.com>
@vdemeester
Copy link
Copy Markdown
Contributor Author

Will reimplement that with engine-api, should be simpler.

@vdemeester vdemeester closed this Feb 15, 2016
@vdemeester vdemeester deleted the carry-pr-45 branch February 15, 2016 21:16
@vdemeester vdemeester mentioned this pull request Feb 22, 2016
1 task
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants