[PATCH] osstest: ts-xen-install: Run ldconfig after install

[PATCH] osstest: ts-xen-install: Run ldconfig after install

[Ian Campbell]

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

Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install

[Ian Jackson]

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.

Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install

[Ian Campbell]

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.

Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install

[Ian Campbell]

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.

Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install

[Ian Jackson]

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.

Re: [PATCH] osstest: ts-xen-install: Run ldconfig after install

[Ian Campbell]

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.