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

docker support #1

Closed
wants to merge 7 commits into from
Closed

docker support #1

wants to merge 7 commits into from

Conversation

ggkollective
Copy link

개발환경 구축하기가 어려워서 만들어 봤습니다.

이미지는 2개로 나뉘는데요. 하나는 라이브러리 이미지(ome_base),
다른 하나는 코드를 받아서 빌드하는 이미지입니다.

일단은 기본 동작만 확인하고 외부 포트, 설정 파일 위치만 확인해서
노출시켰습니다.

@getroot
Copy link
Member

getroot commented Jun 30, 2018

We are very pleased to receive the first pr and thank you. We have plans to make a docker image. And we want it to be based on alpine linux because we want image size to be small. If you can help to solve this problem, we would appreciate it. If it is difficult, please wait until we update the docker image. Thanks again.

@getroot
Copy link
Member

getroot commented Jun 30, 2018

And we are using c++ 14.

@ggkollective
Copy link
Author

I can work with it, of course. since gcc needs at least 100 MB, decreasing the size of these libraries will be the biggest problem. it may affect the performance of your application. I have to close the first request anyway because the pull request comes from the master branch and I'm going to revoke them all.

@getroot
Copy link
Member

getroot commented Jun 30, 2018

I do not think gcc is needed. I think the docker image version should be built with just binaries. I know this is a difficult task. Thank you for your contribution.

@ggkollective ggkollective mentioned this pull request Jun 30, 2018
@ggkollective
Copy link
Author

I got it, thanks. We may continue the discussion in the above issue ticket.

deepsurvey referenced this pull request in Simon-remote/OvenMediaEngine Mar 11, 2021
@kage212 kage212 mentioned this pull request Mar 2, 2022
@IceOwl94 IceOwl94 mentioned this pull request Jun 4, 2022
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