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

Revert "Remove amdadlsdk and amdappsdk." #63638

Merged
merged 1 commit into from Jun 24, 2019

Conversation

oxij
Copy link
Member

@oxij oxij commented Jun 22, 2019

git log

  • Revert "Remove amdadlsdk and amdappsdk."

    This reverts commit 08ea7c7.

    This was a part of Cgminer cleanup #61823.

    Firstly, that commit removed those attributes without removing the files
    they refer to, which is strange.

    Secondly, I have a very old but GPGPU-capabale AMD card which doesn't work
    with newer OpenCL infrastructure and hence needs these packages to do GPGPU.

nix-instantiate environment

  • Host OS: Linux 4.9, SLNOS 19.09
  • Nix: nix-env (Nix) 2.2.2
  • Multi-user: yes
  • Sandbox: yes
  • NIXPKGS_CONFIG:
{
  checkMeta = true;
  doCheckByDefault = true;
}

nix-env -qaP diffs

  • On x86_64-linux:
    • New (6):
      • amdadlsdk
      • amdappsdk26
      • amdappsdk27
      • amdappsdk28
      • amdappsdk
      • amdappsdkFull
  • On aarch64-linux:
    • New (1):
      • amdadlsdk
  • On x86_64-darwin: noop

/cc @ambrop72 @joachifm from that PR

This reverts commit 08ea7c7.

This was a part of NixOS#61823.

Firstly, that commit removed those attributes without removing the files
they refer to, which is strange.

Secondly, I have a very old but GPGPU-capabale AMD card which doesn't work
with newer OpenCL infrastructure and hence needs these packages to do GPGPU.
@matthewbauer matthewbauer merged commit 778bd7a into NixOS:master Jun 24, 2019
@oxij oxij deleted the pkgs/revert-amdsdk-removal branch August 12, 2023 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants