Closed rodrigopitanga closed 10 years ago
Aliás, considerando que temos que suportar multiplataforma, isso é essencial. O código deve ser agnóstico à plataforma. Isso é quebrado aqui:
printrun/pronterface_widgets.py 229: slicer_executable = "Slic3r_windows/slic3r.exe" 231: slicer_executable = "Slic3r_gnulinux/bin/slic3r"
Isso deve ser um parâmetro de configuração. Na hora do release, fazemos um arquivo de configuração com o caminho correto.
Isso faz com que o ticket seja um enhancement e também um "bug de engenharia de software"
Especificamente, slicer_executable em pronterface_widgets.py L229-231 deve ser uma opção gravada no arquivo de configuração
Criei a opção slicer_executable, como no comentário imediatamente acima. Todas as linhas de comando geradas dinamicamente usam isso ao compor a linha de comando do Slic3r.
O "agnosticismo da plataforma" será tratado no ticket #42.
@felipesanches testa, por favor?
Hard coded é ruim para instalações customizadas. Na minha máquina, por exemplo, eu tenho que usar o checkout do git, e o caminho para o executável é diferente, fazendo com que eu tenha que editar o código, o que não é muito prático.