On Fri, May 26, 2017 at 07:11:53AM -0400, John Ferlan wrote: > > > On 05/26/2017 03:15 AM, Bjoern Walk wrote: > > Thanks a lot for your review John. > > > > If you don't mind, you can make the adjustments you mentioned and push, > > as I can only prepare a v2 on Monday and I wanted to get this in v3.4.0. > > > > Best, > > Bjoern > > > > I've made the adjustments and will push this sometime today... FYI this broke the build on OS-X util/virfcp.c:85:5: error: implicit declaration of function 'virReportSystemError' is invalid in C99 [-Werror,-Wimplicit-function-declaration] virReportSystemError(ENOSYS, "%s", _("Not supported on this platform")); util/virfcp.c:83:1: error: no previous prototype for function 'virSysfsIsCapableFCRport' [-Werror,-Wmissing-prototypes] virSysfsIsCapableFCRport(const char *rport ATTRIBUTE_UNUSED) util/virfcp.c:90:1: error: no previous prototype for function 'virSysfsReadFCRport' [-Werror,-Wmissing-prototypes] virSysfsReadFCRport(const char *rport ATTRIBUTE_UNUSED, https://travis-ci.org/libvirt/libvirt/jobs/236407391 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