Juju Charms Collection
mysql package

Merge lp://staging/~mbruzek/charms/precise/mysql/addescapes into lp://staging/charms/mysql

  1. Precise Pangolin (12.04)
  2. addescapes
  3. Merge into trunk
Proposed by Matt Bruzek
Status: Merged
Merged at revision: 120
Proposed branch: lp://staging/~mbruzek/charms/precise/mysql/addescapes
Merge into: lp://staging/charms/mysql
Diff against target: 21 lines (+2/-2)
1 file modified
hooks/master-relation-changed (+2/-2)
To merge this branch: bzr merge lp://staging/~mbruzek/charms/precise/mysql/addescapes
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+215239@code.staging.launchpad.net

Description of the change

I noticed that on a departed relation the mysql command was not properly escaped. Fixing that with this MP.

To post a comment you must log in.
Revision history for this message
Matt Bruzek (mbruzek) wrote :

I used the mysql command 8 lines above to get the proper escape syntax.

Revision history for this message
Charles Butler (lazypower) wrote :

Thanks for the submission Matt,

I've deployed the requested change and validated it works. The merge will show up in the store approx 15 to 30 minutes post merge.

If you have any questions/comments/concerns about the review contact us in #juju on irc.freenode.net or email the mailing list <email address hidden>

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

to all changes: