[PATCH] GDB: fix python config command line arguments based on python version

Chris Johns chrisj at rtems.org
Thu Apr 21 00:12:01 UTC 2022


On 21/4/2022 4:38 am, Karel Gardas wrote:
> 
> Chris, Frank,
> 
> I don't feel particularly comfortable with this patch. It works here on Ubuntu
> 20.04 with python 3.3 and Ubuntu 22.04 with Python 3.10, but I have to admit
> semantics of this cfg language is beyond my liking and a lot is in complete
> darkness here. E.g. how &&, < operator are working here. Delete both int
> conversions and it'll not work at all. Keep one and it'll work well.
> The problem is probably lack of time to go thorough all the docs on
> docs.rtems.org to find relevant bits while working on the patch.
> 
> Anyway, if you have time, please review and test on your platforms.

I think you are right to be concerned. I cannot find in the parser any support
for `int()` which makes me wonder if `int(3)` is being compared but I am not sure.

Leave this with me and I will take a closer look. I think we will need to add a
numerical compare. Hmmm.

Chris


More information about the devel mailing list