[v2,2/2] data_smart: restore the ability to use capital letters

This reverts commit 6eb56624e6d8dc1944e559b4f6584bfe66f566ba.

That commit was meant as an ad-hoc band-aid when "_" was used as
override prefix, and it explicitly blocked capital letters from being
usable in the overrides.  There is no reason to keep it now we use ":"
as prefix.

Signed-off-by: Yann Dirson <yann.dirson@vates.tech>
---
 lib/bb/data_smart.py  | 2 +-
 lib/bb/tests/parse.py | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

On Mon Oct 12, 2026 at 10:39 AM CEST, Yann Dirson via lists.openembedded.org wrote:
> This reverts commit 6eb56624e6d8dc1944e559b4f6584bfe66f566ba.
>
> That commit was meant as an ad-hoc band-aid when "_" was used as
> override prefix, and it explicitly blocked capital letters from being
> usable in the overrides.  There is no reason to keep it now we use ":"
> as prefix.
>
> Signed-off-by: Yann Dirson <yann.dirson@vates.tech>
> ---

Hi Yann,

Thanks for the new version, but we are still seeing the same failure:

FAIL: test_append_in_expanded_override (bb.tests.data.TestOverrides.test_append_in_expanded_override)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/srv/pokybuild/yocto-worker/oe-selftest-armhost/build/layers/bitbake/lib/bb/tests/data.py", line 417, in test_append_in_expanded_override
    self.assertEqual(self.d.getVar("TEST"), "base appended")
AssertionError: 'base' != 'base appended'
- base
+ base appended

https://autobuilder.yoctoproject.org/valkyrie/#/builders/23/builds/5075
https://autobuilder.yoctoproject.org/valkyrie/#/builders/35/builds/5043
https://autobuilder.yoctoproject.org/valkyrie/#/builders/48/builds/4856

Can you have a look at the issue?

Thanks,
Mathieu

On 10/13/26 7:17 AM, Mathieu Dubois-Briand wrote:
> On Mon Oct 12, 2026 at 10:39 AM CEST, Yann Dirson via lists.openembedded.org wrote:
>> This reverts commit 6eb56624e6d8dc1944e559b4f6584bfe66f566ba.
>>
>> That commit was meant as an ad-hoc band-aid when "_" was used as
>> override prefix, and it explicitly blocked capital letters from being
>> usable in the overrides.  There is no reason to keep it now we use ":"
>> as prefix.
>>
>> Signed-off-by: Yann Dirson <yann.dirson@vates.tech>
>> ---
> 
> Hi Yann,
> 
> Thanks for the new version, but we are still seeing the same failure:
> 
> FAIL: test_append_in_expanded_override (bb.tests.data.TestOverrides.test_append_in_expanded_override)
> ----------------------------------------------------------------------
> Traceback (most recent call last):
>    File "/srv/pokybuild/yocto-worker/oe-selftest-armhost/build/layers/bitbake/lib/bb/tests/data.py", line 417, in test_append_in_expanded_override
>      self.assertEqual(self.d.getVar("TEST"), "base appended")
> AssertionError: 'base' != 'base appended'
> - base
> + base appended
> 
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/23/builds/5075
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/35/builds/5043
> https://autobuilder.yoctoproject.org/valkyrie/#/builders/48/builds/4856

Wow, that's funky. Getting the same sha1 tested in the autobuilder I 
don't see any failure, do I have anything special to do to reproduce how 
the autobuilder tests things?

$ git fetch origin 83afc355141f2451058f505f18be4b95486ec050
...
 From https://git.openembedded.org/bitbake
  * branch                83afc355141f2451058f505f18be4b95486ec050 -> 
FETCH_HEAD

$ git checkout FETCH_HEAD
...
HEAD is now at 83afc3551 data_smart: restore the ability to use capital 
letters

$ ./bin/bitbake-selftest 
bb.tests.parse.ParseTest.test_parse_mixedcase_pn_override
.
----------------------------------------------------------------------
Ran 1 test in 0.001s

OK

$ git checkout HEAD^
...
HEAD is now at 6e7bdd494 parse: add a (failing) test for a variable 
override with capital letters

$ ./bin/bitbake-selftest 
bb.tests.parse.ParseTest.test_parse_mixedcase_pn_override
x
----------------------------------------------------------------------
Ran 1 test in 0.002s

OK (expected failures=1)