Re: [OE-core] [PATCH v7 1/6] distro/include: rework debug friendly optimization

[adrian.freihofer@gmail.com]

Hi Hongxu, hi Mathieu

I guess this series is not complete. If I do a git grep on today's
master-next branch with this series applied:

openembedded-core$ git grep DEBUG_BUILD
  scripts/lib/devtool/ide_sdk.py:
    self.debug_build = recipe_d.getVar('DEBUG_BUILD')
  scripts/lib/devtool/ide_sdk.py:
    'You might want to add DEBUG_BUILD = "1" to %s. '
  scripts/lib/devtool/standard.py:
    f.write('DEBUG_BUILD = "1"\n')
  scripts/lib/devtool/standard.py:
    parser_modify.add_argument('--debug-build', action="store_true",
    help='Add DEBUG_BUILD = "1" to the modified recipe')

This breaks the patch from
https://lists.openembedded.org/g/openembedded-core/message/228633
which tries to add a test for remote debugging.

The issue looks like this:
https://autobuilder.yoctoproject.org/valkyrie/#/builders/23/builds/3070

Breaking scripts/lib/devtool/standard.py:
    f.write('DEBUG_BUILD = "1"\n')
is at least an explanation for this issue. Since TDEBUG_BUILD = "1" is
now without any effect, the compiler removes the code where the test
adds the break point.


@Mathieu: would you mind
- dropping this series from your master-next branch until this is fixed
- bring the patch from my v1 series back?
  (v2 seams to be worse than v1)
https://patchwork.yoctoproject.org/project/oe-core/patch/20251230084720.2371227-14-adrian.freihofer@siemens.com/

Of course, I can also send a v3 if this would be easier for you.

Thank you.

Regards,
Adrian


> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#228304):
> https://lists.openembedded.org/g/openembedded-core/message/228304
> Mute This Topic: https://lists.openembedded.org/mt/116902148/4454582
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe:
> https://lists.openembedded.org/g/openembedded-core/unsub [
> adrian.freihofer@gmail.com]
> -=-=-=-=-=-=-=-=-=-=-=-