Project

General

Profile

Actions
Copy link

Bug #2628

closed

Missing fusioninventory-agent in rudder-agent package

Added by François ARMAND over 12 years ago. Updated almost 10 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Matthieu CERDA
Category:
Packaging
Target version:
2.4.0~beta3
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

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.

Actions
Copy link
 #1

Updated by Jonathan CLARKE over 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?

Actions
Copy link
 #2

Updated by Matthieu CERDA over 12 years ago

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

Actions
Copy link
 #3

Updated by Matthieu CERDA over 12 years ago

  • Status changed from 2 to In progress
Actions
Copy link
 #4

Updated by Michael Gliwinski over 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/
Actions
Copy link
 #5

Updated by Jonathan CLARKE over 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

Actions
Copy link
 #6

Updated by Jonathan CLARKE over 12 years ago

  • Target version changed from 50 to 2.4.0~beta3
Actions
Copy link
 #7

Updated by Matthieu CERDA over 12 years ago

  • Status changed from In progress to Pending technical review
  • % Done changed from 0 to 100

Applied in changeset commit:ee78d03880bcf2884bffcc798e9ebb431dc30376.

Actions
Copy link
 #8

Updated by Nicolas PERRON over 12 years ago

It looks good to me.

Actions
Copy link
 #10

Updated by Jonathan CLARKE over 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.

Actions
Copy link
 #11

Updated by Nicolas PERRON almost 12 years ago

  • Project changed from Rudder to 34
  • Category deleted (11)
Actions
Copy link
 #12

Updated by Benoît PECCATTE almost 10 years ago

  • Project changed from 34 to Rudder
  • Category set to Packaging
Actions
Copy link

Also available in: Atom PDF