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

Archive thrust and cub feedstocks #702

Merged
merged 1 commit into from
Apr 7, 2023
Merged

Conversation

kkraus14
Copy link
Contributor

@kkraus14 kkraus14 commented Apr 6, 2023

Checklist:

  • I want to mark a package as broken (or not broken):

    • Make sure your package is in the right spot (broken/* for adding the
      broken label, not_broken/* for removing the broken label, or token_reset/*
      for token resets)
    • Added a description of the problem with the package in the PR description.
    • Pinged the team for the package for their input.
  • I want to archive a feedstock:

    • Pinged the team for that feedstock for their input.
    • Make sure you have opened an issue on the feedstock explaining why it was archived.
    • Linked that issue in this PR description.
    • Added links to any other relevant issues/PRs in the PR description.

Once the cuda-cccl recipe is merged (conda-forge/staged-recipes#21953) there is agreement across all maintainers that we should archive the thrust and cub feedstocks. Please see the issue here for agreement: conda-forge/thrust-feedstock#19.

For additional context: CCCL contains thrust, cub, and libcudacxx and it has been communicated by the CCCL team at NVIDIA that they intend to further blend the three over time where it doesn't make sense to keep them separated.


Fixes conda-forge/thrust-feedstock#19
Fixes conda-forge/cub-feedstock#24

@leofang
Copy link
Member

leofang commented Apr 6, 2023

cc: @conda-forge/thrust & @conda-forge/cub for vis

@jakirkham
Copy link
Member

Noted issues to close in the OP

@jakirkham
Copy link
Member

cc @adibbley @bdice @robertmaynard (for awareness)

@jakirkham jakirkham merged commit 7c297d2 into conda-forge:main Apr 7, 2023
@jakirkham
Copy link
Member

Thanks Keith for the PR and Leo for the review! 🙏

@leofang
Copy link
Member

leofang commented Apr 7, 2023

btw I pinned both issues in the respective feedstocks to make it clear.

@leofang
Copy link
Member

leofang commented Apr 7, 2023

(got it done right before they got archived!)

@jakirkham
Copy link
Member

Thanks Leo! 🙏

FWIW we can unarchive using the same process. So if we need to make other small changes, there's a way to do that.

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

Successfully merging this pull request may close these issues.

Sunsetting this feedstock? Sunsetting this feedstock?
3 participants