Re: [PATCH] tests: Fix the build failure on s390

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

 



On 02/10/2014 06:35 AM, Osier Yang wrote:
> The build works fine on other architectures with commit 0b4f76fc5, but
> for s390:
> 
> TEST: virscsitest
>  1) test1  ... OK
>  2) test2  ... libvirt:  error : SCSI device '1:0:0:0': could not access
>  /builddir/build/BUILD/libvirt-1.1.1/tests/virscsidata/sg8: No such file
>  or directory
> FAILED
> 
> It's caused by the "patch" on the s390 system either doesn't create
> the "empty files", or removed them after the patch was applied. Anyway,
> this patch is to fix it by simply adding useless numbers to the 2
> test input files.
> ---
>  tests/virscsidata/sg0 | 1 +
>  tests/virscsidata/sg8 | 1 +
>  2 files changed, 2 insertions(+)

Why are we modifying upstream?  This sounds like a downstream issue with
patch application, so downstream should come up with alternative ways to
create empty files into existence when applying patches, without
modifying the content of the empty file upstream.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

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