Skip to content

provide configuration option to change sdist build method #252

Closed

Description

@dhellmann

We have several packages that just need to have pep517_build_sdist() called to produce an sdist instead of rolling up the directory after patches are applied. To do that today someone has to write an override method. We should make that a configuration option for a package.

Activity

  1. tiran commented on Jul 26, 2024

    @tiran
    Collaborator

    I would make PEP 517 the default and add an option to use legacy sdist function.

  2. dhellmann commented on Jul 26, 2024

    @dhellmann
    MemberAuthor

    I haven't tried to use pep517 in an unpacked sdist, does that work?

  3. self-assigned this
    on Jul 26, 2024
  4. removed their assignment
    on Aug 21, 2024
  5. LalatenduMohanty commented on Nov 12, 2025

    @LalatenduMohanty
    Member

    @tiran @dhellmann looks like we have some related issues. lets consolidate these

  6. mprpic commented on Nov 5, 2026

    @mprpic
    Contributor

    Closing as done: with the source: configuration from #1414, a package can set build_sdist: pep517 to use the PEP 517 hook instead of a tarball.

    If you feel this change is still needed, feel free to reopen this issue. Thank you!

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions