Merge lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 into lp:ubuntu/precise/lincity-ng
Proposed by
Aditya V
Status: | Rejected |
---|---|
Rejected by: | Martin Pitt |
Proposed branch: | lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 |
Merge into: | lp:ubuntu/precise/lincity-ng |
Diff against target: |
26 lines (+7/-1) 2 files modified
debian/changelog (+6/-0) lincity-ng.desktop (+1/-1) |
To merge this branch: | bzr merge lp:~kroq-gar78/ubuntu/precise/lincity-ng/fix-674519 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel Holbach (community) | Disapprove | ||
Ubuntu branches | Pending | ||
Review via email: mp+99151@code.launchpad.net |
Description of the change
I fixed the typo "electricityand" to "electricity and" in lincity-ng.desktop.
To post a comment you must log in.
Thanks a lot for your work on this.
A few small bits I noticed:
- the package uses the quilt patch system, so it would be nice if you
could use edit-patch to make modifications to the actual source code
- as your change modifies source code (it's not a mere rebuild), it
would make sense to change the revision in debian/changelog to
2.0-2ubuntu1 instead of 2.0-2build2.
The change is pretty small and maybe not worth creating a delta between Debian and Ubuntu, so I would suggest you forward the change to lincity-ng.desktop to Upstream. (As Upstream is moving from berlios to fedorahosted, it might be enough to just mail it to this mailing list.)
Thanks again for your work on this. I hope you agree with the suggested course of action. (I'll mark this as 'Disapprove' just to indicate that we don't merge this as-is, but instead forward to Upstream and let the change trickle back from there.)