stove/tiramisu
3
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix: name is mandatory for config inside a metaconfig

Browse source
This commit is contained in:
egarette@silique.fr 2026-01-01 21:52:36 +01:00
parent cc79c28d40
commit 71fbe6dcbd
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
tests/test_submulti.py
View file

@ -533,7 +533,7 @@ def test_multi_submulti_meta():
od1 = OptionDescription('od', '', [multi]) od1 = OptionDescription('od', '', [multi])
cfg = Config(od1, name='cfg') cfg = Config(od1, name='cfg')
cfg.property.read_write() cfg.property.read_write()
cfg2 = Config(od1) cfg2 = Config(od1, name="cfg2")
cfg2.property.read_write() cfg2.property.read_write()
meta = MetaConfig([cfg, cfg2]) meta = MetaConfig([cfg, cfg2])
meta.property.read_write() meta.property.read_write()
@ -554,7 +554,7 @@ def test_multi_submulti_meta_no_cache():
od1 = OptionDescription('od', '', [multi]) od1 = OptionDescription('od', '', [multi])
cfg = Config(od1, name='cfg') cfg = Config(od1, name='cfg')
cfg.property.read_write() cfg.property.read_write()
cfg2 = Config(od1) cfg2 = Config(od1, name="cfg2")
cfg.property.read_write() cfg.property.read_write()
meta = MetaConfig([cfg, cfg2]) meta = MetaConfig([cfg, cfg2])
meta.property.read_write() meta.property.read_write()

4
tiramisu/config.py
View file

@ -1894,6 +1894,10 @@ class KernelMetaConfig(KernelMixConfig):
raise TypeError(_("child must be a Config or MetaConfig")) raise TypeError(_("child must be a Config or MetaConfig"))
if descr is None: if descr is None:
descr = child.get_description() descr = child.get_description()
if child.impl_getname() is None:
raise ConfigError(
_("children in MetaConfig must have name")
)
elif descr is not child.get_description(): elif descr is not child.get_description():
raise ValueError( raise ValueError(
_( _(