Project

General

Profile

Actions
Copy link

Bug #5066

closed

Missing dependency on which

Bug #5066: Missing dependency on which

Added by Lionel Le Folgoc about 12 years ago. Updated over 11 years ago.

Status:
Released
Priority:
1 (highest)
Assignee:
Jonathan CLARKE
Category:
Packaging
Target version:
2.11.0~beta2
Pull Request:
https://github.com/Normation/rudder-p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

Hi again again,

In rudder-agent-thin (maybe rudder-agent too), /opt/rudder/bin/run-inventory calls the "which" command:

[... blablabla]
if [ -x /opt/rudder/bin/perl ]; then
[...]
elif which fusioninventory-agent > /dev/null 2>&1; then
[...]
else

  # No usable FusionInventory installation has been detected, bail out.
  echo "[ERROR] No usable FusionInventory instance (either from Rudder or the system) detected." 
  exit 1

fi

The test was failing here because the "which" package wasn't installed. Maybe it's worth adding a dependency?

Thanks.

Updated by Lionel Le Folgoc about 12 years ago Actions
Copy link
 #1

RHEL-6, as usual. ;-)

Updated by Matthieu CERDA about 12 years ago Actions
Copy link
 #2

  • Status changed from New to In progress
  • Assignee set to Matthieu CERDA
  • Priority changed from N/A to 1 (highest)
  • Target version set to 2.11.0~beta2

Taking this.

Updated by Matthieu CERDA about 12 years ago Actions
Copy link
 #3

  • Status changed from In progress to Pending technical review
  • Assignee changed from Matthieu CERDA to Jonathan CLARKE
  • % Done changed from 0 to 100
  • Pull Request set to https://github.com/Normation/rudder-packages/pull/384

PR is ready, used "command" POSIX builtin instead of which, see => http://stackoverflow.com/questions/592620/how-to-check-if-a-program-exists-from-a-bash-script

Updated by Matthieu CERDA about 12 years ago Actions
Copy link
 #4

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:a08c459316844906708d323906dabd32d30aabb1.

Updated by Jonathan CLARKE about 12 years ago Actions
Copy link
 #5

Applied in changeset commit:cb8c7cbd84b9427a067645e98813ad531b8b2f41.

Updated by Vincent MEMBRÉ about 12 years ago Actions
Copy link
 #6

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 2.11.0~beta2 (announcement , changelog), which were released today.

Updated by Benoît PECCATTE over 11 years ago Actions
Copy link
 #7

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

Also available in: PDF Atom