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

[Xen-devel] Re: [PATCH] libxenstore should be executable

To: Jeremy Katz <katzj@xxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH] libxenstore should be executable
From: Horms <horms@xxxxxxxxxxxx>
Date: Tue, 11 Jul 2006 21:30:17 +0900 (JST)
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>, Vincent Hanquez <vincent.hanquez@xxxxxxxxxxxx>
Delivery-date: Wed, 12 Jul 2006 01:35:56 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
In-reply-to: <1152124589.20228.96.camel@xxxxxxxxxxxxxx>
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/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: tin/1.8.2-20060425 ("Shillay") (UNIX) (Linux/2.6.17-1-686 (i686))
On Wed, 05 Jul 2006 14:36:29 -0400, Jeremy Katz wrote:
> [-- text/plain, encoding 7bit, charset: ISO-8859-1, 20 lines --]
> 
> On Tue, 2006-07-04 at 11:57 +0200, Vincent Hanquez wrote:
>> On Mon, Jul 03, 2006 at 11:28:03PM -0400, Jeremy Katz wrote:
>> > Shared libraries are supposed to be installed +x
>>
>> I don't think that needed (almost all libraries in my /usr/lib are -x).
>> Even then you shouldn't call INSTALL_**PROG** but just create an
>> INSTALL_LIB or somethign similar..
> 
> Shared libs have traditionally been installed as executables due to some
> long ago accidents of implementation in historical Unixes.  And there
> are various tools which "expect" it (... and this is why they get
> created as executable by default by ld).  
> 
> Switching to INSTALL_LIB here although I think the whole "define all the
> ways you're going to use install" in this makefile a _smidge_ overkill
> 
> Signed-off-by: Jeremy Katz <katzj@xxxxxxxxxx>

I think that this patch is corret (well, perhaps execpt for the _smidge_ bit).
I believe that it is general practice for make install and the like to
install shared libraries as executables for the reason that you give
above. Though it is common practice for distros to remove the executable
bit as part of their packaging process - which explains why many
people won't see it when they go and check /usr/lib, but is of little
concern to this patch.

Acked: Horms <horms@xxxxxxxxxxxx>

-- 
Horms                                           
  H: http://www.vergenet.net/~horms/
  W: http://www.valinux.co.jp/en/


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

<Prev in Thread] Current Thread [Next in Thread>