Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

[Bug]: juce_gui_extra.cpp defaults to including web browser ignoring CMake option

Abierto
  • ¿Algo no está bien?
#1,663 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 2 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
55/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Tranquilo
Stack tecnológico
cpp
Área
build-system

Línea de trabajo

Comienza en modules/juce_gui_extra/juce_gui_extra.h, líneas 75-77, donde JUCE_WEB_BROWSER se define independientemente de la opción NEEDS_WEB_BROWSER de CMake, y luego revisa cómo juce_add_gui_app propaga NEEDS_WEB_BROWSER en cmake/JUCEUtils.cmake y qué incluye después modules/juce_gui_extra/juce_gui_extra.cpp. Decide si el header solo debería definir JUCE_WEB_BROWSER cuando está habilitado explícitamente, o si CMake debería forzarlo a 0. Se considera terminado cuando un destino juce_add_gui_app con NEEDS_WEB_BROWSER desactivado compile sin la definición de compilación manual JUCE_WEB_BROWSER=0.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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
Lenguaje dominante
C++
Estrellas
8.9k
Forks
2.2k
Sin PR fusionados en 30 d

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de juce-framework/JUCE

Todos los issues de juce-framework/JUCE

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.