GNU bug report logs - #48059
[PATCH] gnu: python-libtmux: Fix build by removing tests broken by tmux update

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 48059 in the body.
You can then email your comments to 48059 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Edouard Klein <edk <at> beaver-labs.com>
To: guix-patches <at> gnu.org
Cc: Edouard Klein <edouard <at> hex.lan>
Subject: [PATCH] gnu: python-libtmux: Fix build by removing tests broken by
 tmux update
Date: Tue, 27 Apr 2021 15:20:00 +0200

From: Edouard Klein <edouard <at> hex.lan>

tmux update from 3.1c to 3.2 breaks two tests in this package by quoting the
returned value of a tmux call. No harm done, a fix upstream is unlikely.
---
 gnu/packages/tmux.scm | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/gnu/packages/tmux.scm b/gnu/packages/tmux.scm
index f208fbfd93..9cd632f535 100644
--- a/gnu/packages/tmux.scm
+++ b/gnu/packages/tmux.scm
@@ -178,7 +178,9 @@ windows.")
              ;; https://github.com/tmux-python/libtmux/issues/281
              (invoke "pytest" "-vv" "-k"
                      (string-append "not test_show_option_unknown "
-                                    "and not test_show_window_option_unknown"))
+                                    "and not test_show_window_option_unknown "
+                                    "and not test_split_window_shell "
+                                    "and not test_new_session_shell"))
              #t)))))
     (home-page "https://github.com/tmux-python/libtmux")
     (synopsis "Python API for tmux")
-- 
2.31.1

Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: Edouard Klein <edk <at> beaver-labs.com>
Cc: 48059 <at> debbugs.gnu.org, guix-patches <at> gnu.org,
 Edouard Klein <edouard <at> hex.lan>
Subject: Re: [bug#48059] [PATCH] gnu: python-libtmux: Fix build by removing
 tests broken by tmux update
Date: Tue, 27 Apr 2021 15:39:12 +0200

[Message part 1 (text/plain, inline)]

Edouard,

Edouard Klein 写道：
> tmux update from 3.1c to 3.2 breaks two tests in this package by 
> quoting the
> returned value of a tmux call. No harm done, a fix upstream is 
> unlikely.

Hm... why's that?  Does upstream not support their own test suite?

Thanks!

T G-R

[signature.asc (application/pgp-signature, inline)]

Message #14 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Tobias Geerinckx-Rice <me <at> tobias.gr>
To: Edouard Klein <edk <at> beaver-labs.com>
Cc: 48059 <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#48059] [PATCH] gnu: python-libtmux: Fix build by removing
 tests broken by tmux update
Date: Tue, 27 Apr 2021 15:43:46 +0200

[Message part 1 (text/plain, inline)]

Edouard,

Edouard Klein 写道：
> From: Edouard Klein <edouard <at> hex.lan>

Oh, you should change this to a routable mail address ;-)

 $ git config user.email edk <at> beaver-labs.com

Kind regards,

T G-R

[signature.asc (application/pgp-signature, inline)]

Message #20 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Edouard Klein <edou <at> rdklein.fr>
To: Tobias Geerinckx-Rice <me <at> tobias.gr>
Cc: 48059 <at> debbugs.gnu.org, guix-patches <at> gnu.org,
 Edouard Klein <edouard <at> hex.lan>
Subject: Re: [bug#48059] [PATCH] gnu: python-libtmux: Fix build by removing
 tests broken by tmux update
Date: Tue, 27 Apr 2021 18:24:24 +0200

Tobias Geerinckx-Rice writes:

> Edouard,
>
> Edouard Klein 写道：
>> tmux update from 3.1c to 3.2 breaks two tests in this package by quoting the
>> returned value of a tmux call. No harm done, a fix upstream is unlikely.
>
> Hm... why's that?  Does upstream not support their own test suite?
>

The issue I opened last year about the two other tests failing is still
unresolved
https://github.com/tmux-python/libtmux/issues/281

There seems to be little ongoing development:
https://github.com/tmux-python/libtmux/commits/master

My suspicion is that the maintainer is using debian or another slow
distro that still use tmux 2, which I believe make the test pass (at
leat it did when I tested it last year)
https://packages.debian.org/buster/tmux

There is no way in Python to specify which version of tmux this is
supposed to run against. This is one of the strength of GNU Guix :)

