all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#68631] [PATCH] gnu: Add python-sphinx-issues.
@ 2024-01-21 10:11 Troy Figiel
  2024-01-21 10:11 ` [bug#68631] [PATCH v3] " Troy Figiel
                   ` (4 more replies)
  0 siblings, 5 replies; 8+ messages in thread
From: Troy Figiel @ 2024-01-21 10:11 UTC (permalink / raw)
  To: 68631

* gnu/packages/sphinx.scm (python-sphinx-issues): New variable.
---
 gnu/packages/sphinx.scm | 43 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/gnu/packages/sphinx.scm b/gnu/packages/sphinx.scm
index 7e5a929241..64c97afd8e 100644
--- a/gnu/packages/sphinx.scm
+++ b/gnu/packages/sphinx.scm
@@ -18,6 +18,7 @@
 ;;; Copyright © 2021, 2022 Vinicius Monego <monego@posteo.net>
 ;;; Copyright © 2021 Hugo Lecomte <hugo.lecomte@inria.fr>
 ;;; Copyright © 2021, 2022 Maxim Cournoyer <maxim.cournoyer@gmail.com>
+;;; Copyright © 2024 Troy Figiel <troy@troyfigiel.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -372,6 +373,48 @@ (define-public python-sphinxcontrib-newsfeed
     (home-page "https://bitbucket.org/prometheus/sphinxcontrib-newsfeed")
     (license license:bsd-2)))
 
