Melhora a obtenção das informações do sistema em que o executável foi feito.#19
Melhora a obtenção das informações do sistema em que o executável foi feito.#19samuelveigarangel wants to merge 6 commits into
Conversation
- Altera script para funcionar com interfaces GTK e QT
…idator into fix/build_metadata
pitangainnovare
left a comment
There was a problem hiding this comment.
Ótimo que tenha norteado o script a forçar uso de Qt, quando conveniente. Mas infelizmente outro bug surgiu: o diálogo de seleção de arquivos não mostra nada e pelo que notei, parece ser um bug conhecido dessa backend (pywebview com Qt). Veja
Mais precisamente: o HTML tem accept=".zip". O pywebview passa isso para o diálogo de arquivo do backend. Com Qt, esse filtro parece estar afetando a navegação/listagem de diretórios de forma ruim (não mostra arquivos, nem diretórios). Em GTK, funciona normalmente.
Talvez seja o caso de forçar a app a ficar em GTK.
|
|
||
| echo "Linux executable generated at dist/spsvalidator." | ||
| echo "GTK and Qt backends are bundled; runtime still needs system GTK or Qt libraries." | ||
| echo "Use linuxdeploy/appimagetool to convert it into AppImage." |
There was a problem hiding this comment.
É realmente necessário informar o usuário sobre isso de converter para AppImage?
No futuro pode ser interessante adotarmos o AppImage para disponibilizar os executáveis.
Ambiente de teste
ResumoTestei localmente o #19 (que já inclui todo o diff do #17) rodando o app em modo browser e executando Modo browser ( Script de build (
Causa raiz: a venv de build usa Outros pontos observados na revisão do código
SugestãoConcordo com o comentário do @pitangainnovare sobre forçar o app a ficar em GTK. Acho que resolve o ponto 1 (build quebrado) e também o bug do diálogo de arquivos com Qt relatado na review. Bastaria excluir os bindings Qt/PyQt de forma explícita no excludes=[
"webview.platforms.qt",
"PyQt5",
"PyQt6",
"qtpy",
"webview.platforms.android",
"webview.platforms.cocoa",
"webview.platforms.winforms",
"webview.platforms.edgechromium",
"black",
"pytest",
"isort",
],E remover a instalação e a checagem de |
Rossi-Luciano
left a comment
There was a problem hiding this comment.
Detalhes completos da revisão e da reprodução do build quebrado no comentário acima.
Descrição
Como poderia ser testado
Executar o script de compilação de acordo com o seu sistema.