Skip to content

hashcat: add AMD ROCM support#435291

Merged
prusnak merged 1 commit intoNixOS:masterfrom
99jte:hashcat-rocm-support
Aug 23, 2025
Merged

hashcat: add AMD ROCM support#435291
prusnak merged 1 commit intoNixOS:masterfrom
99jte:hashcat-rocm-support

Conversation

@99jte
Copy link
Copy Markdown
Contributor

@99jte 99jte commented Aug 20, 2025

Just copy what's already done for CUDA. Will enable it when config.rocmSupport is set.

N.b. some of the Python scripts fail (e.g. apfs2hashcat.py) on bad dependencies but that is not caused by this PR, and actual HashCat works just fine.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

Add a 👍 reaction to pull requests you find important.

Just copy what's already done for CUDA.
@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. 9.needs: reviewer This PR currently has no reviewers requested and needs attention. labels Aug 20, 2025
@prusnak prusnak merged commit 57b1c9f into NixOS:master Aug 23, 2025
29 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

9.needs: reviewer This PR currently has no reviewers requested and needs attention. 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants