Rudder

Well, I just built a rudder-agent package and indeed fusioninventory-agent is not here anymore... I'll investigate.

Current change (setting PREFIX to $TMP/...) causes fusion to look for things in the build dir at runtime. Here's a patch that makes it use the correct prefix but still installs it into $TMP/perl:

--- a/rudder-agent/SOURCES/perl-prepare.sh
+++ b/rudder-agent/SOURCES/perl-prepare.sh
@@ -138,8 +140,8 @@ done

 cd $PWD/../../fusioninventory-agent
 mkdir -p $TMP/perl$PERL_PREFIX/share/fusion-utils && cp -a share/* $TMP/perl$PERL_PREFIX/share/fusion-utils
-PERL_MM_USE_DEFAULT=1 $TMP/perl$PERL_PREFIX/bin/perl Makefile.PL --default PREFIX=$TMP/perl$PERL_PREFIX
-$MAKE install
+PERL_MM_USE_DEFAULT=1 $TMP/perl$PERL_PREFIX/bin/perl Makefile.PL --default PREFIX=$PERL_PREFIX
+$MAKE install DESTDIR=$TMP/perl

 cp -a lib/FusionInventory $TMP/perl$PERL_PREFIX/lib/perl5/

Michael Gliwinski wrote:

Current change (setting PREFIX to $TMP/...) causes fusion to look for things in the build dir at runtime. Here's a patch that makes it use the correct prefix but still installs it into $TMP/perl:

Hi Michael,

Thank you very much for contributing this patch. It's good to see!

Before commiting it, can I ask you to sign a CLA? See www.rudder-project.org/HowToContribute for the actual document, and the reasons why we ask. In short, even though this is just a small patch, it's all about keeping things clean.

Thanks!
Jonathan

It looks good to me.