Merge lp:~mfisch/charms/precise/tracks/trunk into lp:charms/tracks
Proposed by
Matt Fischer
Status: | Rejected |
---|---|
Rejected by: | Jorge Castro |
Proposed branch: | lp:~mfisch/charms/precise/tracks/trunk |
Merge into: | lp:charms/tracks |
Diff against target: |
26 lines (+4/-4) 1 file modified
hooks/install (+4/-4) |
To merge this branch: | bzr merge lp:~mfisch/charms/precise/tracks/trunk |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jorge Castro (community) | Needs Fixing | ||
Marco Ceppi (community) | Needs Fixing | ||
Review via email: mp+156418@code.launchpad.net |
Description of the change
Fix for https:/
The issue was that tracks bundled an ancient gem that had links to keyfiles on the original author's website, the files were moved or removed and the gem broke. According to discussions upstream the charm was "unmaintainable" and therefore removed in a recent version.
https:/
To post a comment you must log in.
Unmerged revisions
- 5. By Matt Fischer
-
fixing the issue with has_many_polymorphs that prevented tracks from starting. A newer upstream fixed the problem.
Hi Matt! Thanks for the submission and sorry for the delay. Your changes look good for this (everything installs and moves to a started state). However, when I added the MySQL connection a relation error occurred[1]. Could you look into why this happened and repair? I understand it's broken in it's current state, but this merge (while addresses that issue) appears to break the charm elsewhere. Also, during my investigation I noticed you don't have a config-changed hook and opened this bug[2] to have that rectified but it's not a blocker for getting this merge done.
When you're ready for review make sure to re-assign "charmers" as the reviewer for it to appear back in the queue again.
[1]: http:// paste.ubuntu. com/5702667/ /bugs.launchpad .net/charms/ +source/ tracks/ +bug/1168520
[2]: https:/