[S] Change in flashrom[main]: Makefile,meson.build: Add support for Sphinx versions prior to 4.x

[Anton Samsonov (Code Review)]

Attention is currently required from: Anastasia Klimchuk, Anton Samsonov, Peter Marheine, Thomas Heijligen.

Anton Samsonov has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/77778?usp=email )

Change subject: Makefile,meson.build: Add support for Sphinx versions prior to 4.x
......................................................................

Patch Set 6:

(3 comments)

Commit Message:

https://review.coreboot.org/c/flashrom/+/77778/comment/0a8fb8eb_d3f23d71 :
PS3, Line 7: 3
> Oh I see! Maybe you can rename the title as  […]

Acknowledged

File doc/sphinx-wrapper.sh:

https://review.coreboot.org/c/flashrom/+/77778/comment/95a8f73a_86adf847 :
PS5, Line 8: "$@"
> `$@` means all the command-line parameters, but the parameters needed for this invocation are starting from $5 and until the end?

Yes, `$@` means all the parameters [that are still left]. The script's self path, (denoted by 'sphinx_wrapper' in meson.build), is `$0`. Then there are 2 additional parameters we capture as 'OUTPUT_HOME' and 'MAN_OUTPUTS' in the script, and then `sphinx-build` full path (denoted by 'sphinx' in meson.build), — all 3 are cut from `$@` by `shift 3`, so that only arguments for `sphinx-build` are left.

https://review.coreboot.org/c/flashrom/+/77778/comment/b11042fc_95c948aa :
PS5, Line 14:
> I know this info is in commit message…

I thought this was self-explanatory from `[ "${SPHINXBUILD_MAJOR}" -ge 4 ]`, but you are right in that there is no such thing as a over-commenting. OK, I'll add it a bit later.