Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gerar executável #13

Merged
merged 7 commits into from
Dec 17, 2023
Merged

Conversation

DaviRJ
Copy link
Contributor

@DaviRJ DaviRJ commented Mar 1, 2019

Fiz a adição do pkg, para ser possível usar a aplicação direto de um arquivo executável.
Coloquei dois comandos no package.json.
É simples gerar o arquivo:

  • Para Windows, basta rodar

      npm run build-win
    
  • Para Mac, basta rodar

      npm run build-mac
    

Para gerar a build/executável, será necessário ter o npm 5.2.0 ou superior com o npx.

@luizjr
Copy link

luizjr commented Mar 1, 2019

Porque não fez para Mac também?
Consegue fazer?

@DaviRJ
Copy link
Contributor Author

DaviRJ commented Mar 2, 2019

Porque não fez para Mac também?
Consegue fazer?

Feito! =)

@DaviRJ DaviRJ changed the title Gerar executavel Gerar executável Mar 2, 2019
@rafabp82
Copy link

rafabp82 commented Mar 4, 2019

Quando fui gerar o executável deu erro falando que não achou o modulo "readline-sync", dai o executável não roda. Podem me ajudar, por favor?

@danielschmitz
Copy link
Collaborator

Não acho ser necessário ter um ".exe" já que, em tese, esse projeto poderia ir para o npm e ser executado via

npx video-maker

ou poderia fazer um npm install --global video-maker e executá-lo por linha de comando...

@diegofcruz
Copy link

Não há necessidade de gerar executável, utilize o próprio terminal para iniciar o recurso, no máximo adicione globalmente na sua máquina.

@rafabp82
Copy link

rafabp82 commented Mar 4, 2019

Obrigado pelas respostas, mas continuo com dificuldades, rs. Não sou programador nem nada, sou só um curioso, então não estou conseguindo rodar de jeito nenhum isso. Eu tenho o npm e npx instalados e baixei os codes, qual o procedimento para rodar?

@misaeldossantos
Copy link

rafabp82, basta executar npm install para instalar as dependências

@rafabp82
Copy link

rafabp82 commented Mar 5, 2019

Valeu pela ajuda pessoal, consegui rodar! Mas depois da segunda pergunta ele para de funcionar

@DaviRJ
Copy link
Contributor Author

DaviRJ commented Mar 5, 2019

A minha ideia de gerar o executável é para que qualquer pessoa possa usar, sem a necessidade de ter que instalar alguma dependência ou abir o terminal. Somente o simples e intuitivo duplo clique =)

@luizjr
Copy link

luizjr commented Mar 5, 2019

A minha ideia de gerar o executável é para que qualquer pessoa possa usar, sem a necessidade de ter que instalar alguma dependência ou abir o terminal. Somente o simples e intuitivo duplo clique =)

Eu achei brilhante, por isso eu pedi para o MAC, Isso vai ser ainda mais importante no resultado de tudo, quando será apenas executado e aguardaremos os bots trabalharem.

Copy link

@lffg lffg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DaviRJ, boa ideia cara!!

Só uma dúvida: por qual motivo você instalou a dependência pkg, mas está a utilizar o npx? Se a dependência está instalada, não existem motivos para usar o npx. :)

@Vinicius-CS
Copy link

É normal quando ele já pega o texto fechar o executável?

@DaviRJ
Copy link
Contributor Author

DaviRJ commented Mar 14, 2019

@DaviRJ, boa ideia cara!!

Só uma dúvida: por qual motivo você instalou a dependência pkg, mas está a utilizar o npx? Se a dependência está instalada, não existem motivos para usar o npx. :)

Desculpa a demora pela resposta. Uma vez que a dependência estiver instalada, o npx executa o que está na pasta node_modules ao invés de baixar e instalar o pkg.

É normal quando ele já pega o texto fechar o executável?

Sim. Pois de fato, o programa terminou. Você pode usar o node-pause, por exemplo, para manter a janela aberta.

Copy link

@lffg lffg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uma vez que a dependência estiver instalada, o npx executa o que está na pasta node_modules ao invés de baixar e instalar o pkg.

@DaviRJ , não sabia desse comportamento. De qualquer modo, no entanto, o npx é inútil nesse caso, tendo em vista que já temos a dependência instalada no package.json. Assim, quando executamos um npm script, todas as dependências que possuem binários tem o seu PATH configurado temporariamente para a execução dos scripts.

package.json Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@DaviRJ
Copy link
Contributor Author

DaviRJ commented Mar 16, 2019

@DaviRJ , não sabia desse comportamento. De qualquer modo, no entanto, o npx é inútil nesse caso, tendo em vista que já temos a dependência instalada no package.json. Assim, quando executamos um npm script, todas as dependências que possuem binários tem o seu PATH configurado temporariamente para a execução dos scripts.

Tem razão. Fiz as alterações e aproveitei para atualizar o pr. Obrigado =)

@lemacedo
Copy link

Ótima ideia!

@matbrgz
Copy link
Collaborator

matbrgz commented Oct 31, 2019

Como está o andamento dessa PR? Posso resolver o conflito (ou alguém façã isso por mim por favor) e dar merge?

@matbrgz matbrgz added the enhancement New feature or request label Oct 31, 2019
@DaviRJ
Copy link
Contributor Author

DaviRJ commented Nov 6, 2019

Resolvi o conflito! Se alguém puder validar, agradeço!

@matbrgz matbrgz requested a review from leodutra November 19, 2019 01:44
@leodutra
Copy link
Collaborator

Devo realizar o review amanhã.
Se não aparecer, por favor me lembrem.

@matbrgz
Copy link
Collaborator

matbrgz commented Dec 5, 2019

@leodutra Friendly ping!

@matbrgz matbrgz requested review from leodutra and removed request for leodutra March 10, 2020 13:54
package.json Outdated
"url": "git+https://github.com/filipedeschamps/video-maker.git"
},
"author": "",
"license": "ISC",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

License está errado. O projeto usa MIT.

@matbrgz
Copy link
Collaborator

matbrgz commented Dec 17, 2023

Licença atualizada, PR aceito.

@matbrgz matbrgz merged commit a227d82 into filipedeschamps:master Dec 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.