[Libvirt-ci] Fixed: libvirt/libvirt#1866 (master - e6f53e7)

Travis CI builds at travis-ci.org
Thu Dec 13 21:52:35 UTC 2018


Build Update for libvirt/libvirt
-------------------------------------

Build: #1866
Status: Fixed

Duration: 21 mins and 17 secs
Commit: e6f53e7 (master)
Author: John Ferlan
Message: storage: Fix build issue with MOUNT and VGCHANGE commands

Turns out there some build platforms that must not define MOUNT
or VGCHANGE in config.h... So moving the commands from the storage
backend specific module into a common storage_util module causes
issues for those platforms.

So instead of assuming they are there, let's just pass the command
string to the storage util API's from the storage backend specific
code (as would have been successful before).  Also modify the test
to determine whether the MOUNT and/or VGCHANGE doesn't exist and
just define it to (for example) what Fedora has for the path. Could
have just used "mount" and "vgchange" in the call, but that defeats
the purpose of adding the call to virTestClearCommandPath.

Signed-off-by: John Ferlan <jferlan at redhat.com>

View the changeset: https://github.com/libvirt/libvirt/compare/fe2bd0210e80...e6f53e7a4b43

View the full build log and details: https://travis-ci.org/libvirt/libvirt/builds/467723003?utm_medium=notification&utm_source=email

--

You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-ci/attachments/20181213/a87ed330/attachment.htm>


More information about the Libvirt-ci mailing list