Juju Charms Collection
wordpress package

Code review comment for lp:~dave-cheney/charms/precise/wordpress/trunk

  1. Precise Pangolin (12.04)
  2. trunk
  3. Merge into trunk
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

This looks great, though you've removed the save_pwd bits, which is fine, but the hook needs to cd back to $CHARM_DIR or other hooks will fail when they call hooks/install inline. If you could add the cd back to $CHARM_DIR at the bottom of the hooks/install then I can merge this.

review: Needs Fixing

« Back to merge proposal