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

Update Dockerfile #38

Merged
merged 1 commit into from
Aug 12, 2020
Merged

Update Dockerfile #38

merged 1 commit into from
Aug 12, 2020

Conversation

oriolrius
Copy link
Contributor

it doesn't work because of permissions of the aedes binary file.

it doesn't work because of permissions of the aedes binary file.
@robertsLando
Copy link
Member

I don't understand the reason of this change, could you explain what isn't working and what this fixes?

@oriolrius
Copy link
Contributor Author

oriolrius commented Aug 10, 2020 via email

@robertsLando
Copy link
Member

Could you show me the command you used to start it? I never faced such problem

@oriolrius
Copy link
Contributor Author

oriolrius commented Aug 10, 2020 via email

@lagnat
Copy link

lagnat commented Aug 12, 2020

Same issue. Please consider merging the PR.

% docker run --rm -it -p 1883:1883 -v $(pwd):/data moscajs/aedes:latest
docker: Error response from daemon: OCI runtime create failed: container_linux.go:349: starting container process caused "exec: \"/usr/src/app/bin/aedes\": permission denied": unknown.

@lagnat
Copy link

lagnat commented Aug 12, 2020

Potentially another way to fix it would be to change the mode in git.

diff --git a/bin/aedes b/bin/aedes
old mode 100644
new mode 100755

@robertsLando
Copy link
Member

Lgtm

@robertsLando robertsLando merged commit 2dba0c6 into moscajs:master Aug 12, 2020
@robertsLando
Copy link
Member

Will be available in docker dev tag

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.

3 participants