[PATCH] crecord: stop trying to import wcurses

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[PATCH] crecord: stop trying to import wcurses

Manuel Jacob
# HG changeset patch
# User Manuel Jacob <[hidden email]>
# Date 1593281981 -7200
#      Sat Jun 27 20:19:41 2020 +0200
# Node ID 94987aadb22cee1d7da52f365b3f532a94bbd4a6
# Parent  d2227d4c9e6b97b84b5c0c24b00f6ac3330be3fa
# EXP-Topic curses
crecord: stop trying to import wcurses

The original import of crecord in 2008 already said "I have no idea if wcurses
works with crecord...".

The last reference to a Python package called wcurses is
https://web.archive.org/web/20101025073658/http://adamv.com/dev/python/curses/.
However, the Python package from there is called "curses" and not "wcurses".

I didn’t find any evidence that it ever worked.

diff --git a/mercurial/crecord.py b/mercurial/crecord.py
--- a/mercurial/crecord.py
+++ b/mercurial/crecord.py
@@ -63,15 +63,7 @@
 
     curses.error
 except (ImportError, AttributeError):
-    # I have no idea if wcurses works with crecord...
-    try:
-        import wcurses as curses
-
-        curses.error
-    except (ImportError, AttributeError):
-        # wcurses is not shipped on Windows by default, or python is not
-        # compiled with curses
-        curses = False
+    curses = False
 
 
 class fallbackerror(error.Abort):
_______________________________________________
Mercurial-devel mailing list
[hidden email]
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel
Reply | Threaded
Open this post in threaded view
|

Re: [PATCH] crecord: stop trying to import wcurses

Yuya Nishihara
On Sat, 27 Jun 2020 21:54:56 +0200, Manuel Jacob wrote:
> # HG changeset patch
> # User Manuel Jacob <[hidden email]>
> # Date 1593281981 -7200
> #      Sat Jun 27 20:19:41 2020 +0200
> # Node ID 94987aadb22cee1d7da52f365b3f532a94bbd4a6
> # Parent  d2227d4c9e6b97b84b5c0c24b00f6ac3330be3fa
> # EXP-Topic curses
> crecord: stop trying to import wcurses

Queued, thanks.
_______________________________________________
Mercurial-devel mailing list
[hidden email]
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel