From 62f5d5222cd49babc8c90d6f3c1613abc4b0162e Mon Sep 17 00:00:00 2001
From: zi3m5f <113244000+zi3m5f@users.noreply.github.com>
Date: Tue, 19 May 2026 21:09:41 +0200
Subject: [PATCH] sambaFull: fix samba-tool missing cryptography module

samba-tool depends on cryptography python module so add it to pythonPath
if enableDomainController and also check with versionSambaTool test.

(cherry picked from commit a59124a4b0d5c8a4eb07da90f10187768d0091f3)
---
 pkgs/servers/samba/4.x.nix | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/pkgs/servers/samba/4.x.nix b/pkgs/servers/samba/4.x.nix
index 22f9cbf3aba3..5ca73ec6c351 100644
--- a/pkgs/servers/samba/4.x.nix
+++ b/pkgs/servers/samba/4.x.nix
@@ -285,6 +285,9 @@ stdenv.mkDerivation (finalAttrs: {
     python3Packages.dnspython
     python3Packages.markdown
     tdb
+  ]
+  ++ lib.optionals enableDomainController [
+    python3Packages.cryptography
   ];
 
   strictDeps = true;
@@ -348,6 +351,13 @@ stdenv.mkDerivation (finalAttrs: {
       command = "${finalAttrs.finalPackage}/bin/smbd -V";
       package = finalAttrs.finalPackage;
     };
+  }
+  // lib.optionalAttrs enableDomainController {
+    versionSambaTool = testers.testVersion {
+      command = "${lib.getExe' finalAttrs.finalPackage "samba-tool"} --version";
+      package = finalAttrs.finalPackage;
+      version = finalAttrs.version;
+    };
   };
 
   meta = {