Juju Charms Collection
kibana package

Merge lp://staging/~evarlast/charms/trusty/kibana/fix-tarball-install-check into lp://staging/charms/trusty/kibana

  1. Trusty Tahr (14.04)
  2. fix-tarball-install-check
  3. Merge into trunk
Proposed by Jay R. Wren
Status: Merged
Merged at revision: 17
Proposed branch: lp://staging/~evarlast/charms/trusty/kibana/fix-tarball-install-check
Merge into: lp://staging/charms/trusty/kibana
Diff against target: 12 lines (+1/-1)
1 file modified
hooks/install (+1/-1)
To merge this branch: bzr merge lp://staging/~evarlast/charms/trusty/kibana/fix-tarball-install-check
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+255380@code.staging.launchpad.net

Description of the change

Fix a bug introduced in https://code.launchpad.net/~canonical-ci-engineering/charms/trusty/kibana/local-tarball/+merge/254638

[! isn't valid bash, a space is needed, and since its bash, may as well use [[ and ]] to prevent forking /usr/bin/[

To post a comment you must log in.
Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM Thanks for the catch!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches