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 do not merge #11

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mileo
Copy link

@mileo mileo commented Feb 8, 2021

@danielgoncalves já tentou usar com Docker?

Oq vc acha da ideia?

Abraços!

Comment on lines +10 to +32
build-base \
python3 \
python3-dev \
# wget dependency
openssl \
# dev dependencies
bash \
git \
py3-pip \
sudo \
# Pillow dependencies
freetype-dev \
fribidi-dev \
harfbuzz-dev \
jpeg-dev \
lcms2-dev \
openjpeg-dev \
tcl-dev \
tiff-dev \
tk-dev \
zlib-dev \
zbar-dev && \
pip install -r requirements.txt
Copy link
Author

Choose a reason for hiding this comment

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

TODO: Melhorar essa lista de dependências, pois muitas são desnecessárias.

@danielgoncalves
Copy link
Contributor

Acho sensacional. Muito bom! Só tenho duas ressalvas pra gente pensar: (1) como entregar a libsat para equipamentos específicos; (2) se é uma boa ideia mapear /dev todinho assim como você fez. Pra primeira, a ideia seria implementar uma maneira de enviar o arquivo da biblioteca através da interface web, como parte do processo de configuração. Ou então criar imagens específicas (tanca, bema, elgin, etc...).

@mileo
Copy link
Author

mileo commented Feb 8, 2021

Acho sensacional. Muito bom! Só tenho duas ressalvas pra gente pensar: (1) como entregar a libsat para equipamentos específicos; (2) se é uma boa ideia mapear /dev todinho assim como você fez. Pra primeira, a ideia seria implementar uma maneira de enviar o arquivo da biblioteca através da interface web, como parte do processo de configuração. Ou então criar imagens específicas (tanca, bema, elgin, etc...).

Até tentei s/ o -d mapeado, mas não consegui:

image

Vou dar uma pesquisada pra ver se descubro algo.

O comando usado foi:

docker run --name sathub -p 5000:5000 --device=/dev/ttyTS0 --rm sathub:latest

@mileo
Copy link
Author

mileo commented Feb 8, 2021

@danielgoncalves melhor versão que eu consegui. Pelo que falei com o @ananiasfilho realmente é necessário colocar o parâmetro --privileged

@mileo
Copy link
Author

mileo commented Feb 8, 2021

Respondendo a sua pergunta creio que seja viável criar uma imagem de base e uma imagem para cada fabricante que dependa dessa imagem de base.

@mileo
Copy link
Author

mileo commented Feb 9, 2021

image

Documentando como foi feito com docker-compose

@danielgoncalves danielgoncalves marked this pull request as ready for review March 9, 2022 11:27
Copy link
Contributor

@danielgoncalves danielgoncalves left a comment

Choose a reason for hiding this comment

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

Excelente!

@mileo
Copy link
Author

mileo commented Apr 17, 2022

Excelente!

Eu avancei um pouco mais nesse PR, amanhã subo os commits que eu esqueci.

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