Skip to content

Docker Support#94

Merged
patrickelectric merged 2 commits into
mavlink:masterfrom
ericjohnson97:feature/docker_support
Mar 15, 2024
Merged

Docker Support#94
patrickelectric merged 2 commits into
mavlink:masterfrom
ericjohnson97:feature/docker_support

Conversation

@ericjohnson97
Copy link
Copy Markdown
Contributor

This PR runtime docker support

Comment thread Dockerfile
@@ -0,0 +1,28 @@
# Build
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@patrickelectric I am not sure if you would like to have a separate build container for building or if you prefer it like this. let me know and I can adjust it.

Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated

RUN chmod +x mavlink2rest

ENTRYPOINT ./mavlink2rest -c ${MAVLINK_SRC} -s ${SERVER_PORT}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we add an env variable for generic arguments ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

let me know what you think of the revised args

@ericjohnson97
Copy link
Copy Markdown
Contributor Author

@patrickelectric Let me know what you think of this dockerfile. If this is not what you want or would like to do it a different way, I understand. I can either revise or let you do it in some time in the future.

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