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

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



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


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