-
-
Notifications
You must be signed in to change notification settings - Fork 41
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Browse files
Browse the repository at this point in the history
- Loading branch information
cauachagas
committed
Jun 15, 2020
1 parent
9cad2a3
commit 472ed7d
Showing
4 changed files
with
38 additions
and
12 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
472ed7d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cauachagas você adicionou o if, talvez seria interessante colocar um
else
:E também, perceba que você deixou de redirecionar os erros também, então recomendo adicionar os erros e unificar os nomes:
472ed7d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cauachagas quanto menos códigos duplicados melhor, diminui probabilidade de erros.
472ed7d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eu tentei fazer algo parecido, mas sem sucesso. Se usar
o
latex_commando
retorna falso. Então não funcionará.A ideia era verificar se "latexliveonfly" existe no sistema e, se existir, compilar primeiro com ele e depois com o comando padrão do Limarka (
latexmk --xelatex
). Somente usar olatexliveonfly
não funciona. Ele ignora as referências bibliográficas.O commit funciona em Unix, mas não tenho certeza se o
whereis
funciona no Windows. Aliás, não sei se para Windows é viável porque teria que instalar o Python. Então talvez possamos limitar essa funcionalidade limitada aos sistemas Unix, @edusantana ?472ed7d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Verdade, o texliveonfly precisa ser chamado uma vez antes.
Penso que talvez ele seja uma aplicação para ser adicionada na imagem customizada. Pois o limarka não precisa dele, daí a imagem aumenta com instalação do python. Temos que avaliar isso.
472ed7d
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Comparando aqui https://github.com/abntex/limarka/blob/master/containers/ruby-latex-pandoc.dockerfile . O com tag
latest
é o que o Limarka usa atualmente. Não sei se dá pra otimizar o Alpine, que deu até maior que a versão do Ubuntu. Na minha versão do Ubuntu dotexliveonfly
, acho que redução do tamanho é por conta do--no-install-recommends
. E o Python no Ubuntu não é uma dependência extensa.