Re: [PATCH] build: silence a clang warning in virsh.c

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

 



On Tue, Nov 5, 2013 at 3:58 AM, Doug Goldstein <cardoe@xxxxxxxxxx> wrote:
> On Mon, Nov 4, 2013 at 12:26 PM, Ryota Ozaki <ozaki.ryota@xxxxxxxxx> wrote:
>> On Tue, Nov 5, 2013 at 1:08 AM, Eric Blake <eblake@xxxxxxxxxx> wrote:
>>> On 11/04/2013 03:49 AM, Ryota Ozaki wrote:
>>>> This patch shuts up the following warning of clang:
>>>>
>>>>   virsh.c:2761:22: error: assigning to 'char *' from 'const char [6]' discards qualifiers
>>>>       [-Werror,-Wincompatible-pointer-types-discards-qualifiers]
>>>>       rl_readline_name = "virsh";
>>>>                        ^ ~~~~~~~
>>>
>>> Huh? <readline/readline.h> lists:
>>>
>>> /* The name of the calling program.  You should initialize this to
>>>    whatever was in argv[0].  It is used when parsing conditionals. */
>>> extern const char *rl_readline_name;
>>>
>>> at least for readline-devel-6.2 on Fedora 19.  Why is clang failing to
>>> see that the assignment is to something already marked const?  Is it a
>>> bug in your readline header?
>>
>> Grrr. It seems that this is Mac OS X specific problem :-/
>> /usr/include/readline/readline.h of Mac OS X is still:
>>   extern char             *rl_readline_name;
>>
>> The readline.h seems to be checked out from an old NetBSD
>> and not updated for a long time.
>>
>> BTW recent *BSD seem to have migrated to the above
>> const char * version.
>
> We likely should forward a note on to Apple about that.

# I'm sorry for late replying.

Yah, I will ask them about this (and rpcgen).

And also I'm sending a new patch with additional comments
about this matter.

Thanks,
  ozaki-r

>
> --
> Doug Goldstein

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]