So:
- the change in behaviour between tmux 2 and 3 is not
important enough to be noticeable in any of my use cases,
- I have little time,
- the maintainer upstream is likely using tmux 2, 


all-in-all, this makes me say that an upstream fix is unlikely because I
can't write it myself and nobody else is likely to, either.

> Thanks!
>
Thank you for maintaining Guix :)

> T G-R

Message #26 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Edouard Klein <edou <at> rdklein.fr>
To: Tobias Geerinckx-Rice <me <at> tobias.gr>
Cc: 48059 <at> debbugs.gnu.org, guix-patches <at> gnu.org
Subject: Re: [bug#48059] [PATCH] gnu: python-libtmux: Fix build by removing
 tests broken by tmux update
Date: Tue, 27 Apr 2021 18:26:05 +0200

Tobias Geerinckx-Rice writes:

> Edouard,
>
> Edouard Klein 写道：
>> From: Edouard Klein <edouard <at> hex.lan>
>
> Oh, you should change this to a routable mail address ;-)
>
>  $ git config user.email edk <at> beaver-labs.com
>
$ git config --get user.email
edk <at> beaver-labs.com

I don't know why git send-email is using my hostname. I'll have to
investigate. Thanks for pointing it out :)

> Kind regards,
>
> T G-R

Message #32 received at 48059 <at> debbugs.gnu.org (full text, mbox):

From: edk <at> beaver-labs.com
To: 48059 <at> debbugs.gnu.org
Subject: Re: bug#48059: Acknowledgement ([PATCH] gnu: python-libtmux: Fix
 build by removing tests broken by tmux update)
Date: Fri, 28 May 2021 09:38:50 +0200

Sorry to be a bother, but could we please merge this ? We depend on this
package and we have to use guix time machine because it is broken. We
would very much like to enjoy the new 1.3 features.

Thanks in advance,

Cheers,

Edouard.
GNU bug Tracking System writes:

> Thank you for filing a new bug report with debbugs.gnu.org.
>
> This is an automatically generated reply to let you know your message
> has been received.
>
> Your message is being forwarded to the package maintainers and other
> interested parties for their attention; they will reply in due course.
>
> Your message has been sent to the package maintainer(s):
>  guix-patches <at> gnu.org
>
> If you wish to submit further information on this problem, please
> send it to 48059 <at> debbugs.gnu.org.
>
> Please do not send mail to help-debbugs <at> gnu.org unless you wish
> to report a problem with the Bug-tracking system.

Message #35 received at 48059 <at> debbugs.gnu.org (full text, mbox):

From: Maxime Devos <maximedevos <at> telenet.be>
To: edk <at> beaver-labs.com, 48059 <at> debbugs.gnu.org
Subject: Re: [bug#48059] Acknowledgement ([PATCH] gnu: python-libtmux: Fix
 build by removing tests broken by tmux update)
Date: Fri, 28 May 2021 09:56:44 +0200

[Message part 1 (text/plain, inline)]

edk <at> beaver-labs.com schreef op vr 28-05-2021 om 09:38 [+0200]:
> Sorry to be a bother, but could we please merge this ? We depend on this
> package and we have to use guix time machine because it is broken. We
> would very much like to enjoy the new 1.3 features.

I guess it is good to merge, but someone else would have to actually commit
it! (I am not committer)

Sorry for the delay. Bugs and patches can easily get forgotten after a while.

Greetings,
Maxime

[signature.asc (application/pgp-signature, inline)]

Message #38 received at 48059 <at> debbugs.gnu.org (full text, mbox):

From: Edouard Klein <edou <at> rdklein.fr>
To: Maxime Devos <maximedevos <at> telenet.be>
Cc: 48059 <at> debbugs.gnu.org
Subject: Re: [bug#48059] Acknowledgement ([PATCH] gnu: python-libtmux: Fix
 build by removing tests broken by tmux update)
Date: Fri, 28 May 2021 18:17:41 +0200

Thanks, no worries, I understand :)
Maxime Devos writes:

> edk <at> beaver-labs.com schreef op vr 28-05-2021 om 09:38 [+0200]:
>> Sorry to be a bother, but could we please merge this ? We depend on this
>> package and we have to use guix time machine because it is broken. We
>> would very much like to enjoy the new 1.3 features.
>
> I guess it is good to merge, but someone else would have to actually commit
> it! (I am not committer)
>
> Sorry for the delay. Bugs and patches can easily get forgotten after a while.
>
> Greetings,
> Maxime

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.