unofficial mirror of guix-patches@gnu.org 
 help / color / mirror / code / Atom feed
* [bug#68431] [PATCH 1/3] gnu: Add python-krb5.
  2024-01-13 18:09 [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build Troy Figiel
@ 2024-01-13 17:58 ` Troy Figiel
  2024-01-13 17:59 ` [bug#68431] [PATCH 2/3] gnu: python-requests-kerberos: Fix build Troy Figiel
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Troy Figiel @ 2024-01-13 17:58 UTC (permalink / raw)
  To: 68431

* gnu/packages/python-xyz.scm (python-krb5): New variable.
---
 gnu/packages/python-xyz.scm | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/gnu/packages/python-xyz.scm b/gnu/packages/python-xyz.scm
index 37d7dd0dd5..41b7bdabcf 100644
--- a/gnu/packages/python-xyz.scm
+++ b/gnu/packages/python-xyz.scm
@@ -19500,6 +19500,25 @@ (define-public python-inflection
 underscored string.")
     (license license:expat)))
 
+(define-public python-krb5
+  (package
+    (name "python-krb5")
+    (version "0.5.1")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (pypi-uri "krb5" version))
+       (sha256
+        (base32 "1jivws8dsscsmyqngbc71r4qw8qmn98pqmhylz01r5dd1ljfw9bi"))))
+    (build-system pyproject-build-system)
+    (inputs (list mit-krb5))
+    (native-inputs (list python-cython python-k5test python-pytest))
+    (home-page "https://github.com/jborean93/pykrb5")
+    (synopsis "Kerberos API bindings for Python")
+    (description "This library provides Python functions that wraps the
+Kerberos 5 C API.")
+    (license license:expat)))
+
 (define-public python-pylev
   (package
     (name "python-pylev")
-- 
2.42.0





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#68431] [PATCH 2/3] gnu: python-requests-kerberos: Fix build.
  2024-01-13 18:09 [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build Troy Figiel
  2024-01-13 17:58 ` [bug#68431] [PATCH 1/3] gnu: Add python-krb5 Troy Figiel
@ 2024-01-13 17:59 ` Troy Figiel
  2024-01-13 18:04 ` [bug#68431] [PATCH 3/3] gnu: python-requests-kerberos: Enable tests Troy Figiel
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: Troy Figiel @ 2024-01-13 17:59 UTC (permalink / raw)
  To: 68431

* gnu/packages/python-web.scm (python-requests-kerberos)[propagated-inputs]:
Add python-gssapi and python-krb5.
---
 gnu/packages/python-web.scm | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
index 567b01f396..e13716081f 100644
--- a/gnu/packages/python-web.scm
+++ b/gnu/packages/python-web.scm
@@ -3307,8 +3307,15 @@ (define-public python-requests-kerberos
                (base32
                 "1lbgjs779cjxhz07lfl9dbyp3qfh8a3ir2393042wfg518jd3afd"))))
     (build-system python-build-system)
-    (propagated-inputs (list python-cryptography python-pyspnego
-                             python-requests))
+    (propagated-inputs (list python-cryptography
+                             python-pyspnego
+                             python-requests
+
+                             ;; Required through pyspnego[kerberos]. Only
+                             ;; `python-krb5' is directly used in the source
+                             ;; code though.
+                             python-gssapi
+                             python-krb5))
     (home-page "https://github.com/requests/requests-kerberos")
     (synopsis "Kerberos authentication handler for python-requests")
     (description "This package provides a Kerberos authentication handler for
-- 
2.42.0





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#68431] [PATCH 3/3] gnu: python-requests-kerberos: Enable tests.
  2024-01-13 18:09 [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build Troy Figiel
  2024-01-13 17:58 ` [bug#68431] [PATCH 1/3] gnu: Add python-krb5 Troy Figiel
  2024-01-13 17:59 ` [bug#68431] [PATCH 2/3] gnu: python-requests-kerberos: Fix build Troy Figiel
@ 2024-01-13 18:04 ` Troy Figiel
  2024-01-13 18:30 ` [bug#68431] Typo Troy Figiel
  2024-01-15 22:02 ` bug#68431: Sharlatan Hellseher
  4 siblings, 0 replies; 6+ messages in thread
From: Troy Figiel @ 2024-01-13 18:04 UTC (permalink / raw)
  To: 68431

* gnu/packages/python-web.scm (python-requests-kerberos): Enable tests.
[source]: Replace the PyPI release by the Github release.
[arguments]<#:phases>: Replace the default check phase with a pytest
invocation.
[native-inputs]: Add python-pytest and python-pytest-mock.
---
 gnu/packages/python-web.scm | 23 +++++++++++++++++------
 1 file changed, 17 insertions(+), 6 deletions(-)

diff --git a/gnu/packages/python-web.scm b/gnu/packages/python-web.scm
index e13716081f..7b25bee15a 100644
--- a/gnu/packages/python-web.scm
+++ b/gnu/packages/python-web.scm
@@ -3300,13 +3300,23 @@ (define-public python-requests-kerberos
   (package
     (name "python-requests-kerberos")
     (version "0.14.0")
-    (source (origin
-              (method url-fetch)
-              (uri (pypi-uri "requests-kerberos" version))
-              (sha256
-               (base32
-                "1lbgjs779cjxhz07lfl9dbyp3qfh8a3ir2393042wfg518jd3afd"))))
+    ;; No tests in the PyPI tarball.
+    (source
+     (origin
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/requests/requests-kerberos")
+             (commit (string-append "v" version))))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0s30pcnlir3j2jmf7yh065f294cf3x0x5i3ldskn8mm0a3657mv3"))))
     (build-system python-build-system)
+    (arguments
+     '(#:phases (modify-phases %standard-phases
+                  (replace 'check
+                    (lambda* (#:key tests? #:allow-other-keys)
+                      (when tests?
+                        (invoke "pytest" "-vv")))))))
     (propagated-inputs (list python-cryptography
                              python-pyspnego
                              python-requests
@@ -3316,6 +3326,7 @@ (define-public python-requests-kerberos
                              ;; code though.
                              python-gssapi
                              python-krb5))
+    (native-inputs (list python-pytest python-pytest-mock))
     (home-page "https://github.com/requests/requests-kerberos")
     (synopsis "Kerberos authentication handler for python-requests")
     (description "This package provides a Kerberos authentication handler for
-- 
2.42.0





^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build
@ 2024-01-13 18:09 Troy Figiel
  2024-01-13 17:58 ` [bug#68431] [PATCH 1/3] gnu: Add python-krb5 Troy Figiel
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: Troy Figiel @ 2024-01-13 18:09 UTC (permalink / raw)
  To: 68431

This patch series fixes the broken python-requests-kerberos build and enables tests along the way.

Troy Figiel (3):
  gnu: Add python-krb5.
  gnu: python-requests-kerberos: Fix build.
  gnu: python-requests-kerberos: Enable tests.

 gnu/packages/python-web.scm | 34 ++++++++++++++++++++++++++--------
 gnu/packages/python-xyz.scm | 19 +++++++++++++++++++
 2 files changed, 45 insertions(+), 8 deletions(-)


base-commit: 31e736dac29e6f0ff8a2de3f28e210a68684ad21
-- 
2.42.0





^ permalink raw reply	[flat|nested] 6+ messages in thread

* [bug#68431] Typo
  2024-01-13 18:09 [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build Troy Figiel
                   ` (2 preceding siblings ...)
  2024-01-13 18:04 ` [bug#68431] [PATCH 3/3] gnu: python-requests-kerberos: Enable tests Troy Figiel
@ 2024-01-13 18:30 ` Troy Figiel
  2024-01-15 22:02 ` bug#68431: Sharlatan Hellseher
  4 siblings, 0 replies; 6+ messages in thread
From: Troy Figiel @ 2024-01-13 18:30 UTC (permalink / raw)
  To: 68431

There is a small typo in the description of python-krb5: sed
's/wraps/wrap/'. I see the typo comes from the upstream README.

Best wishes,

Troy




^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#68431:
  2024-01-13 18:09 [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build Troy Figiel
                   ` (3 preceding siblings ...)
  2024-01-13 18:30 ` [bug#68431] Typo Troy Figiel
@ 2024-01-15 22:02 ` Sharlatan Hellseher
  4 siblings, 0 replies; 6+ messages in thread
From: Sharlatan Hellseher @ 2024-01-15 22:02 UTC (permalink / raw)
  To: 68431-done; +Cc: troy

Hi,

Thank you for your work.

I've slightly adjusted them and pushed as
19db1551dc6f6180d2cda9084c2dec37bf4923c9.

- 1/3 gnu: Add python-krb5.
  It was applied without modifications.

- 2/3 gnu: python-requests-kerberos: Fix build.
  I've included only python-gssapi, and added python-krb5 as propagated input in
  python-pyspnego.

- 3/3 gnu: python-requests-kerberos: Enable tests.
  I've used pyproject-build-system, which does not require modifying 'check
  phase.

Thanks,
Oleg

-- 
VCS: https://github.incerto.xyz/; https://git.sr.ht/~hellseher/
GPG: 9847 81DE 689C 21C2 6418 0867 76D7 27BF F62C D2B5

… наш разум - превосходная объяснительная машина которая способна
найти смысл почти в чем угодно, истолковать любой феномен, но
совершенно не в состоянии принять мысль о непредсказуемости.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-01-15 22:03 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-13 18:09 [bug#68431] [PATCH 0/3] Fix python-requests-kerberos build Troy Figiel
2024-01-13 17:58 ` [bug#68431] [PATCH 1/3] gnu: Add python-krb5 Troy Figiel
2024-01-13 17:59 ` [bug#68431] [PATCH 2/3] gnu: python-requests-kerberos: Fix build Troy Figiel
2024-01-13 18:04 ` [bug#68431] [PATCH 3/3] gnu: python-requests-kerberos: Enable tests Troy Figiel
2024-01-13 18:30 ` [bug#68431] Typo Troy Figiel
2024-01-15 22:02 ` bug#68431: Sharlatan Hellseher

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/guix.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).