Skip to content

bpo-37705: Remove orphaned PC/errmap.mak#29724

Merged
miss-islington merged 1 commit into
python:mainfrom
arhadthedev:clean-after-bpo-37705
Feb 2, 2022
Merged

bpo-37705: Remove orphaned PC/errmap.mak#29724
miss-islington merged 1 commit into
python:mainfrom
arhadthedev:clean-after-bpo-37705

Conversation

@arhadthedev

@arhadthedev arhadthedev commented Nov 23, 2021

Copy link
Copy Markdown
Member

After GH-15623 deleted generrmap.c, a related mak-file stopped working. The mak contains generrmap-related rules only so it should be removed altogether.

Further search for errmap\.mak|generrmap regex through content of CPython files shows no dangling reference left.

Since generrmap is already effectively removed, this pull request contains no blurp.

https://bugs.python.org/issue37705

Automerge-Triggered-By: GH:zware

After pythonGH-15623 deleted `generrmap.c`, a related mak-file stopped
working. The mak contains generrmap-related rules only so it should be
removed altogether.

Further search for `errmap\.mak|generrmap` regex through content
of CPython files shows no dangling reference left.
@arhadthedev
arhadthedev requested a review from a team as a code owner November 23, 2021 11:19
@arhadthedev arhadthedev changed the title bpo-37705: Remove stray PC/errmap.mak bpo-37705: Remove orphaned PC/errmap.mak Dec 5, 2021
@arhadthedev

Copy link
Copy Markdown
Member Author

I believe this PR does not require a separate NEWS entry because the removed file was not functional anyway so it has no interested parties.

@zware zware added the skip news label Dec 5, 2021
@zware

zware commented Feb 2, 2022

Copy link
Copy Markdown
Member

To test a recent fix to @miss-islington, @JelleZijlstra will be adding the automerge label shortly.

@zware

zware commented Feb 2, 2022

Copy link
Copy Markdown
Member

We now have a well-behaved bot :). Thanks @JelleZijlstra!

And thanks @arhadthedev for the patch, which I'll now properly merge :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants