Using sudo instead su

[farribeiro]

Link to Issue #35

Tanks @jsalatiel

[farribeiro]

Anything else?

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

[farribeiro]

Link to PR #34

[jsalatiel]

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]

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]

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]

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]

Não prefiro, eu apenas usava sem senha de root. Isso foi sugerido por outros usuários (Issue #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]

Refeito histórico do PR

@jsalatiel ok?