diff mbox series

[bug#55466,v2,2/2] gnu: emacs-explain-pause-mode: Fix failing tests.

Message ID 20220517014845.10441-2-luishenriquegh2701@gmail.com
State Accepted
Headers show
Series [bug#55466,v2,1/2] gnu: emacs-racer: Update to 1e63e98. | expand

Checks

Context Check Description
cbaines/comparison success View comparision
cbaines/git branch success View Git branch
cbaines/applying patch success View Laminar job
cbaines/issue success View issue

Commit Message

Luis Henrique Gomes Higino May 17, 2022, 1:48 a.m. UTC
From: luishgh <51053603+luishgh@users.noreply.github.com>

* gnu/packages/emacs-xyz.scm (emacs-explain-pause-mode): Fix failing tests.
---
 gnu/packages/emacs-xyz.scm | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

Comments

Nicolas Goaziou May 17, 2022, 7:17 a.m. UTC | #1
Hello,

Luis Henrique Gomes Higino <luishenriquegh2701@gmail.com> writes:

>      (package
>        (name "emacs-explain-pause-mode")
> -      (version (git-version "0.1" revision commit))
> +      (version (git-version "0.2" revision commit))

I removed this ninja version bump. The main file still references "0.1".

Regards,
Luis Henrique Gomes Higino May 17, 2022, 2:38 p.m. UTC | #2
Hi Nicolas,

thanks for your feedback and for enhancing my patches before 
applying them! I just have one question:

Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

> Hello,
>
> Luis Henrique Gomes Higino <luishenriquegh2701@gmail.com> 
> writes:
>
>>      (package
>>        (name "emacs-explain-pause-mode")
>> -      (version (git-version "0.1" revision commit))
>> +      (version (git-version "0.2" revision commit))
>
> I removed this ninja version bump. The main file still 
> references "0.1".
>
> Regards,

That's funny, because in the README [0] it says the development 
version is 0.2, that's why I bumped it. Which should we use?

0: 
https://github.com/lastquestion/explain-pause-mode/blob/master/README.md#explain-pause-mode

Regards,
Nicolas Goaziou May 17, 2022, 8:02 p.m. UTC | #3
Luis Henrique Gomes Higino <luishenriquegh2701@gmail.com> writes:

> That's funny, because in the README [0] it says the development
> version is 0.2, that's why I bumped it. Which should we use?

We usually refer to the Version keyword in main file when there is no
tag. Note that (git-version "0.2" revision commit) is actually a greater
version than "0.2", so it could be confusing when "0.2" is eventually
released.
Luis Henrique Gomes Higino May 17, 2022, 8:24 p.m. UTC | #4
Nicolas Goaziou <mail@nicolasgoaziou.fr> writes:

> Luis Henrique Gomes Higino <luishenriquegh2701@gmail.com> 
> writes:
>
>> That's funny, because in the README [0] it says the development
>> version is 0.2, that's why I bumped it. Which should we use?
>
> We usually refer to the Version keyword in main file when there 
> is no
> tag. Note that (git-version "0.2" revision commit) is actually a 
> greater
> version than "0.2", so it could be confusing when "0.2" is 
> eventually
> released.

Makes sense, thanks again Nicolas!

Regards,
diff mbox series

Patch

diff --git a/gnu/packages/emacs-xyz.scm b/gnu/packages/emacs-xyz.scm
index c8a19e5e9e..6dfb1ea3a1 100644
--- a/gnu/packages/emacs-xyz.scm
+++ b/gnu/packages/emacs-xyz.scm
@@ -4709,7 +4709,7 @@  (define-public emacs-explain-pause-mode
         (revision "0"))
     (package
       (name "emacs-explain-pause-mode")
-      (version (git-version "0.1" revision commit))
+      (version (git-version "0.2" revision commit))
       (source
        (origin
          (method git-fetch)
@@ -4724,6 +4724,13 @@  (define-public emacs-explain-pause-mode
        (list emacs-buttercup))
       (arguments
        '(#:tests? #t
+         #:phases
+         (modify-phases %standard-phases
+           ;; This causes the byte-compilation before unit-tests to fail
+           (add-after 'unpack 'remove-error-on-warn
+             (lambda _
+               (substitute* "Makefile"
+                 (("--eval '\\(setq byte-compile-error-on-warn t\\)'") "")))))
          ;; Don't run case-tests as they will fail to create sockets because
          ;; the path is too long
          #:test-command '("make" "byte-compile" "unit-tests")