#1 gerar executável para linux

[acmesquita]

Fixed #1

[diego3g]

Por que adicionar o package-lock.json no .gitignore?

[acmesquita]

Por ser um arquivo gerado, não vejo sentido ir no commit.

[diego3g]

Estamos usando Yarn nesse pacote e por isso usamos o yarn.lock e por isso não é legal você incluir o package-lock.json de qualquer forma, porém os arquivos .lock por mais que são gerados, eles devem ir ao Git, isso porque armazenam cache para acelerar a instalação dos pacotes do zero :)

[acmesquita]

Entendi, obrigada pela explicação... =D Irei remover e adicionar mais opções de downloads.

[diego3g]

Essa linha não pode ser removida, existe uma maneira de verificar se o dock não existe, dá uma olhada no #8

[acmesquita]

Vou ajustar isso, obrigada!

[diego3g]

Vou testar no meu Ubuntu certinho e daí aprovo a PR, acho legal ajustar apenas os pontos que mencionei

[WilianZilv]

Vc pode fazer if(app.dock){app.dock.hide()}, como eu fiz na minha PR do Windows

[diego3g]

Por que colocar as options do spawn em um array? Algum motivo específico?

[acmesquita]

Por que colocar as options do spawn em um array? Algum motivo específico?

Segundo a documentação, https://nodejs.org/api/child_process.html#child_process_child_process_spawn_command_args_options as options devem ser passadas por array.

[diego3g]

@acmesquita Na verdade lá diz que options é Object, aquele `[, options] nos parâmetros não quer dizer um array, é apenas uma forma de demonstrar que é opcional.

[acmesquita]

@acmesquita Na verdade lá diz que options é Object, aquele `[, options] nos parâmetros não quer dizer um array, é apenas uma forma de demonstrar que é opcional.

Realmente, me enganei quando coloquei o array, funcionou porque retornou um undefined para o spawn e "funcionou" no meu computador.
Descobri que por conta do PATH do meu terminal o spawn não identificava o comando 'code' e não executava o comando quando clicava. Contudo, quando deixo a opção padrão { env: process.env }, ele identifica o comando e funciona normalmente.

Dúvida, se remover o env funciona nos outros terminais e nos outros sistemas operacionais?

[renandecarlo]

Não é melhor usar "snap", que tem compatibilidade maior?

[diego3g]

Acredito que podemos gerar até mais de um formato, o que acham?

[diego3g]

@acmesquita O env é essencial para funcionar em produção. Sem ele até funciona no Mac em desenvolvimento mas em produção simplesmente não rola, ele não reconhece o caminho para meu executável do code. Acho que seria legal só ajustarmos o package-lock.json (pode removê-lo por completo, nem precisa incluir no ignore). Podemos também aceitar mais opções de executáveis pro Linux além do .deb?

[diego3g]

Executei um merge para sincronizar com a versão na master que havia sido atualizada por outra PR, com isso acredito que possamos aceitar. Obrigado pela contribuição @acmesquita :)

[diego3g]

@acmesquita Rodei o yarn dist aqui e não está gerando os artefatos .dev, apenas os de Mac, sabe se preciso fazer algo a mais?

[acmesquita]

@diego3g, o electron-builder reconhece a plataforma que está sendo gerado os pacotes como na imagem abaixo

Para rodar para a plataforma linux é preciso rodar o comando com a flag --linux. Não rodei no mac, mas acredito que rodando o comando npm run dist:linux possa gerar os arquivos .deb, .snap e .rpm.

[diego3g]

@acmesquita Consegui rodar também com o comando yarn dist -mwl que gerou executável para Mac, Windows e Linux.