Rewrite the manual page generation script in Python.
This:
Also change the release process to swap gen-manpages and update RC steps, so that the pages will have the correct rc and/or final version.
Does anyone understand this mypy output?
0contrib/devtools/gen-manpages.py:26: error: No overload variant of "run" matches argument types "List[str]", "bool", "bool", "bool" [call-overload]
1contrib/devtools/gen-manpages.py:26: note: Possible overload variants:
2contrib/devtools/gen-manpages.py:26: note: def run(args: Union[Union[bytes, str], Sequence[Union[str, bytes, PathLike[str], PathLike[bytes]]]], bufsize: int = ..., executable: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., stdin: Union[None, int, IO[Any]] = ..., stdout: Union[None, int, IO[Any]] = ..., stderr: Union[None, int, IO[Any]] = ..., preexec_fn: Optional[Callable[[], Any]] = ..., close_fds: bool = ..., shell: bool = ..., cwd: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., env: Union[Mapping[bytes, Union[str, bytes, PathLike[str], PathLike[bytes]]], Mapping[str, Union[str, bytes, PathLike[str], PathLike[bytes]]], None] = ..., universal_newlines: bool = ..., startupinfo: Any = ..., creationflags: int = ..., restore_signals: bool = ..., start_new_session: bool = ..., pass_fds: Any = ..., *, check: bool = ..., encoding: str, errors: Optional[str] = ..., input: Optional[str] = ..., timeout: Optional[float] = ...) -> CompletedProcess[str]
3contrib/devtools/gen-manpages.py:26: note: def run(args: Union[Union[bytes, str], Sequence[Union[str, bytes, PathLike[str], PathLike[bytes]]]], bufsize: int = ..., executable: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., stdin: Union[None, int, IO[Any]] = ..., stdout: Union[None, int, IO[Any]] = ..., stderr: Union[None, int, IO[Any]] = ..., preexec_fn: Optional[Callable[[], Any]] = ..., close_fds: bool = ..., shell: bool = ..., cwd: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., env: Union[Mapping[bytes, Union[str, bytes, PathLike[str], PathLike[bytes]]], Mapping[str, Union[str, bytes, PathLike[str], PathLike[bytes]]], None] = ..., universal_newlines: bool = ..., startupinfo: Any = ..., creationflags: int = ..., restore_signals: bool = ..., start_new_session: bool = ..., pass_fds: Any = ..., *, check: bool = ..., encoding: Optional[str] = ..., errors: str, input: Optional[str] = ..., timeout: Optional[float] = ...) -> CompletedProcess[str]
4contrib/devtools/gen-manpages.py:26: note: <3 more similar overloads not shown, out of 5 total overloads>
5contrib/devtools/gen-manpages.py:31: error: Argument 1 to "join" has incompatible type "Optional[str]"; expected "Union[str, PathLike[str]]" [arg-type]
6contrib/devtools/gen-manpages.py:38: error: Argument 1 to "join" has incompatible type "Optional[str]"; expected "Union[str, PathLike[str]]" [arg-type]
7contrib/devtools/gen-manpages.py:40: error: No overload variant of "run" matches argument types "List[str]", "bool", "bool" [call-overload]
8contrib/devtools/gen-manpages.py:40: note: Possible overload variants:
9contrib/devtools/gen-manpages.py:40: note: def run(args: Union[Union[bytes, str], Sequence[Union[str, bytes, PathLike[str], PathLike[bytes]]]], bufsize: int = ..., executable: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., stdin: Union[None, int, IO[Any]] = ..., stdout: Union[None, int, IO[Any]] = ..., stderr: Union[None, int, IO[Any]] = ..., preexec_fn: Optional[Callable[[], Any]] = ..., close_fds: bool = ..., shell: bool = ..., cwd: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., env: Union[Mapping[bytes, Union[str, bytes, PathLike[str], PathLike[bytes]]], Mapping[str, Union[str, bytes, PathLike[str], PathLike[bytes]]], None] = ..., universal_newlines: bool = ..., startupinfo: Any = ..., creationflags: int = ..., restore_signals: bool = ..., start_new_session: bool = ..., pass_fds: Any = ..., *, check: bool = ..., encoding: str, errors: Optional[str] = ..., input: Optional[str] = ..., timeout: Optional[float] = ...) -> CompletedProcess[str]
10contrib/devtools/gen-manpages.py:40: note: def run(args: Union[Union[bytes, str], Sequence[Union[str, bytes, PathLike[str], PathLike[bytes]]]], bufsize: int = ..., executable: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., stdin: Union[None, int, IO[Any]] = ..., stdout: Union[None, int, IO[Any]] = ..., stderr: Union[None, int, IO[Any]] = ..., preexec_fn: Optional[Callable[[], Any]] = ..., close_fds: bool = ..., shell: bool = ..., cwd: Union[str, bytes, PathLike[str], PathLike[bytes], None] = ..., env: Union[Mapping[bytes, Union[str, bytes, PathLike[str], PathLike[bytes]]], Mapping[str, Union[str, bytes, PathLike[str], PathLike[bytes]]], None] = ..., universal_newlines: bool = ..., startupinfo: Any = ..., creationflags: int = ..., restore_signals: bool = ..., start_new_session: bool = ..., pass_fds: Any = ..., *, check: bool = ..., encoding: Optional[str] = ..., errors: str, input: Optional[str] = ..., timeout: Optional[float] = ...) -> CompletedProcess[str]
11contrib/devtools/gen-manpages.py:40: note: <3 more similar overloads not shown, out of 5 total overloads>
I can’t reproduce it locally with mypy 0.761 and python 3.8.10. I don’t think I’m passing anything weird to subprocess.run or path.join.
Edit: also tried with mypy 0.931. No dice.
Edit.2: FIgured it out, I think. capture_output is new in Python 3.7 so too recent.
Rewrite the manual page generation script in Python.
This:
- Solves '-' stripping issue (fixes #22681)
- Makes that copyright footer is generated again
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
Reviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
Something that is weird, but outside scope of this particular PR to fix. is that the copyright message is emitted inconsistently between binaries:
| binary | -help | -version |
|---|---|---|
| bitcoind | yes | no |
| bitcoin-cli | no | no |
| bitcoin-tx | no | no |
| bitcoin-wallet | no | no |
| bitcoin-util | no | no |
| bitcoin-qt | no | yes |
I don’t have a strong opinion on whether it should be shown with -version or -help, but it probably should be either, and consistent between binaries.
40+ r = subprocess.run([abspath, '--version'], stdout=subprocess.PIPE, universal_newlines=True)
41+ except IOError:
42+ print(f'{abspath} not found or not an executable', file=sys.stderr)
43+ sys.exit(1)
44+ # take first line (which must contain version)
45+ verstr = r.stdout.split('\n')[0]
Nit
0 verstr = r.stdout.splitlines()[0]
35+copyright = None
36+versions = []
37+for relpath in BINARIES:
38+ abspath = os.path.join(builddir, relpath)
39+ try:
40+ r = subprocess.run([abspath, '--version'], stdout=subprocess.PIPE, universal_newlines=True)
check=True here?
Code Review ACK 87f54060ffffdb56c97594efdca378bace5323df
Had to remind myself of the situation but everything looks good, just a few nits that are 100% ignore-able.
