Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove --sdist and --wheel because they are implicit #152

Merged

Conversation

ziegenberg
Copy link
Collaborator

As discussed in PR #145 removing the implicit building options as they
disable the default behavior. According to the documentation:

By default, a source distribution (sdist) is built from {srcdir} and
a binary distribution (wheel) is built from the sdist. This is recommended
as it will ensure the sdist can be used to build wheels.
Pass -s/–sdist and/or -w/–wheel to build a specific distribution.
If you do this, the default behavior will be disabled, and all artifacts
will be built from {srcdir} (even if you combine -w/–wheel with -s/–sdist,
the wheel will be built from {srcdir}).

As discussed in PR pycontribs#145 removing the implicit building options as they
disable the default behavior. According to the documentation:

	By default, a source distribution (sdist) is built from {srcdir} and
	a binary distribution (wheel) is built from the sdist. This is recommended
	as it will ensure the sdist can be used to build wheels.
	Pass -s/–sdist and/or -w/–wheel to build a specific distribution.
	If you do this, the default behavior will be disabled, and all artifacts
	will be built from {srcdir} (even if you combine -w/–wheel with -s/–sdist,
	the wheel will be built from {srcdir}).

Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
@ziegenberg ziegenberg added the skip-changelog Can be missed from the changelog. label Nov 2, 2021
@ssbarnea ssbarnea merged commit c8efc47 into pycontribs:main Nov 2, 2021
@ziegenberg ziegenberg deleted the adjust-build-to-follow-recommendation branch November 2, 2021 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog Can be missed from the changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants