WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH 0 of 7] tools/check fixes for xen-unstable

To: Olaf Hering <olaf@xxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH 0 of 7] tools/check fixes for xen-unstable
From: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
Date: Fri, 7 Oct 2011 14:48:20 +0100
Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date: Fri, 07 Oct 2011 06:49:04 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <patchbomb.1317738109@xxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Newsgroups: chiark.mail.xen.devel
References: <patchbomb.1317738109@xxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Olaf Hering writes ("[Xen-devel] [PATCH 0 of 7] tools/check fixes for 
xen-unstable"):
> 
> Remove the need for executable permissions of some files in the SCM.

Thanks for this contribution but I'm afraid I don't really agree with
the need for this change.  It is simpler in our build system (which is
quite complicated enough already) just to execute these kind of
scripts directly.

If the arrangements you use for producing a source tree don't always
put the +x permissions back correctly, then I think that's something
that shouldn't be too hard to fix.  There should be no problem with
setting +x on files that don't need it (and if there is, we can
probably fix that) so you don't have to be 100% discriminating in that
fixup.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel