Parsing of VHDL configurations does not handle spaces
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 46/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Área
- testing-qa
Línea de trabajo
Start by reading VUnit’s VHDL configuration parsing code and its parser tests; the issue does not name specific files. Reproduce the ambiguity error with the shown configuration using spaces in rtl, then compare it with the working form without spaces. Done means the spaced valid VHDL configuration is recognized and the existing parser tests still pass.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
There is a bug in the parsing of VHDL configurations by VUnit. If you have spaces around you architecture instantiations VUnit will not recognize them properly.
Taking the example VHDL Configurations as a base we can show this bug by adding some spaces.
This works
-- Configuration declarations
configuration rtl of tb_selecting_dut_with_vhdl_configuration is
for tb
for test_fixture
for dut : dff
use entity work.dff(rtl);
end for;
end for;
end for;
end;
This is valid VHDL but it doesn't work and gives the error: RuntimeError: Ambiguous use of test_lib.dff
configuration rtl of tb_selecting_dut_with_vhdl_configuration is
for tb
for test_fixture
for dut : dff
use entity work.dff( rtl );
end for;
end for;
end for;
end;
The ambiguous use error seems to come from VUnit not recognizing the use entity work.dff( rtl ); statement anymore. I suspect the pattern matching of the VUnit parser does not account for spaces.
- Lenguaje dominante
- VHDL
- Estrellas
- 850
- Forks
- 305
- Merge medio
- 2 d 10 h
- PR fusionados (30 d)
- 7
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de VUnit/vunit
-
KeyError in test scheduler when using TestBench.scan_tests_from_file()Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
GHDL: generics leak between configurations when `ghdl.sim_flags` is setPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
VUnit/vunit#1223 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Los mantenedores suelen responder en 1 día
-
AXI slave VCs have identical stalling behaviorPosiblemente ocupada @Lay007 la tomó hace 28 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
VUnit/vunit#1219 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de VUnit/vunit
Issues similares
-
tests: pin that the atomic writers in utils/paths.py stage the temporary file in the target's directoryPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodifficulty:2 enhancement good first issue
Dificultad 1/5 1-3 horas Aptitud para principiantes 92/100
MakazhanAlpamys/Soup#2178 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Snapshot capture failure (EPERM spawning git) fails the whole step despite the best-effort contractPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertoreproduced
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
anomalyco/opencode#58685 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
JakeChampion/lang#13132 ·
Los mantenedores suelen responder en 1 día