Skip to content

bpo-32429: Abort compilation on outdated Modules/Setup file.#5015

Closed
JulienPalard wants to merge 1 commit intopython:masterfrom
JulienPalard:modules_setup_warning
Closed

bpo-32429: Abort compilation on outdated Modules/Setup file.#5015
JulienPalard wants to merge 1 commit intopython:masterfrom
JulienPalard:modules_setup_warning

Conversation

@JulienPalard
Copy link
Copy Markdown
Member

@JulienPalard JulienPalard commented Dec 26, 2017

Copy link
Copy Markdown
Contributor

@xdegaye xdegaye left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, see my comment

Comment thread Makefile.pre.in
echo "check to make sure you have all the updates you"; \
echo "need in your Modules/Setup file."; \
echo "Usually, copying Modules/Setup.dist to Modules/Setup will work."; \
echo "Or if you want to keep your modifications, touch Modules/Setup"; \
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When Modules/Setup.dist has been changed as for example by 7d895ac in September 2016, touching Modules/Setup is not sufficient. I suggest to remove that line.

@taleinat
Copy link
Copy Markdown
Contributor

taleinat commented Sep 6, 2018

Given that PR #8229 (bpo-32430) has been merged, ISTM this should be closed.

@JulienPalard
Copy link
Copy Markdown
Member Author

That's right.

@JulienPalard JulienPalard deleted the modules_setup_warning branch June 16, 2019 14:06
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