Log message in west command __init__ func may cause infinite call stack about west HOT 3 OPEN

Initialization is often tricky: ordering dependencies can be hard. So it's not a major surprise that some things don't work (yet) in an __init__() method. I just had a look at initializations in zephyr/scripts/west_commands/ and none of them has any logic, just constants. This should probably be better documented but: "In Rome, do as the Romans do?" I recommend you defer any logic somewhere after __init__() time.

This being said, error-handling should be better: better message and nothing should ever trigger an infinite loop/recursion.

from west.

Thanks for the report! Can you please share:

• The west commit used
• The one-line diff
• Just in case, the manifest used.

from west.

Hi, @marc-hb , sorry my description is not quite clear. The issue happens on a west extension command and my installed west version is v1.2.0

Hook following extension command to a west project can reproduce the issue.

# example.py
from west.commands import WestCommand
class Example(WestCommand):
    def __init__(self):
        super().__init__(
          'example',
          # Keep this in sync with the string in west-commands.yml.
          'example',
          "example")
        
        # This may raise an exception
        self.err("show a log")

    def do_add_parser(self, parser_adder):
        parser = self._parser(parser_adder, epilog='')
        return parser
    
    def do_run(self, args):
        pass

from west.