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

Docs don't mention docker-compose also requires read permission. #2523

Closed
wants to merge 3 commits into from
Closed

Docs don't mention docker-compose also requires read permission. #2523

wants to merge 3 commits into from

Conversation

XavierTalpe
Copy link

When installing docker-compose on Ubuntu I ran into the following problem:

Cannot open self /usr/local/bin/docker-compose or archive /usr/local/bin/docker-compose.pkg

Based on some Googling and information in #1135, adding the read permission solved the issue for me and other people.

XavierTalpe and others added 2 commits December 9, 2015 14:08
Signed-off-by: Xavier Talpe <xavier.talpe@virdata.com>
Signed-off-by: Harald Albers <github@albersweb.de>
@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "bugfix/CannotOpenSelf" git@github.com:XavierTalpe/compose.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@aanand
Copy link

aanand commented Dec 9, 2015

Looks like this needs a rebase.

@dnephin
Copy link

dnephin commented Dec 9, 2015

How did you download the binary? It's very strange that it would be downloaded without read permissions. Are you sure the issue wasn't the user ownership ?

@XavierTalpe
Copy link
Author

@dnephin Good point. I redid the steps I did yesterday and now it seems to work... /confused. The read flag is indeed set by default. Not sure how I managed to run into this issue. I guess this PR can be closed as invalid.

@aanand
Copy link

aanand commented Dec 10, 2015

I suppose it wouldn't hurt to add +rx though?

@dnephin
Copy link

dnephin commented Dec 10, 2015

Since it's not necessary, I think it might just serve to confuse people instead of actually fixing anything.

@dnephin dnephin closed this Jan 12, 2016
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.

None yet

5 participants