Re: [PATCH v3 44/48] util: change identity class attribute names

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

 



On Tue, 2019-07-30 at 16:23 +0200, Christophe de Dinechin wrote:
> Daniel P. Berrangé writes:
> >              virTypedParamsAddString(&tmpparams, nparams, &maxparams,
> >                                      VIR_CLIENT_INFO_UNIX_USER_NAME,
> 
> There are some spots where "UNIX" remains...

That's because VIR_CLIENT_INFO_UNIX_* are pre-existing defines which,
since they're part of the public API, can't be changed without
breaking existing applications.

It is true, however, that we're basically introducing a whole new set
of defines where VIR_CLIENT_INFO_* already cover exactly the values
we want to pass around... Is there any way we can reuse those instead
of coming up with our own ever so slightly different set of defines?

-- 
Andrea Bolognani / Red Hat / Virtualization

--
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]

  Powered by Linux