all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc
@ 2022-09-21  2:21 Fredrik Salomonsson
  2022-09-21  2:23 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: Change source Fredrik Salomonsson
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Fredrik Salomonsson @ 2022-09-21  2:21 UTC (permalink / raw)
  To: 57964; +Cc: Fredrik Salomonsson

It seems that ELPA regenerated the archive for the source of emacs-jsonrpc, as
discoverd by Josselin Poiret in issue 57899[0]. Which breaks the package in
guix as now the hash mismatch.

This patch changes the source to using Emacs' Git repository. Which will
hopefully avoid this issue in the future.

**NOTE** I used https://github.com/emacs-mirror/emacs when testing this, as
https://git.savannah.gnu.org/git/emacs.git/ was dead slow for me (~16Mb/s vs
~50kb/s). But I figured it might just be a hickup on savannah and it is better
to use the original source than a mirror.

[0] https://issues.guix.gnu.org/57899

Fredrik Salomonsson (1):
  gnu: emacs-jsonrpc: Change source.

 gnu/packages/emacs-xyz.scm | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)


base-commit: f256614891335e250207edab129bdf8601c22774
-- 
2.37.3





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

* [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: Change source.
  2022-09-21  2:21 [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Fredrik Salomonsson
@ 2022-09-21  2:23 ` Fredrik Salomonsson
  2022-09-21  6:35 ` [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Josselin Poiret via Guix-patches via
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Fredrik Salomonsson @ 2022-09-21  2:23 UTC (permalink / raw)
  To: 57964; +Cc: Fredrik Salomonsson

* gnu/packages/emacs-xyz.scm (emacs-jsonrpc)
[source]: Change to Emacs' Git repository.
---
 gnu/packages/emacs-xyz.scm | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index 32e4c33f16..889820350e 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -16345,12 +16345,23 @@ (define-public emacs-jsonrpc
     (version "1.0.15")
     (source
      (origin
-       (method url-fetch)
-       (uri (string-append "https://elpa.gnu.org/packages/"
-                           "jsonrpc-" version ".tar"))
+       (method git-fetch)
+       (uri (git-reference
+             (url "https://git.savannah.gnu.org/git/emacs.git/")
+             (commit "1231a601ebe1fd9fe454c504dbeb9267440242e7")))
+       (file-name (git-file-name name version))
        (sha256
-        (base32 "1hx378rg12jz2zm105cvrqk0nqyzsn04l59d903l98d6lbd96rsw"))))
+        (base32
+         "00i5n150ibkdn2i3p8q3d1ag5jnikfc988f3hz2yfdk4z0ajj536"))))
     (build-system emacs-build-system)
+    (arguments
+     (list #:phases
+           #~(modify-phases %standard-phases
+               (replace 'unpack
+                 (lambda* (#:key source #:allow-other-keys)
+                   (mkdir "source")
+                   (chdir "source")
+                   (copy-file (string-append source "/lisp/jsonrpc.el") "jsonrpc.el"))))))
     (home-page "http://elpa.gnu.org/packages/jsonrpc.html")
     (synopsis "JSON-RPC library")
     (description
-- 
2.37.3





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

* [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc
  2022-09-21  2:21 [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Fredrik Salomonsson
  2022-09-21  2:23 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: Change source Fredrik Salomonsson
@ 2022-09-21  6:35 ` Josselin Poiret via Guix-patches via
  2022-09-23 23:42 ` [bug#57964] [PATCH 0/1] Correct hash Fredrik Salomonsson
  2022-09-23 23:42 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: " Fredrik Salomonsson
  3 siblings, 0 replies; 6+ messages in thread
From: Josselin Poiret via Guix-patches via @ 2022-09-21  6:35 UTC (permalink / raw)
  To: Fredrik Salomonsson, 57964; +Cc: Fredrik Salomonsson

Hi Fredrik,

Fredrik Salomonsson <plattfot@posteo.net> writes:

> It seems that ELPA regenerated the archive for the source of emacs-jsonrpc, as
> discoverd by Josselin Poiret in issue 57899[0]. Which breaks the package in
> guix as now the hash mismatch.
>
> This patch changes the source to using Emacs' Git repository. Which will
> hopefully avoid this issue in the future.

Builds and runs for me, and patch is clean!
Reviewed-by: Josselin Poiret <dev@jpoiret.xyz>

Best,
-- 
Josselin Poiret




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

* [bug#57964] [PATCH 0/1] Correct hash
  2022-09-21  2:21 [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Fredrik Salomonsson
  2022-09-21  2:23 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: Change source Fredrik Salomonsson
  2022-09-21  6:35 ` [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Josselin Poiret via Guix-patches via
@ 2022-09-23 23:42 ` Fredrik Salomonsson
  2022-09-23 23:42 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: " Fredrik Salomonsson
  3 siblings, 0 replies; 6+ messages in thread
From: Fredrik Salomonsson @ 2022-09-23 23:42 UTC (permalink / raw)
  To: 57964; +Cc: Fredrik Salomonsson

Hi,

I saw that my patch was merged. But when I did a guix pull and built it
I got a hash mismatch again. Looks like when it got cleaned up to use
the commit which bumped jsonrpc.el to 1.0.15 the hash wasn't updated.

I added a patch to use the correct hash.

Fredrik Salomonsson (1):
  gnu: emacs-jsonrpc: Correct hash.

 gnu/packages/emacs-xyz.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


base-commit: 08d515233241ee0921b8b5ab706f98170c62437c
-- 
2.37.3





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

* [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: Correct hash.
  2022-09-21  2:21 [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Fredrik Salomonsson
                   ` (2 preceding siblings ...)
  2022-09-23 23:42 ` [bug#57964] [PATCH 0/1] Correct hash Fredrik Salomonsson
@ 2022-09-23 23:42 ` Fredrik Salomonsson
  2022-09-24  5:20   ` bug#57964: " Liliana Marie Prikler
  3 siblings, 1 reply; 6+ messages in thread
From: Fredrik Salomonsson @ 2022-09-23 23:42 UTC (permalink / raw)
  To: 57964; +Cc: Fredrik Salomonsson

* gnu/packages/emacs-xyz.scm (emacs-jsonrpc)[source]: Correct hash.
---
 gnu/packages/emacs-xyz.scm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index b98b435e9b..c413bc971c 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -16357,7 +16357,7 @@ (define-public emacs-jsonrpc
          (file-name (git-file-name name version))
          (sha256
           (base32
-           "0005n150ibkdn2i3p8q3d1ag5jnikfc988f3hz2yfdk4z0ajj536"))))
+           "0srwb171dxha1nfzppk6x9v4bdj3lk74pksqnhalw8jw9c67b72y"))))
       (build-system emacs-build-system)
       (arguments
        (list #:phases
-- 
2.37.3





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

* bug#57964: [PATCH 1/1] gnu: emacs-jsonrpc: Correct hash.
  2022-09-23 23:42 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: " Fredrik Salomonsson
@ 2022-09-24  5:20   ` Liliana Marie Prikler
  0 siblings, 0 replies; 6+ messages in thread
From: Liliana Marie Prikler @ 2022-09-24  5:20 UTC (permalink / raw)
  To: Fredrik Salomonsson, 57964-done

Am Freitag, dem 23.09.2022 um 23:42 +0000 schrieb Fredrik Salomonsson:
> * gnu/packages/emacs-xyz.scm (emacs-jsonrpc)[source]: Correct hash.
Sorry for not double-checking.  Pushed!




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

end of thread, other threads:[~2022-09-24  5:21 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-21  2:21 [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Fredrik Salomonsson
2022-09-21  2:23 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: Change source Fredrik Salomonsson
2022-09-21  6:35 ` [bug#57964] [PATCH 0/1] Change source for emacs-jsonrpc Josselin Poiret via Guix-patches via
2022-09-23 23:42 ` [bug#57964] [PATCH 0/1] Correct hash Fredrik Salomonsson
2022-09-23 23:42 ` [bug#57964] [PATCH 1/1] gnu: emacs-jsonrpc: " Fredrik Salomonsson
2022-09-24  5:20   ` bug#57964: " Liliana Marie Prikler

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.