Skip to content

Improve fpm detection

Pedro Pombeiro requested to merge fix-fpm-install into master

What does this MR do?

I added && fpm --help 1>/dev/null to the package-deps target. Also replaced non-standard which command with command -v.

Why was this MR needed?

I ran into issues on my system, where I did have an fpm symlink, but it was a non-working one. This way the script will detect that situation and reinstall the gem if needed.

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Added tests for this feature/bug
  • In case of conflicts with master - branch was rebased

What are the relevant issue numbers?

Merge request reports