[Libvir] [PATCH] fix compilation if readline lib is not available

[Jim Meyering]

Jim Meyering <jim at meyering.net> wrote:

> Daniel Veillard <veillard at redhat.com> 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.