 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
|
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [Xen-devel] [PATCH 1 of 7] tools/check: remove check_logging [and 1 more messages], Ian Jackson |
|---|---|
| Next by Date: | [Xen-devel] [PATCH 0 of 3] libxl: fixup hard tabs and indentation, Ian Campbell |
| Previous by Thread: | [Xen-devel] [PATCH 5 of 7] tools: call cross-install with shell, Olaf Hering |
| Next by Thread: | Re: [Xen-devel] [PATCH] tools/check: check for yajl (needed by libxl), Andre Przywara |
| Indexes: | [Date] [Thread] [Top] [All Lists] |
