aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorjethro <jtg@xtrabass.com>2017-05-29 18:39:49 -0400
committerjethro <jtg@xtrabass.com>2017-06-18 10:08:37 -0400
commite1f3dfccc8a099eb08e0d9ef758b216d035afd78 (patch)
treeabf35aacb54779d49ef4616d7a2b13ed718fc709 /src
parentMerge pull request #2055 (diff)
downloadmonero-e1f3dfccc8a099eb08e0d9ef758b216d035afd78.tar.xz
Add readline support to cli
This PR adds readline support to the daemon and monero-wallet-cli. Only GNU readline is supported (e.g. not libedit) and there are cmake checks to ensure this. There is a cmake variable, Readline_ROOT_DIR that can specify a directory to find readline, otherwise some default paths are searched. There is also a cmake option, USE_READLINE, that defaults to ON. If set to ON, if readline is not found, the build continues but without readline support. One negative side effect of using readline is that the color prompt in the wallet-cli now has no color and just uses terminal default. I know how to fix this but it's quite a big change so will tackle another time.
Diffstat (limited to 'src')
-rw-r--r--src/common/command_line.cpp7
-rw-r--r--src/common/password.cpp7
2 files changed, 14 insertions, 0 deletions
diff --git a/src/common/command_line.cpp b/src/common/command_line.cpp
index f71b3e576..8c2796bbe 100644
--- a/src/common/command_line.cpp
+++ b/src/common/command_line.cpp
@@ -37,6 +37,10 @@
#include "cryptonote_config.h"
#include "string_tools.h"
+#ifdef HAVE_READLINE
+ #include "readline_buffer.h"
+#endif
+
namespace command_line
{
namespace
@@ -49,6 +53,9 @@ namespace command_line
std::string input_line(const std::string& prompt)
{
+#ifdef HAVE_READLINE
+ rdln::suspend_readline pause_readline;
+#endif
std::cout << prompt;
std::string buf;
diff --git a/src/common/password.cpp b/src/common/password.cpp
index bdc9c69c0..5c04023f4 100644
--- a/src/common/password.cpp
+++ b/src/common/password.cpp
@@ -42,6 +42,10 @@
#include <unistd.h>
#endif
+#ifdef HAVE_READLINE
+ #include "readline_buffer.h"
+#endif
+
namespace
{
#if defined(_WIN32)
@@ -238,6 +242,9 @@ namespace tools
boost::optional<password_container> password_container::prompt(const bool verify, const char *message)
{
+#ifdef HAVE_READLINE
+ rdln::suspend_readline pause_readline;
+#endif
password_container pass1{};
password_container pass2{};
if (is_cin_tty() ? read_from_tty(verify, message, pass1.m_password, pass2.m_password) : read_from_file(pass1.m_password))