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
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://bugs.launchpad.net/charms/+source/tracks/+bug/1155358

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://www.assembla.com/spaces/tracks-tickets/tickets/1272#/activity/ticket:

To post a comment you must log in.
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

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/
[2]: https://bugs.launchpad.net/charms/+source/tracks/+bug/1168520

review: Needs Fixing
Revision history for this message
Marius B. Kotsbak (mariusko) wrote :

Any progress here?

Revision history for this message
Jorge Castro (jorge) wrote :

This MP has been languishing, since it's abandoned I'll go ahead and close this out.

review: Needs Fixing

Unmerged revisions

5. By Matt Fischer

fixing the issue with has_many_polymorphs that prevented tracks from starting. A newer upstream fixed the problem.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/install'
2--- hooks/install 2013-02-21 03:31:54 +0000
3+++ hooks/install 2013-04-01 20:45:31 +0000
4@@ -3,6 +3,7 @@
5 # ToDo:
6 # - remove imagemagick?
7
8+VERSION=2.2.1
9 set -eux
10
11 juju-log "installing pre-reqs"
12@@ -12,11 +13,10 @@
13 imagemagick graphicsmagick libgraphicsmagick1-dev librmagick-ruby \
14 libxslt-dev libxml2-dev pwgen rubygems mysql-client wget
15
16-juju-log "installing tracks"
17-wget -O tracks.zip https://github.com/TracksApp/tracks/zipball/v2.1
18+juju-log "installing tracks v${VERSION}"
19+wget -O tracks.zip https://github.com/TracksApp/tracks/archive/v${VERSION}.zip
20 unzip tracks.zip
21-mv TracksApp-tracks-* tracks
22-cd tracks
23+cd tracks-${VERSION}
24 chmod -R 664 *
25 find -type d -exec chmod a+x '{}' \;
26 chmod -R a+x script/*

Subscribers

People subscribed via source and target branches

to all changes: