Bug #2628
closedMissing fusioninventory-agent in rudder-agent package
Description
In rudder-agent_2.5.0~alpha1~git201207041034-squeeze0_i386.deb (last generted package to date), the file /opt/rudder/bin/fusioninventory-agent is missing, what make inventory generation impossible.
Updated by Jonathan CLARKE about 12 years ago
- Status changed from New to 2
- Assignee changed from Jonathan CLARKE to Matthieu CERDA
That's weird! I wonder if it's related to the recent version change in FusionInventory.
Matthieu, can you look into this please?
Updated by Matthieu CERDA about 12 years ago
Well, I just built a rudder-agent package and indeed fusioninventory-agent is not here anymore... I'll investigate.
Updated by Matthieu CERDA about 12 years ago
- Status changed from 2 to In progress
Updated by Michael Gliwinski about 12 years ago
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/
Updated by Jonathan CLARKE about 12 years ago
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
Updated by Jonathan CLARKE about 12 years ago
- Target version changed from 50 to 2.4.0~beta3
Updated by Matthieu CERDA about 12 years ago
- Status changed from In progress to Pending technical review
- % Done changed from 0 to 100
Applied in changeset commit:ee78d03880bcf2884bffcc798e9ebb431dc30376.
Updated by Jonathan CLARKE almost 12 years ago
- Status changed from Pending technical review to Released
Nicolas PERRON wrote:
It looks good to me.
I'll have to trust you on this one, I don't fully understand the change, but I know it works.
Updated by Nicolas PERRON over 11 years ago
- Project changed from Rudder to 34
- Category deleted (
11)
Updated by Benoît PECCATTE over 9 years ago
- Project changed from 34 to Rudder
- Category set to Packaging