Merge lp://staging/~abentley/charms/precise/juju-reports/full-path-ini into lp://staging/~juju-qa/charms/precise/juju-reports/trunk

Proposed by Aaron Bentley
Status: Merged
Merged at revision: 28
Proposed branch: lp://staging/~abentley/charms/precise/juju-reports/full-path-ini
Merge into: lp://staging/~juju-qa/charms/precise/juju-reports/trunk
Diff against target: 29 lines (+4/-4)
2 files modified
hooks/common.py (+1/-1)
templates/crontab.tmpl (+3/-3)
To merge this branch: bzr merge lp://staging/~abentley/charms/precise/juju-reports/full-path-ini
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+219228@code.staging.launchpad.net

Description of the change

Always use the production.ini in the project root. Otherwise, scripts invoked from the crontab will use /lib/python2.7/site-packages/jujureports-0.0-py2.7.egg/production.ini

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you.

review: Approve (code)

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: