[libvirt] [PATCH] util: compilation fix if build without GNUTLS library
Cole Robinson
crobinso at redhat.com
Sun May 22 14:44:24 UTC 2016
On 05/22/2016 10:23 AM, John Ferlan wrote:
>
>
> On 05/21/2016 10:22 AM, Mikhail Feoktistov wrote:
>> ---
>> src/util/vircrypto.c | 13 +++++++------
>> 1 file changed, 7 insertions(+), 6 deletions(-)
>>
>
> Ugh... Sorry about this - forgot about the unused attributes <sigh>
>
> I should have just gone with the entire function w/ the
> HAVE_GNUTLS_CIPHER_ENCRYPT macro instead of the intermixed
> ATTRIBUTE_UNUSED as those would have a tendency for someone to come
> along and try to patch again pointing out they are used without
> realizing it was the gnutls macro that needed to be broader...
>
> I'll send something short with 2 patches that will do that. I started
> working on the next task and realized I needed a slight adjustment anyway...
FWIW there's an old BiteSizedTask about this type of issue wrt to GNUTLS:
http://wiki.libvirt.org/page/BiteSizedTasks#Abstract_away_most_usage_of_WITH_GNUTLS
So I guess this has come up before
- Cole
More information about the libvir-list
mailing list