mcux-sdk icon indicating copy to clipboard operation
mcux-sdk copied to clipboard

Search in guiconfig broken due to missing Python import

Open stefanct opened this issue 11 months ago • 1 comments

The GUI of kconfig allows users to search in its database (e.g., via make guiconfig, ctrl+f) but in the MCUX SDK this is broken due to re missing. When search for any string nothing happens in the GUI but one can see this in the text output:

  File ".../scripts/kconfig/guiconfig.py", line 1867, in _update_jump_to_matches
    except re.error as e:
           ^^
NameError: name 're' is not defined

https://github.com/nxp-mcuxpresso/mcux-sdk/blob/acc105f1e6304bf3a16ebbb503f8882a4daa1408/scripts/kconfig/guiconfig.py#L1867

A simple import re at the top of the file fixes the issue. I have not checked how other projects have handled this.

stefanct avatar Mar 04 '24 17:03 stefanct

Hi @stefanct Thanks for your feedback. Actually I have found this issue before and created the ticket here: https://github.com/ulfalizer/Kconfiglib/issues/136. But I don't receive any feedback. I'll fix it in our repo later anyway.

FelixWang47831 avatar Mar 27 '24 02:03 FelixWang47831