Peter Krempa wrote:

> On 05/02/14 11:37, Daniel P. Berrange wrote:
> > On Thu, May 01, 2014 at 10:51:02PM +0400, Roman Bogorodskiy wrote:
> >> Currently firewalling is supported on Linux only, so skip the
> >> virfirewalltest on other platforms.
> >> ---
> >>  tests/virfirewalltest.c | 58 
> >> +++++++++++++++++++++++++++++--------------------
> >>  1 file changed, 35 insertions(+), 23 deletions(-)
> > 
> > This code is actually intended to be portable across all platforms.
> > 
> > What actual problem are you seeing.
> 
> There were build failures with the new test, but according to our build
> systems they should be fixed now by a few patches from Pavel and Jan so
> there shouldn't be a need to skip the test now.
> 
> Roman, could you please re-try running it with the current HEAD? (last
> patch went in just recently)

The test still fails on current HEAD.

Roman Bogorodskiy

Attachment: pgpqtO8Jj7GVl.pgp
Description: PGP signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to