Melhorias no postinst do le-interface
@ae11 e @dhp14, esse último merge request ficou feio =P Seguem algumas sugestões
- Remover rm do postinst
Os arquivos que estão dentro de um pacote debian são instalados no sistema. Quando você atualiza um pacote, o que acontece é a remoção do pacote antigo e instalação do novo pacote. Dessa forma, os arquivos que estavam na versão antiga do pacote mas não estão na nova serão removidos. Portanto as linhas 73 a 77 no postinst do le-interface (aqueles rm) não são necessárias. Aqui vale um adendo também. Utilize os parâmetros do comando antes dos argumentos (rm -f <arquivo>
ao invés de rm <arquivo> -f
), esse é o padrão e muitos comandos não funcionam de outra forma.
- Reestruturar o if que contém o comando exit
Esse exit tem uma change enorme de dar problema no futuro. Coloque os comandos relacionados ao if
dentro do bloco if-else
. No futuro alguém vai esquecer desse exit e levar tempo pra descobrir porque as linhas mais abaixo nunca são executadas. Também não entendi de cara o que essa linha faz, então coloque um comentário melhor em cima.
* [ ] Melhorar aquele "REINICIE O COMPUTADOR" (item movido para tarefa #305 (closed))
As vezes quando atualizo o Ubuntu aparecer pra mim um aviso dizendo que devo reiniciar o computador para que as atualizações surtam efeito. Veja se consegue descobrir como eles fazem isso (baixa prioridade aqui, invista no máximo 30min).