Re: [PATCH 3/3] src: Drop most of #ifdef WITH_GNUTLS

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

 



On Tue, Jun 05, 2018 at 10:45:57AM +0200, Michal Privoznik wrote:
> Now that GnuTLS is a requirement, we can drop a lot of
> conditionally built code. However, not all ifdef-s can go because
> we still want libvirt_setuid to build without gnutls.
> 
> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
> ---
>  src/locking/lock_daemon.c           |  4 ---
>  src/logging/log_daemon.c            |  4 ---
>  src/lxc/lxc_controller.c            |  2 --
>  src/qemu/qemu_migration_cookie.c    | 12 +++-----
>  src/remote/remote_daemon.c          | 23 ---------------
>  src/remote/remote_daemon_dispatch.c |  2 --
>  src/rpc/virnetdaemon.h              |  4 +--
>  src/rpc/virnetserver.c              |  6 ----
>  src/rpc/virnetserver.h              |  6 +---
>  src/rpc/virnetserverclient.c        | 57 +++----------------------------------
>  src/rpc/virnetserverclient.h        |  8 ------
>  src/rpc/virnetserverservice.c       | 24 ----------------
>  src/rpc/virnetserverservice.h       | 10 -------
>  src/util/vircrypto.c                | 43 ++--------------------------
>  tests/qemuxml2argvtest.c            | 15 ----------
>  tests/vircryptotest.c               | 24 +++++-----------
>  tests/virfilecachetest.c            | 18 +++---------
>  tests/virnetdaemontest.c            |  8 ------
>  tests/virnetserverclienttest.c      |  2 --
>  tests/virrandommock.c               |  8 ++----
>  20 files changed, 27 insertions(+), 253 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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