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

Re: [Xen-devel] [osstest PATCH] target_cmd_build: honour DistccHosts host property

Ian Jackson writes ("[osstest PATCH] target_cmd_build: honour DistccHosts host 
> Unfortunately there is another problem which is that ts-xen-build has
> been made to require root access (which osstest doesn't have on army).

Here's a fix to this.

I have not pushed this to pretest yet because I want to be sure we get
a pass to fix the freebsd hostflags problem and I'm not 100.000% sure
that this change will pass.

For now it's in xenbits.xen.org:/home/iwj/ext/osstest.git#wip.distcc


From: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Date: Thu, 28 Nov 2013 18:20:04 +0000
Subject: [PATCH] target_file_exists: Use non-root access for this check

The new call of this in built_stash_file makes ts-*-build require root
access, which is not desirable.

The only other caller of target_file_exists is in ts-xen-install where
it is called on
all of which should be accessible as a normal user in a default config.

So change the access to use target_cmd_output rather than

Signed-off-by: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
 Osstest/TestSupport.pm |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Osstest/TestSupport.pm b/Osstest/TestSupport.pm
index 4954d84..3bad5ca 100644
--- a/Osstest/TestSupport.pm
+++ b/Osstest/TestSupport.pm
@@ -457,7 +457,7 @@ sub target_putfilecontents_root_stash ($$$$;$) {
 sub target_file_exists ($$) {
     my ($ho,$rfile) = @_;
-    my $out= target_cmd_output_root($ho, "if test -e $rfile; then echo y; fi");
+    my $out= target_cmd_output($ho, "if test -e $rfile; then echo y; fi");
     return 1 if $out =~ m/^y$/;
     return 0 if $out !~ m/\S/;
     die "$rfile $out ?";

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.