Ian Campbell
2013-Feb-05  14:08 UTC
[PATCH] osstest: ts-xen-install: Run ldconfig after install
This appears to be needed when libraries are outside /lib and /usr/lib
(specifically when they are in /usr/local/lib)
Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
---
 ts-xen-install |    1 +
 1 file changed, 1 insertion(+)
diff --git a/ts-xen-install b/ts-xen-install
index 5ee97af..cafc71a 100755
--- a/ts-xen-install
+++ b/ts-xen-install
@@ -45,6 +45,7 @@ sub extract () {
         target_extract_jobdistpath($ho, $part, "path_${part}dist",
 				   $r{"${part}buildjob"}, \%distpath);
     }
+    target_cmd_root($ho, ''/sbin/ldconfig'');
 }
 
 sub adjustconfig () {
-- 
1.7.10.4
Ian Jackson
2013-Feb-05  18:23 UTC
Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install
Ian Campbell writes ("[PATCH] osstest: ts-xen-install: Run ldconfig after
install"):> This appears to be needed when libraries are outside /lib and /usr/lib
> (specifically when they are in /usr/local/lib)
I have applied this and pushed it to osstest''s equivalent of staging.
Ian.
Ian Campbell
2013-Feb-06  08:26 UTC
Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install
On Tue, 2013-02-05 at 18:23 +0000, Ian Jackson wrote:> Ian Campbell writes ("[PATCH] osstest: ts-xen-install: Run ldconfig after install"): > > This appears to be needed when libraries are outside /lib and /usr/lib > > (specifically when they are in /usr/local/lib) > > I have applied this and pushed it to osstest''s equivalent of staging.Thanks, we''ll probably want that to graduate before we reconsider the "install in /usr/local" patches. BTW, I think you were going to be merging up your standalone branch into osstest master, shall I rebase yet? Ian.
Ian Campbell
2013-Feb-06  08:32 UTC
Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install
On Wed, 2013-02-06 at 08:26 +0000, Ian Campbell wrote:> On Tue, 2013-02-05 at 18:23 +0000, Ian Jackson wrote: > > Ian Campbell writes ("[PATCH] osstest: ts-xen-install: Run ldconfig after install"): > > > This appears to be needed when libraries are outside /lib and /usr/lib > > > (specifically when they are in /usr/local/lib) > > > > I have applied this and pushed it to osstest''s equivalent of staging. > > Thanks, we''ll probably want that to graduate before we reconsider the > "install in /usr/local" patches. > > BTW, I think you were going to be merging up your standalone branch into > osstest master, shall I rebase yet?Oh and BTW2, did you see the other standalone fixlets in <1357903462.20328.35.camel@zakaz.uk.xensource.com> ? Ian.
Ian Jackson
2013-Feb-06  11:03 UTC
Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install
Ian Campbell writes ("Re: [Xen-devel] [PATCH] osstest: ts-xen-install: Run
ldconfig after install"):> On Wed, 2013-02-06 at 08:26 +0000, Ian Campbell wrote:
> > BTW, I think you were going to be merging up your standalone branch
into
> > osstest master, shall I rebase yet?
No, I''m afraid.  I''ve merged the two together but I
haven''t pushed the
result into staging yet.  The merge was nontrivial so you''d best wait
for it to be properly tested.
> Oh and BTW2, did you see the other standalone fixlets in
> <1357903462.20328.35.camel@zakaz.uk.xensource.com> ?
I haven''t merged those yet - thanks for reminding me.
Ian.
Ian Campbell
2013-Feb-06  11:09 UTC
Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install
On Wed, 2013-02-06 at 11:03 +0000, Ian Jackson wrote:> Ian Campbell writes ("Re: [Xen-devel] [PATCH] osstest: ts-xen-install: Run ldconfig after install"): > > On Wed, 2013-02-06 at 08:26 +0000, Ian Campbell wrote: > > > BTW, I think you were going to be merging up your standalone branch into > > > osstest master, shall I rebase yet? > > No, I''m afraid. I''ve merged the two together but I haven''t pushed the > result into staging yet. The merge was nontrivial so you''d best wait > for it to be properly tested.ACK.> > Oh and BTW2, did you see the other standalone fixlets in > > <1357903462.20328.35.camel@zakaz.uk.xensource.com> ? > > I haven''t merged those yet - thanks for reminding me.NP. Ian.