[Bug]: juce_gui_extra.cpp defaults to including web browser ignoring CMake option
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 55/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Pouca atividade
- Stack de tecnologia
- cpp
- Domínio
- build-system
Direção de pesquisa
Comece em modules/juce_gui_extra/juce_gui_extra.h, linhas 75-77, onde JUCE_WEB_BROWSER é definido independentemente da opção NEEDS_WEB_BROWSER do CMake, e então verifique como juce_add_gui_app propaga NEEDS_WEB_BROWSER em cmake/JUCEUtils.cmake e o que modules/juce_gui_extra/juce_gui_extra.cpp inclui em seguida. Decida se o header deveria definir JUCE_WEB_BROWSER apenas quando explicitamente habilitado, ou se o CMake deveria forçá-lo para 0. Considera-se concluído quando um alvo juce_add_gui_app com NEEDS_WEB_BROWSER desativado compila sem a definição de compilação manual JUCE_WEB_BROWSER=0.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
Detailed steps on how to reproduce the bug
When using, for example, juce_add_gui_app() and with the default option of NEEDS_WEB_BROWSER set to false, the following in juce_gui_extra.h causes attempts to import headers in juce_gui_extra.cpp that are then correctly not linked by cmake, causing a build error:
https://github.com/juce-framework/JUCE/blob/3ba67d4585e9d1fbcdb26a877c7978608b1f802e/modules/juce_gui_extra/juce_gui_extra.h#L75-L77
Workaround is to set the compile definition JUCE_WEB_BROWSER=0 but given the default is this already when adding the target, this definition should be set to 0 by default, or the setting of it in that file should be removed so it's only ever explicitly defined when enabled.
What is the expected behaviour?
Builds successfully.
Operating systems
Linux
What versions of the operating systems?
Fedora 44.
Architectures
x86_64
Stacktrace
Plug-in formats (if applicable)
No response
Plug-in host applications (DAWs) (if applicable)
No response
Testing on the develop branch
I have not tested against the develop branch
Code of Conduct
- I agree to follow the Code of Conduct
- Linguagem predominante
- C++
- Estrelas
- 8.9k
- Forks
- 2.2k
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Preparar o ambiente
Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de juce-framework/JUCE
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
juce-framework/JUCE#1774 ·
-
[Bug]: drawFittedText / addFittedText ignore maximumLines when the text contains a line breakAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
juce-framework/JUCE#1765 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
juce-framework/JUCE#1764 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 74/100
juce-framework/JUCE#1762 ·
-
Linux: WebBrowserComponent shows nothing unless the GTK and WebKit development packages are installedTalvez já em andamento Um pull request vinculado a esta issue está aberto ou já foi mesclado. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
juce-framework/JUCE#1730 ·
Todas as issues de juce-framework/JUCE
Issues semelhantes
-
A-tools P-medium S-ready-to-start studio task
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
Status: Needs confirmation Status: Needs triage Type: Code Quality
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Mantenedores costumam responder em até 1 dia
-
Image.resolveAssetSource's TypeScript declaration drops the null it returns for an unknown assetAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
react/react-native#59648 ·
Mantenedores costumam responder em até 2 dias
-
platform: Linux
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 80/100
keepassxreboot/keepassxc#13947 ·
Mantenedores costumam responder em até 1 dia
-
Shell: serve the home app's Cookie Policy on a hard reloadTalvez já em andamento @2001anshkaushik assumiu hoje. Abertafeature module:ai
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
rocketride-org/rocketride-server#2899 · 1 comentário ·
Mantenedores costumam responder em até 2 dias