search

sydro / atom-ansible-vault

Atom package to create and modify ansible-vault file
MIT License
32 stars 10 forks source link

[ERROR] ansible-vault "command not found" #22

Closed ecwpz91 closed 7 years ago

ecwpz91 commented 7 years ago

Problem

Path name resolution for Ansible Value absolute path default package setting, /usr/local/bin/ansible-vault, does not exist.

Message

ERROR: ansible-vault not found using path in package settings. Ansible-vault will be deactivate!

/bin/sh: /usr/local/bin/ansible-vault: No such file or directory

Bypass

  1. Create symlink to missing reference file:
# ln -s /usr/bin/ansible-vault /usr/local/bin/ansible-vault
  1. Manually update Atom package settings.

Release

NAME=Fedora
VERSION="26 (Workstation Edition)"
ID=fedora
VERSION_ID=26
PRETTY_NAME="Fedora 26 (Workstation Edition)"
ANSI_COLOR="0;34"
CPE_NAME="cpe:/o:fedoraproject:fedora:26"
HOME_URL="https://fedoraproject.org/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="Fedora"
REDHAT_BUGZILLA_PRODUCT_VERSION=26
REDHAT_SUPPORT_PRODUCT="Fedora"
REDHAT_SUPPORT_PRODUCT_VERSION=26
PRIVACY_POLICY_URL=https://fedoraproject.org/wiki/Legal:PrivacyPolicy
VARIANT="Workstation Edition"
VARIANT_ID=workstation
sydro commented 7 years ago

Hi, this is not a bug. You need set ansible-vault binary path in package settings. Let see screenshot https://raw.githubusercontent.com/sydro/atom-ansible-vault/master/images/screenshot-settings.png.

ecwpz91 commented 7 years ago

Yes. I already said that under Bypass section above.

But, I could've been better about explaining my situation.

This happens during first-time installation.

My immediate experience after downloading was, pop-up - "ERROR".

How do I know if this is expected behavior or not?

In fact, my initial response was: "This package is broken. Uninstall."

Since I value your tool, my suggestion(s) are:

  1. Check for Cross-Platform Compatibility, and use this to handle setting the default Ansible Value absolute path to a valid reference file location on target environments.
  2. Document expected behavior of all configurable settings.
ecwpz91 commented 7 years ago

You could label this as an "enhancement" as well?

Anyway, hope that helps clarify some things.

sydro commented 7 years ago

Thanks, i think that i will change the error message with settings reference.

sydro commented 7 years ago

I changed the error message with reference to this repository and updated the README.md with little description of any thing that you can set.

ecwpz91 commented 7 years ago

+1, looks great. Thanks! On a side note, you may want to add documentation guidelines for repo contribution in case I, or someone else, would like to issue a new statement in the future.

sydro commented 7 years ago

I added CONTRIBUTING.md file with essential guidelines as you suggested! Thanks :+1: