Jim Meyering <jim@xxxxxxxxxxxx> wrote: > Daniel Veillard <veillard@xxxxxxxxxx> wrote: >> The current CVS code doesn't compile on RHEL5/F6 because readline lib >> there require ncurses, and we removed that check earlier. Sor we find >> the readline headers but the lib is not found. Unfortunately virsh.c >> tests the availablility of the header to use readline, and link time failures >> follow. The patch below: >> - exports a READLINE_CFLAGS from configure if used >> - use the READLINE_CFLAGS when compiling virsh.c >> - change virsh.c to rely on this instead of the header test >> - adds a message about readline usage at the end of configure >> >> configure: Miscellaneous >> configure: >> configure: Debug: no >> configure: Readline: yes >> configure: >> >> Another patch available soon should fix the readline library detection itself, > ... >> +USE_READLINE= >> +READLINE_CFLAGS= > > That looks fine, but would you please do > s/READLINE_CFLAGS/VIRSH_CPPFLAGS/? Oops. Seeing the rest of your patch, you're probably right. Even using _CFLAGS is ok, for consistency with all of the other libvirt-specific _CFLAGS variable names. so +1 for your unmodified patch. -- Libvir-list mailing list Libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list