◐ Shell
reader mode source ↗
Skip to content

bpo-36876: Add a tool that identifies unsupported global C variables.#15877

Merged
ericsnowcurrently merged 118 commits into
python:masterfrom
ericsnowcurrently:globals-checker
Sep 11, 2019
Merged

bpo-36876: Add a tool that identifies unsupported global C variables.#15877
ericsnowcurrently merged 118 commits into
python:masterfrom
ericsnowcurrently:globals-checker

Conversation

@ericsnowcurrently

@ericsnowcurrently ericsnowcurrently commented Sep 10, 2019 •
edited
Loading

Copy link
Copy Markdown
Member

This PR does 2 things:

  • make the globals checker tool more robust and maintainable
  • add a test that invokes the checker and fails if any unsupported globals have been added

For now the test is marked as a known failure. I'll be working to fix all of unsupported globals, at which point I'll unmark the test.

Note that this PR adds code to Tools/c-analyzer or adds new tests. The only existing code I touch is where I factor out an existing testing helper from tests for another tool.

https://bugs.python.org/issue36876

@vstinner

vstinner commented Sep 10, 2019

Copy link
Copy Markdown
Member

115 commits, 51 files changes, and wow, that's a huge PR.

@vstinner

vstinner commented Sep 10, 2019 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

" bpo-36876: Make sure no new globals get added."

So far, the C API don't provide any solution for function "static PyObject *local_global_var = ...;" :-)

@ericsnowcurrently ericsnowcurrently changed the title bpo-36876: Make sure no new globals get added. Sep 11, 2019
@ericsnowcurrently

ericsnowcurrently commented Sep 11, 2019

Copy link
Copy Markdown
Member Author

Yeah, it's a lot, but it's all under Tools/. :) Also, I've updated the title to be more clear.

@ericsnowcurrently

ericsnowcurrently commented Sep 11, 2019

Copy link
Copy Markdown
Member Author

FYI, all the failures are strictly with patchcheck, which is complaining about whitespace in some files. I cannot reproduce that locally.

@ericsnowcurrently ericsnowcurrently merged commit ee536b2 into python:master Sep 11, 2019
@ericsnowcurrently ericsnowcurrently deleted the globals-checker branch September 11, 2019 18:49
@bedevere-bot

bedevere-bot commented Sep 11, 2019

Copy link
Copy Markdown

@ericsnowcurrently: Please replace # with GH- in the commit message next time. Thanks!

82 hidden items Load more…
@ericsnowcurrently
Distinguish "static" vars in output.
48b2460
@ericsnowcurrently
Special-case variables named "id".
c0a631b
@ericsnowcurrently
Fix a test.
3989a69
@ericsnowcurrently
Print totals.
8118454
@ericsnowcurrently
Fail if we couldn't find any of the symbols.
cae5e3a
@ericsnowcurrently
Factor out _check_results().
87ebe87
@ericsnowcurrently
Fill in gaps in known.tsv.
91e4fb1
@ericsnowcurrently
Mark _Py_IDENTIFIER() as unsupported.
21c4175
@ericsnowcurrently
Keep "static" in output.
a245849
@ericsnowcurrently
Fix a typo in __main__.py.
1437d56
@ericsnowcurrently
Honor provided dirnames.
7feea9d
@ericsnowcurrently
Fix tests.
ae23801
@ericsnowcurrently
Deal with default dirnames properly.
26aaccb
@ericsnowcurrently
Supporting hiding objects in output.
4aaad5f
@ericsnowcurrently
Recognize more object types.
3d36cfc
@ericsnowcurrently
Ignore known non-statics.
b7420d5
@ericsnowcurrently
Always support "static const" (non-object) variables.
da151a7
@ericsnowcurrently
"statics" -> "globals".
6470eb1
@ericsnowcurrently
Maybe limit the variables in known.tsv.
1233cf9
@ericsnowcurrently
Generate the ignored.tsv file.
fc6c97e
@ericsnowcurrently
Ignore variables with benign races.
6f8a223
@ericsnowcurrently
Mark more variables as PyObject.
6c1db44
@ericsnowcurrently
Consider private, non-static globals.
5888d48
@ericsnowcurrently
Update ignored global variables.
cdeb1d9
@ericsnowcurrently
Skip the check where "nm" isn't available.
569c57e
@ericsnowcurrently
Ignore REPL-related variables.
7b0745f
@ericsnowcurrently
Fix whitespace.
b5dd31b
DinoV pushed a commit to DinoV/cpython that referenced this pull request Sep 12, 2019
@ericsnowcurrently @DinoV
bpo-36876: Add a tool that identifies unsupported global C variables. (… …
7654be1 
…python#15877)
@ericsnowcurrently ericsnowcurrently mentioned this pull request Apr 10, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ericsnowcurrently @vstinner @bedevere-bot @the-knights-who-say-ni