+(define-public python-sphinx-issues
+  (package
+    (name "python-sphinx-issues")
+    (version "4.0.0")
+    (source
+     (origin
+       ;; No tests in the PyPI tarball.
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/sloria/sphinx-issues")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0q4as8gibvin0n6h5y1q4cwz3b1nwgs0idfc94dbndx42pjiz1vn"))))
+    (build-system pyproject-build-system)
+    (arguments
+     (list
+      #:phases #~(modify-phases %standard-phases
+                   (add-before 'check 'patch-sphinx-build-path
+                     (lambda _
+                       ;; The path to the sphinx-build binary is hardcoded to
+                       ;; be in the same directory as the python
+                       ;; executable. That does not work when building the
+                       ;; package.
+                       (substitute* "tests/test_sphinx_issues.py"
+                         (((string-append "Path\\(sys\\.executable\\)"
+                                          "\\.parent\\.joinpath\\"
+                                          "(\"sphinx-build\"\\)"))
+                          (string-append "\""
+                                         #$(this-package-native-input
+                                            "python-sphinx")
+                                         "/bin/sphinx-build\""))))))))
+    (native-inputs (list python-flit-core python-pytest python-sphinx))
+    (home-page "https://github.com/sloria/sphinx-issues")
+    (synopsis "Sphinx extension for linking to your project's issue tracker")
+    (description
+     "This package provides a Sphinx extension for linking to your project's
+issue tracker.  This includes roles for linking to issues, pull requests and
+user profiles.  Support for GitHub is built-in, but other services can also be
+supported with @code{sphinx-issues}.")
+    (license license:expat)))
+
 (define-public python-sphinx-panels
   (package
     (name "python-sphinx-panels")

base-commit: c79ffe25e98607d6803f960d5187e4098e1dc7c2
-- 
2.42.0





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

* [bug#68631] [PATCH v2] gnu: Add python-sphinx-issues.
  2024-01-21 10:11 [bug#68631] [PATCH] gnu: Add python-sphinx-issues Troy Figiel
  2024-01-21 10:11 ` [bug#68631] [PATCH v3] " Troy Figiel
@ 2024-01-21 10:11 ` Troy Figiel
  2024-01-25 22:54   ` Troy Figiel
  2024-01-25 20:21 ` [bug#68631] [PATCH] " Sharlatan Hellseher
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 8+ messages in thread
From: Troy Figiel @ 2024-01-21 10:11 UTC (permalink / raw)
  To: 68631

* gnu/packages/sphinx.scm (python-sphinx-issues): New variable.
---
 gnu/packages/sphinx.scm | 43 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/gnu/packages/sphinx.scm b/gnu/packages/sphinx.scm
index 7e5a929241..c34d72f7ca 100644
--- a/gnu/packages/sphinx.scm
+++ b/gnu/packages/sphinx.scm
@@ -18,6 +18,7 @@
 ;;; Copyright © 2021, 2022 Vinicius Monego <monego@posteo.net>
 ;;; Copyright © 2021 Hugo Lecomte <hugo.lecomte@inria.fr>
 ;;; Copyright © 2021, 2022 Maxim Cournoyer <maxim.cournoyer@gmail.com>
+;;; Copyright © 2024 Troy Figiel <troy@troyfigiel.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -372,6 +373,48 @@ (define-public python-sphinxcontrib-newsfeed
     (home-page "https://bitbucket.org/prometheus/sphinxcontrib-newsfeed")
     (license license:bsd-2)))
 
+(define-public python-sphinx-issues
+  (package
+    (name "python-sphinx-issues")
+    (version "4.0.0")
+    (source
+     (origin
+       ;; No tests in the PyPI tarball.
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/sloria/sphinx-issues")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0q4as8gibvin0n6h5y1q4cwz3b1nwgs0idfc94dbndx42pjiz1vn"))))
+    (build-system pyproject-build-system)
+    (arguments
+     (list
+      #:phases #~(modify-phases %standard-phases
+                   (add-before 'check 'patch-sphinx-build-path
+                     (lambda _
+                       ;; The path to the sphinx-build binary is hardcoded to
+                       ;; be in the same directory as the python
+                       ;; executable. That does not work when building the
+                       ;; package.
+                       (substitute* "tests/test_sphinx_issues.py"
+                         (((string-append "Path\\(sys\\.executable\\)"
+                                          "\\.parent\\.joinpath\\"
+                                          "(\"sphinx-build\"\\)"))
+                          (string-append "\""
+                                         #$(this-package-native-input
+                                            "python-sphinx")
+                                         "/bin/sphinx-build\""))))))))
+    (native-inputs (list python-flit-core python-pytest python-sphinx))
+    (home-page "https://github.com/sloria/sphinx-issues")
+    (synopsis "Sphinx extension for linking to a project's issue tracker")
+    (description
+     "This package provides a Sphinx extension for linking to your project's
+issue tracker.  This includes roles for linking to issues, pull requests and
+user profiles.  Support for GitHub is built-in, but other services can also be
+supported with @code{sphinx-issues}.")
+    (license license:expat)))
+
 (define-public python-sphinx-panels
   (package
     (name "python-sphinx-panels")

base-commit: 9b657bab84be02935d66483da1ebbe4bc78c52fb
-- 
2.42.0





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

* [bug#68631] [PATCH v3] gnu: Add python-sphinx-issues.
  2024-01-21 10:11 [bug#68631] [PATCH] gnu: Add python-sphinx-issues Troy Figiel
@ 2024-01-21 10:11 ` Troy Figiel
  2024-01-21 10:11 ` [bug#68631] [PATCH v2] " Troy Figiel
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 8+ messages in thread
From: Troy Figiel @ 2024-01-21 10:11 UTC (permalink / raw)
  To: 68631

* gnu/packages/sphinx.scm (python-sphinx-issues): New variable.
---
 gnu/packages/sphinx.scm | 43 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 43 insertions(+)

diff --git a/gnu/packages/sphinx.scm b/gnu/packages/sphinx.scm
index 7e5a929241..e280c06f2b 100644
--- a/gnu/packages/sphinx.scm
+++ b/gnu/packages/sphinx.scm
@@ -18,6 +18,7 @@
 ;;; Copyright © 2021, 2022 Vinicius Monego <monego@posteo.net>
 ;;; Copyright © 2021 Hugo Lecomte <hugo.lecomte@inria.fr>
 ;;; Copyright © 2021, 2022 Maxim Cournoyer <maxim.cournoyer@gmail.com>
+;;; Copyright © 2024 Troy Figiel <troy@troyfigiel.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -372,6 +373,48 @@ (define-public python-sphinxcontrib-newsfeed
     (home-page "https://bitbucket.org/prometheus/sphinxcontrib-newsfeed")
     (license license:bsd-2)))
 
+(define-public python-sphinx-issues
+  (package
+    (name "python-sphinx-issues")
+    (version "4.0.0")
+    (source
+     (origin
+       ;; No tests in the PyPI tarball.
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://github.com/sloria/sphinx-issues")
+             (commit version)))
+       (file-name (git-file-name name version))
+       (sha256
+        (base32 "0q4as8gibvin0n6h5y1q4cwz3b1nwgs0idfc94dbndx42pjiz1vn"))))
+    (build-system pyproject-build-system)
+    (arguments
+     (list
+      #:phases #~(modify-phases %standard-phases
+                   (add-before 'check 'patch-sphinx-build-path
+                     (lambda _
+                       ;; The path to the sphinx-build binary is hardcoded to
+                       ;; be in the same directory as the python
+                       ;; executable. That does not work when building the
+                       ;; package.
+                       (substitute* "tests/test_sphinx_issues.py"
+                         (((string-append "Path\\(sys\\.executable\\)"
+                                          "\\.parent\\.joinpath\\"
+                                          "(\"sphinx-build\"\\)"))
+                          (string-append "\""
+                                         #$(this-package-native-input
+                                            "python-sphinx")
+                                         "/bin/sphinx-build\""))))))))
+    (native-inputs (list python-flit-core python-pytest python-sphinx))
+    (home-page "https://github.com/sloria/sphinx-issues")
+    (synopsis "Sphinx extension for linking to a project's issue tracker")
+    (description
+     "This package provides a Sphinx extension for linking to a project's
+issue tracker.  This includes roles for linking to issues, pull requests and
+user profiles.  Support for GitHub is built-in, but other services can also be
+supported with @code{sphinx-issues}.")
+    (license license:expat)))
+
 (define-public python-sphinx-panels
   (package
     (name "python-sphinx-panels")

base-commit: 9b657bab84be02935d66483da1ebbe4bc78c52fb
-- 
2.42.0





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

* [bug#68631] [PATCH] gnu: Add python-sphinx-issues.
  2024-01-21 10:11 [bug#68631] [PATCH] gnu: Add python-sphinx-issues Troy Figiel
  2024-01-21 10:11 ` [bug#68631] [PATCH v3] " Troy Figiel
  2024-01-21 10:11 ` [bug#68631] [PATCH v2] " Troy Figiel
@ 2024-01-25 20:21 ` Sharlatan Hellseher
  2024-01-25 20:32   ` Troy Figiel
  2024-01-25 22:49 ` [bug#68631] [PATCH v2] " Troy Figiel
  2024-01-26 22:16 ` bug#68631: [PATCH] " Sharlatan Hellseher
  4 siblings, 1 reply; 8+ messages in thread
From: Sharlatan Hellseher @ 2024-01-25 20:21 UTC (permalink / raw)
  To: 68631; +Cc: Troy Figiel

[-- Attachment #1: Type: text/plain, Size: 332 bytes --]

Hi,

Thank you for the patch.

It looks good.

> +    (synopsis "Sphinx extension for linking to your project's issue
tracker")

Just one neat pick to synopsis, drop "your": "Sphinx extension for linking
to project's issue tracker"

Nix has 1:1 the same synopsis as on project's GitHub page.
Debian has "your" silent.

Thanks,
Oleg

[-- Attachment #2: Type: text/html, Size: 737 bytes --]

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

* [bug#68631] [PATCH] gnu: Add python-sphinx-issues.
  2024-01-25 20:21 ` [bug#68631] [PATCH] " Sharlatan Hellseher
@ 2024-01-25 20:32   ` Troy Figiel
  0 siblings, 0 replies; 8+ messages in thread
From: Troy Figiel @ 2024-01-25 20:32 UTC (permalink / raw)
  To: 68631; +Cc: Sharlatan Hellseher


[-- Attachment #1.1.1: Type: text/plain, Size: 280 bytes --]

Hi Oleg,

I don't mind dropping "your", but grammatically we would need some
article. Something like "Sphinx extension for linking to a project's
issue tracker"? I checked the Debian synopsis and it's grammatically
incorrect, which reads a bit clunky.

Best wishes,

Troy

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 6367 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* [bug#68631] [PATCH v2] gnu: Add python-sphinx-issues.
  2024-01-21 10:11 [bug#68631] [PATCH] gnu: Add python-sphinx-issues Troy Figiel
                   ` (2 preceding siblings ...)
  2024-01-25 20:21 ` [bug#68631] [PATCH] " Sharlatan Hellseher
@ 2024-01-25 22:49 ` Troy Figiel
  2024-01-26 22:16 ` bug#68631: [PATCH] " Sharlatan Hellseher
  4 siblings, 0 replies; 8+ messages in thread
From: Troy Figiel @ 2024-01-25 22:49 UTC (permalink / raw)
  To: 68631

Hi Oleg,

- Rebased on master
- "your" -> "a" in the synopsis

Thanks for the review!

Best wishes,

Troy




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

* [bug#68631] [PATCH v2] gnu: Add python-sphinx-issues.
  2024-01-21 10:11 ` [bug#68631] [PATCH v2] " Troy Figiel
@ 2024-01-25 22:54   ` Troy Figiel
  0 siblings, 0 replies; 8+ messages in thread
From: Troy Figiel @ 2024-01-25 22:54 UTC (permalink / raw)
  To: 68631



On 2024-01-21 11:11, Troy Figiel wrote:
> +    (synopsis "Sphinx extension for linking to a project's issue tracker")
> +    (description
> +     "This package provides a Sphinx extension for linking to your project's
> +issue tracker.  This includes roles for linking to issues, pull requests and
> +user profiles.  Support for GitHub is built-in, but other services can also be
> +supported with @code{sphinx-issues}.")

Oof... description still has the "your project's issue tracker".
Smallest changes are the hardest. I will send you a v3 with that "your"
replaced by "a" as well.




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

* bug#68631: [PATCH] gnu: Add python-sphinx-issues.
  2024-01-21 10:11 [bug#68631] [PATCH] gnu: Add python-sphinx-issues Troy Figiel
                   ` (3 preceding siblings ...)
  2024-01-25 22:49 ` [bug#68631] [PATCH v2] " Troy Figiel
@ 2024-01-26 22:16 ` Sharlatan Hellseher
  4 siblings, 0 replies; 8+ messages in thread
From: Sharlatan Hellseher @ 2024-01-26 22:16 UTC (permalink / raw)
  To: 68631-done

[-- Attachment #1: Type: text/plain, Size: 77 bytes --]


Pushed as 0f646b0ec2b534c7338de5d402a928bc0b46611c to master.

Thanks,
Oleg

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 832 bytes --]

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

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

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-01-21 10:11 [bug#68631] [PATCH] gnu: Add python-sphinx-issues Troy Figiel
2024-01-21 10:11 ` [bug#68631] [PATCH v3] " Troy Figiel
2024-01-21 10:11 ` [bug#68631] [PATCH v2] " Troy Figiel
2024-01-25 22:54   ` Troy Figiel
2024-01-25 20:21 ` [bug#68631] [PATCH] " Sharlatan Hellseher
2024-01-25 20:32   ` Troy Figiel
2024-01-25 22:49 ` [bug#68631] [PATCH v2] " Troy Figiel
2024-01-26 22:16 ` bug#68631: [PATCH] " Sharlatan Hellseher

Code repositories for project(s) associated with this external index

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

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.