Skip to content

FIX: skip version update on build_ext if .py does not exist#297

Merged
effigies merged 1 commit into
python-versioneer:masterfrom
mgorny:build_py-before-build_ext
Apr 26, 2022
Merged

FIX: skip version update on build_ext if .py does not exist#297
effigies merged 1 commit into
python-versioneer:masterfrom
mgorny:build_py-before-build_ext

Conversation

@mgorny

@mgorny mgorny commented Apr 19, 2022

Copy link
Copy Markdown
Contributor

Skip version update on build_ext command if the respective version
file does not exist. This could be the case if build_ext is called
explicitly without calling build_py prior to it.

Closes #296

@mgorny mgorny force-pushed the build_py-before-build_ext branch from a3a792c to a394e75 Compare April 19, 2022 16:13
@mgorny mgorny changed the title FIX: ensure that build_py is called prior to build_ext FIX: skip version update on build_ext if .py does not exist Apr 19, 2022
@effigies

Copy link
Copy Markdown
Contributor

@pkittenis Could I bug you for a review here, since you added the build_ext implementation? I'm curious if this is going to cause problems for any prominent use cases.

@effigies effigies left a comment

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.

In the absence of another review, I'll go ahead and merge, but can we make the message more explicit?

Comment thread src/cmdclass.py Outdated
@mgorny mgorny force-pushed the build_py-before-build_ext branch from 0b23523 to ec37649 Compare April 26, 2022 15:33
@mgorny

mgorny commented Apr 26, 2022

Copy link
Copy Markdown
Contributor Author

Thanks, squashed that change into my commit.

@mgorny

mgorny commented Apr 26, 2022

Copy link
Copy Markdown
Contributor Author

Hmm, I see I need to shorten the lines now. I'll do that in a minute.

@effigies

Copy link
Copy Markdown
Contributor

Thanks. Sorry about that.

Skip version update on `build_ext` command if the respective version
file does not exist.  This could be the case if `build_ext` is called
explicitly without calling `build_py` prior to it.

Closes python-versioneer#296
@mgorny mgorny force-pushed the build_py-before-build_ext branch from ec37649 to ff62f26 Compare April 26, 2022 16:26
@mgorny

mgorny commented Apr 26, 2022

Copy link
Copy Markdown
Contributor Author

No problem. Pushed a fix now.

@effigies

Copy link
Copy Markdown
Contributor

Thanks!

@effigies effigies merged commit 280fb6c into python-versioneer:master Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

build_ext fails if build_py was not called before

2 participants