search

farribeiro / wscef-docker

Warsaw in docker container
MIT License
64 stars 23 forks source link

Using sudo instead su #36

Closed farribeiro closed 6 years ago

farribeiro commented 6 years ago

Link to Issue https://github.com/farribeiro/wscef-docker/issues/35

Tanks @jsalatiel

farribeiro commented 6 years ago

Anything else?

~PS: This will be opened until tuesday (01/30).~

farribeiro commented 6 years ago

Link to PR https://github.com/farribeiro/wscef-docker/pull/34

jsalatiel commented 6 years ago

As linhas do sudo estão erradas. sudo "apt update && apt -y upgrade && apt -y install /src/GBPCEFwr64.deb" sudo "/etc/init.d/warsaw start"

O correto seria: sudo apt update && sudo apt -y install /src/GBPCEFwr64.deb sudo /etc/init.d/warsaw start

Se você coloca as aspas ele vai achar que tudo entre aspas é um comando apenas.

jsalatiel commented 6 years ago

Ainda está errado. Você precisa prefixar com sudo CADA comando que precisa ser rodado como root. Você colocou: sudo apt update && apt -y upgrade && apt -y install /src/GBPCEFwr64.deb

o correto é: sudo apt update && sudo apt -y upgrade && sudo apt -y install /src/GBPCEFwr64.deb

farribeiro commented 6 years ago

Então poderia deixar como está pois o código está simples e também funcional. Por que não fazer como fiz com su?

Outro detalhe... o sudo serve, em especial, para fazer log do acesso ao root e que neste caso dispensaria o seu uso. Se não chegar um consenso, será arquivado.

jsalatiel commented 6 years ago

Cara, se você prefere um container pedindo senha deixe como está :) Eu não estou mais usando seu container, estou usando o fork que fiz com centOS. De toda forma, sua jogada pra fazer o warsar funcionar no container foi muito boa. Parabéns!

farribeiro commented 6 years ago

Não prefiro, eu apenas usava sem senha de root. Isso foi sugerido por outros usuários (Issue https://github.com/farribeiro/wscef-docker/issues/29).

Embora não gostaria de digitar senhas, só não quero muitas alterações no funcionamento do script. Tem um branch sugerido pelo Laércio para eu abandonar o Ubuntu em prol do Alpine e reduzir drasticamente o tamanho da imagem!

Irei refazer o PR

farribeiro commented 6 years ago

Refeito histórico do PR

@jsalatiel ok?