Skip to content

shell.nix fix#5995

Open
banderlog wants to merge 1 commit into
openwall:bleeding-jumbofrom
banderlog:banderlog-patch-1
Open

shell.nix fix#5995
banderlog wants to merge 1 commit into
openwall:bleeding-jumbofrom
banderlog:banderlog-patch-1

Conversation

@banderlog
Copy link
Copy Markdown

Current shell.nix is broken, problem and fix were stated here: https://discourse.nixos.org/t/how-to-build-john-the-ripper-with-opencl-for-nvidia/42292

Also:

  • file did not include a way to build it NVIDIA compatible
  • python package for office2john.py was missing

Comment thread shell.nix Outdated
# then you can add 'intel-ocl' entry to 'nativeBuildInputs' below. Please
# note however if you are running and intel based system and remove intel-ocl then
# you will not have opencl support when building openwall/john. For the avoidance of
# doubt, this file does not contain any unfree software.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably remove the sentence "Please note ..." along with this update. There's also a typo in "and" in that sentence (which would be gone along with it).

@solardiz
Copy link
Copy Markdown
Member

solardiz commented Jun 4, 2026

Please don't add commits - amend your first and only commit, and force-push. We don't want temporary history internal to one PR to stay in the project.

@banderlog banderlog force-pushed the banderlog-patch-1 branch from 43b64c6 to 143d5e3 Compare June 5, 2026 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants