Skip to content

Remove cryptography (depends on #5952) #5618

Closed

Description

@srittau

cryptography added py.typed in release 3.4.4, released 2021-02-09.

Activity

  1. changed the title [-]Remove cryptography (August 2021)[/-] [+]Remove cryptography (not before August 2021)[/+] on Jun 11, 2021
  2. changed the title [-]Remove cryptography (not before August 2021)[/-] [+]Remove cryptography (not before September 2021)[/+] on Jun 11, 2021
  3. hauntsaninja commented on Jun 11, 2021

    @hauntsaninja
    Collaborator

    One fun and different thing about types-cryptography is that other typeshed packages depend on it: paramiko, PyJWT, openssl-python.

    So we will need to ensure stub packages can depend on non-typeshed-stub-packages.

  4. hauntsaninja commented on Jun 17, 2021

    @hauntsaninja
    Collaborator

    Just in case, does anyone feel strongly against allowing stub packages from depending on real PEP 561 packages? cc @JukkaL

  5. srittau commented on Jun 17, 2021

    @srittau
    CollaboratorAuthor

    What do you mean by "real" PEP 561 packages?

  6. Akuli commented on Jun 17, 2021

    @Akuli
    Collaborator

    Perhaps packages that contain .py files and a py.typed file.

  7. hauntsaninja commented on Jun 17, 2021

    @hauntsaninja
    Collaborator

    Yup, I started with "real packages", clarified to "PEP 561 packages", but missed editing out the "real". Basically packages that can be arbitrarily big, do complicated setup.py things, be system dependent, yada yada

  8. srittau commented on Jun 17, 2021

    @srittau
    CollaboratorAuthor

    Not speaking about stubs packages in general, but I think it's reasonable if typeshed packages depend on dependencies of the corresponding upstream package. I.e. if package foo depends on cryptography, it's fine that our foo-types depends on cryptography as well, since it doesn't introduce "new" dependencies. Apart from that we should vet dependencies carefully.

  9. JukkaL commented on Jun 18, 2021

    @JukkaL
    Contributor

    I agree with @srittau.

    I think that it would be sufficient to maintain an allowlist of vetted dependencies. I'd expect that the number of such dependencies will be fairly small, so it shouldn't be much effort to maintain.

  10. srittau commented on Aug 24, 2021

    @srittau
    CollaboratorAuthor

    Blocked by #5952.

  11. Akuli commented on Dec 5, 2021

    @Akuli
    Collaborator

    What is the current status of this? Can we now make other packages depend on cryptography?

  12. changed the title [-]Remove cryptography (not before September 2021)[/-] [+]Remove cryptography (depends on #5952)[/+] on Apr 16, 2022
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

    stubs: removalPending removal of third-party distributions

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions