[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [PATCH v2 2/2] vsh: Define HAVE_STDARG_H before including readline

On a Thursday in 2020, Michal Privoznik wrote:
As it turned out my previous commits which switched from HAVE_ to
WITH_ and dropped stdarg.h detection were a bit too aggressive.
Because of reasons described in 9ea3424a178 we need to define
HAVE_STDARG_H before including readline otherwise macos build
fails. Honestly, I still don't fully understand the problem so I
am not going to bother you with "explanation".

Signed-off-by: Michal Privoznik <mprivozn redhat com>
tools/vsh.c | 4 ++++
1 file changed, 4 insertions(+)

diff --git a/tools/vsh.c b/tools/vsh.c
index 2511089e6e..ef2a3f62d7 100644
--- a/tools/vsh.c
+++ b/tools/vsh.c
@@ -30,6 +30,10 @@
#include <signal.h>

+/* In order to have proper rl_message declaration with older
+ * versions of readline, we have to declare this. See 9ea3424a178

I don't think it's because of the 'older version', I'm surprised it
works for me on Linux without it.

+ * for more info. */
+# define HAVE_STDARG_H
# include <readline/readline.h>
# include <readline/history.h>

Reviewed-by: Ján Tomko <jtomko redhat com>

(just don't expect it to fix the pipeline on master, some other issues
have creeped in in the meantime)


Attachment: signature.asc
Description: PGP signature

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]