Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx> writes: > Hi Jon/Greg, > > This series replace get_abi.pl with a Python version. > > I originally started it due to some issues I noticed when searching for > ABI symbols. While I could just go ahead and fix the already existing > script, I noticed that the script maintainance didn't have much care over > all those years, probably because it is easier to find Python programmers > those days. > > Also, the code is complex and was not using modules or classes and > were using lots of global variables. > > So, I decided to rewrite it in Python. I started with a manual conversion > for each function. Yet, to avoid future maintainership issues, I opted to > divide the main code on three classes, each on a sepaparate file. > > Just like the original RFC, I opted to keep the Sphinx kernel-abi module > on three different phases: > > - call get_abi.py as an exec file; > - import AbiParser on a minimal integration scenario; > - cleanup the code to avoid needing to parse line numbers from the text. > > This way, if something goes wrong, it would be easier to just revert any > offending patches, It also provides a better rationale about what each > logical change is doing. > > The initial patches on this series do some preparation work and > cleans some ABI symbol bugs that lack ":" delimiter. > > I opted to place on this series the Sphinx and Python version updates. > > I still have ~10 patches here with additional cleanups, from the original > series I sent as RFC but let's get the main changes merged first. OK, I have applied this set - it seems to work for me, though it does lead to some changes in the organization of Documentation/admin-guide/abi.html in the output. It would be nice if, eventually, we could put the README link up at the top rather than under "ABI file", or even just include its contents there directly. Anyway, let's see how this goes :) Thanks, jon
